Formatting

• Two main schools:
  • Opening brace on separate line (GNU, Linux, Windows):
        if (ia->ia_maddr != MADDRUNK)
        {
            sc->pPacketPagePhys = ia->ia_maddr;
        }
    
  • Opening brace on same line (BSD Java)
        if ((cp = strchr(*argv, ':')) != NULL) {
            *cp++ = '\0';
            a_gid(cp);
        }
    
• Both are OK
• Inconsitency is a problem
  xpupil.y = Xy(w->eyes.pupil[0].x, w->eyes.pupil[0].y, &w->eyes.t);
  xnewpupil.x =  Xx(newpupil[0].x, newpupil[0].y, &w->eyes.t);
  xnewpupil.y =  Xy(newpupil[0].x, newpupil[0].y, &w->eyes.t);
  if (!XPointEqual (xpupil, xnewpupil)) {
  
• Why? All explanations spell trouble:
  • Undisciplined programmer
  • Haphazard integration
  • Uncoordinated team

• PreviousIndentation
• Contents
• NextComments

 Last change: Thursday, April 15, 2004 11:16 am
 Unless otherwise expressly stated, all original material on this page created by Diomidis Spinellis is licensed under a Creative Commons Attribution-Share Alike 3.0 Greece License.