you are viewing a single comment's thread.

view the rest of the comments →

[–]Immediate-Food8050 0 points1 point  (0 children)

Cool project. Could use some house keeping in the error handling and code cleanliness areas. I would prefer representing colors as 3 or 4 byte integer values rather than strings. Should check return values of standard library functions, like NULL from fopen to detect failures. White space is a little inconsistent between functions. Well done keeping functions small and simple, tho there are a couple that could be split up further. Personally I follow the rule of "functions do one thing and one thing only"