[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: stencil-integrate.cc: root out "slopes" (issue 246590043 by address@
From: |
lemzwerg |
Subject: |
Re: stencil-integrate.cc: root out "slopes" (issue 246590043 by address@hidden) |
Date: |
Tue, 30 Jun 2015 05:55:51 +0000 |
Oh, I haven't meant that you should reformat everything but just the
code you are actually modifying! But thanks anyway.
For long names I suggest a formatting like this
Grob::maybe_pure_internal_simple_skylines_from_extents(
foo arg1,
bar arg2,
...)
to avoid exactly the problem you mention. On the other hand, such long
function names are overly descriptive IMHO. I think they don't really
increase readability of the code...
https://codereview.appspot.com/246590043/