[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdebase/kicker
From:       Aaron J. Seigo <aseigo () kde ! org>
Date:       2005-03-03 22:04:47
Message-ID: 20050303220447.AB5281751E () office ! kde ! org
[Download RAW message or body]

CVS commit by aseigo: 

for people who can't be bothered to read the full document, but who still
commit directly to the code base anyways.


  M +36 -0     HACKING   1.7


--- kdebase/kicker/HACKING  #1.6:1.7
@@ -1,2 +1,38 @@
+The Short Story
+---------------
+Four space tabs, braces on their own lines, 80 character lines.
+Code should look something like this:
+
+QString ExtensionManager::uniqueId()
+{
+    QString idBase = "Extension_%1";
+    QString newId;
+    int i = 0;
+    bool unique = false;
+
+    while (!unique)
+    {
+        ++i;
+        newId = idBase.arg(i);
+
+        unique = true;
+        ExtensionList::iterator itEnd = _containers.end();
+        for (ExtensionList::iterator it = _containers.begin();
+             it != itEnd;
+             ++it)
+        {
+            if ((*it)->extensionId() == newId)
+            {
+                unique = false;
+                break;
+            }
+        }
+    }
+
+    return newId;
+}
+
+Full Explanation
+----------------
 As of 2004 kicker's codebase isn't the prettiest thing in the world. It's seen
 a lot of work over a number of years without any sort of consistent coding


[prev in list] [next in list] [prev in thread] [next in thread]