[haiku-commits] Re: r40800 - haiku/trunk/src/add-ons/translators/raw

  • From: Ryan Leavengood <leavengood@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 4 Mar 2011 13:35:49 -0500

On Fri, Mar 4, 2011 at 1:19 PM, Axel Dörfler <axeld@xxxxxxxxxxxxxxxx> wrote:
>
> Well, it might, but investigating the issue certainly makes sense.

I'll take a look when I get a chance, maybe tonight.

> I don't remember if I just copied that code from somewhere, or if I
> even wrote it myself - but I agree with Ingo that it's rather poor :-)

So my suggested changes seem OK? It doesn't change much beyond fixing
any future type conversion issues. I'll put the comparision back to >
0 and try to fix the real problem in the JPEGTranslator.

> I just wonder why changing something in the RAW decoder makes a JPEG
> file readable.

Because that header is used here:

http://dev.haiku-os.org/browser/haiku/trunk/src/add-ons/translators/jpeg/exif_parser.cpp

-- 
Regards,
Ryan

Other related posts: