Hi!

Thanks a lot for the patches!

Avoid-RMW  - seems legit :)

Correct-aperture - It is mostly a cosmetic change, and since it will break
lens assignments, I'd rather leave it be, since it is only an internal
identifier.

PNG-output: I struggled a bit making something that would work on libpng
both prior and after v1.5, where they did some type changes. Which libpng
version(s) have you testing against?

Profile-should-return-NULL: Nice spotted!

I will also get a look at your database suggestion, when I get the time to
go through it in detail - it's very good to have such detailed feedback!


Regards, Klaus Post

http://www.klauspost.com


On Mon, Jun 4, 2012 at 12:31 AM, John Sheu <
[email protected]> wrote:

> I was doing some header file cleanup for other things (also learning the
> RawStudio code layout along the way), and have a few misc fixes, as
> attached.
>
> Caveat: patch 2 (correct-max...) changes the lens identifier string, and
> might cause a "new lens" to show up in users' lens DBs.
>
> -John Sheu
>
> _______________________________________________
> Rawstudio-dev mailing list
> [email protected]
> http://rawstudio.org/cgi-bin/mailman/listinfo/rawstudio-dev
>
>
_______________________________________________
Rawstudio-dev mailing list
[email protected]
http://rawstudio.org/cgi-bin/mailman/listinfo/rawstudio-dev

Reply via email to