summaryrefslogtreecommitdiff
path: root/HACKING
diff options
context:
space:
mode:
authorTravis Reitter <travis.reitter@collabora.co.uk>2010-12-28 17:21:53 -0800
committerTravis Reitter <travis.reitter@collabora.co.uk>2010-12-29 17:19:07 -0800
commitdc3acdfca04a03f00f0fb04d375fa6946145ce4b (patch)
tree875ed7a826f8fdbe83e8808e6f145dc2732bfac3 /HACKING
parent6aa04aae7ffda13afe2654a8a3e6b1d9c42c4ff3 (diff)
Refer to non-local variables and methods with full name.
The benefit here is to make it easier to tell when variables are local. Methods are referred to similarly for consistency. Fixes bgo#629083
Diffstat (limited to 'HACKING')
-rw-r--r--HACKING6
1 files changed, 6 insertions, 0 deletions
diff --git a/HACKING b/HACKING
index fdaa818..051a49c 100644
--- a/HACKING
+++ b/HACKING
@@ -100,3 +100,9 @@ Vala-specific rules
9. As in most languages, avoid casting. Casting is usually a sign of an error
which should be fixed and reduces readability.
+
+10. Refer to non-local variables and methods with their qualified name. Within a
+ class function, refer to private data members like 'this._foo' and foreign
+ package symbols like 'package_name.symbol'.
+
+ This makes scope immediately clear, helping readability.