Merge remote branch 'origin/bug_530_add_dm800se_support'

[enigma2.git] / doc / RULES

diff --git a/doc/RULES b/doc/RULES
index ed75b15e20ebe50c935a7d3bb520c802375672f5..55aa4bf40e71d50c8940e22fd75a6c971003148c 100644 (file)
--- a/doc/RULES
+++ b/doc/RULES
@@ -94,7 +94,7 @@ An ascii file ends with \n, and preferrable not with other empty lines.
 
 That means: make sure the last line doesn't contain any characters, thanks.
 
-4.) usage of 'print'
+5.) usage of 'print'
 
 While it's great to dump out debug stuff, especially if your code can crash,
 expect your code to be stable at some point.
@@ -112,7 +112,15 @@ not nothing. Going trough all prints to find the offending one is definitely
 no fun. Something like "print 'obj', obj" is fine. Something like "print 
 'mySpecialPlugin actionmap is', actionMap" is even better.
 
-6.-99.) Threads are bad.
+6.) usage of 'import'
+
+Please avoid 'import *'.
+Use "from foo import bar" only if bar is clearly identifiable to belong to
+foo (e.g.: it's ok to "from Screens.MessageBox import MessageBox", but it's
+not ok to do "from os import path". Use "import os.path", then os.path.fnc.
+Of course "from os.path import *" is even worse.)
+
+7.-99.) Threads are bad.
 
 (Unless they are worker threads. And sleep()ing is definitely not working.
 So if you every having a thread which sleeps, you did something wrong. There