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