Re: Small patch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
_______________________________________________
Gegl-developer mailing list
Gegl-developer@xxxxxxxxxxxxxxxxxxxxxx
https://lists.XCF.Berkeley.EDU/mailman/listinfo/gegl-developer

[Index of Archives]     [Yosemite News]     [Yosemite Photos]     [gtk]     [GIMP Users]     [KDE]     [Gimp's Home]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux