Greetings. While trying to determine if I found a bug in the GIMP (or possibly undocumented behaviour), I ran across what looks like a bit of redundant code. In the file app/pdb/convert_cmds.c around line 153 is an if statement that checks for a valid range for dither_type. Later (around line 175) is a switch statement that also appears to be checking that the dither_type is one of four allowed values. It seems to me that the switch is not needed. -- Cheers! Kevin. (http://www.interlog.com/~kcozens/) Owner of Elecraft K2 #2172 |"What are we going to do today, Borg?" E-mail:kcozens at interlog dot com|"Same thing we always do, Pinkutus: Packet:ve3syb@ve3yra.#con.on.ca.na| Try to assimilate the world!" #include <disclaimer/favourite> | -Pinkutus & the Borg
