diff options
author | Travis Reitter <travis.reitter@collabora.co.uk> | 2010-12-28 17:21:53 -0800 |
---|---|---|
committer | Travis Reitter <travis.reitter@collabora.co.uk> | 2010-12-29 17:19:07 -0800 |
commit | dc3acdfca04a03f00f0fb04d375fa6946145ce4b (patch) | |
tree | 875ed7a826f8fdbe83e8808e6f145dc2732bfac3 /HACKING | |
parent | 6aa04aae7ffda13afe2654a8a3e6b1d9c42c4ff3 (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-- | HACKING | 6 |
1 files changed, 6 insertions, 0 deletions
@@ -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. |