Difference between revisions of "Talk:New code guidelines"

From Mumble Wiki
Jump to: navigation, search
m
m
Line 1: Line 1:
<Kissaki> what are constants? Any var I declare const?
 
 
 
<Kissaki> I would add {} to if() return; as well<br>
 
<Kissaki> I would add {} to if() return; as well<br>
 
removes a special <br>
 
removes a special <br>
 
and something may be added even to a return block
 
and something may be added even to a return block
 
<Kissaki> I don't like the look of //!. It makes it look important, where it is just a normal comment.<br>
 
instead ///
 
 
<Kissaki> I'd rather not "default: break;" in one line
 
  
 
<Kissaki> The constructor example code specifically shows why I would prefer opening braces in a new line. You can't easily distinguish where the head ends and the body block begins.
 
<Kissaki> The constructor example code specifically shows why I would prefer opening braces in a new line. You can't easily distinguish where the head ends and the body block begins.

Revision as of 17:29, 18 September 2013

<Kissaki> I would add {} to if() return; as well
removes a special
and something may be added even to a return block

<Kissaki> The constructor example code specifically shows why I would prefer opening braces in a new line. You can't easily distinguish where the head ends and the body block begins.

 Foo::Foo(int a, QObject *parent)
      : QObject(parent)
      , m_a(a) {
      doStuff();
  };