On 11 mai 2012, at 09:09, d...@gnu.org wrote: > > http://codereview.appspot.com/6201068/diff/1/lily/pure-from-neighbor-engraver.cc > File lily/pure-from-neighbor-engraver.cc (right): > > http://codereview.appspot.com/6201068/diff/1/lily/pure-from-neighbor-engraver.cc#newcode56 > lily/pure-from-neighbor-engraver.cc:56: in_same_column (Grob *g1, Grob > *g2) > On 2012/05/11 05:23:27, MikeSol wrote: >> On 2012/05/10 21:40:48, janek wrote: >>> That's problably the most stupid question ever, but why this doesn't > it begin >>> with Pure_from_neighbor_engraver:: ? I don't see it being special. > >> It doesn't need to be a class method - it'll never be called outside > of this >> file. It is just a small helper function. > > Then it should be declared static, right? >
Why would it be part of the class? It's just a small helper function - architecturally, I don't see it as being part of a class. There are small helper functions like this all over the code base. Cheers, MS _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel