[Gegl-developer] Small patch

Henrik Akesson h.m.akesson at gmail.com
Thu Mar 12 03:16:04 PDT 2009


Hi,

here's a small patch on gegl-processor.c

I've kept it very small in order to get some feedback on what I'm doing.

The patch contains:
- comments
- refactoring of the render_rectangle function: extraction of code into a
separate function "get_band_size". No functional changes.
- debug statements (well a few)

I would especially like to get some feedback on the extraction (refactoring)
of methods. There are some functions with about 300 lines, wich I wouldn't
mind breaking up into smaller functions to make it easier to understand
them.

Thanks,

Henrik
-------------- next part --------------
An HTML attachment was scrubbed...
URL: /lists/gegl-developer/attachments/20090312/c872ae17/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gegl-processor.diff
Type: application/octet-stream
Size: 14656 bytes
Desc: not available
Url : /lists/gegl-developer/attachments/20090312/c872ae17/attachment.obj 


More information about the Gegl-developer mailing list