[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [45305] trunk/blender/source/blender/bmesh /operators/bmo_dupe.c: Minor code cleanups for bmo_dupe.c.

Nicholas Bishop nicholasbishop at gmail.com
Sun Apr 1 09:37:08 CEST 2012


Ah, I had thought we always put if/for/while/switch opening brace on
the same line as the conditional. I don't care either way either, but
if it's something you feel strongly about then the coding style doc
should probably be more explicit on this point, e.g. add section on
braces that mentions opening braces go on same line, except with
multi-line conditionals.

Regarding <120 column lines, I know the style doc specifies 120
columns as the max, but is there objection to splitting further where
convenient? I try to keep most of my code down to 80 columns, since
that's pretty standard everywhere, and I can put multiple source
buffers side-by-side. Again, don't mind changing it if you don't like
this?

I can commit these edits back tomorrow, unless you get there first :)

-Nicholas


More information about the Bf-committers mailing list