[Gegl-developer] Small patch

Martin Nordholts enselic at gmail.com
Thu Mar 12 09:44:11 PDT 2009


Sven Neumann wrote:
> Splitting the code into smaller functions is cool. But please consider
> to name the functions according to the file it lives in. So this would
> become gegl_processor_get_band_size(). This makes it easier to interpret
> stack traces and to discuss the code as it is clear where the function
> is located.

Let's not forget the IMO most important aspect: being able to set
breakpoints based on function name. If one does not namespace static
functions it is likely that a project ends up with many different
variants of the same static function. If it's a virtual method it's as
good as inevitable.

- Martin


More information about the Gegl-developer mailing list