[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Rapp-dev] Pixelwise threshold and conditional pixel operations path
From: |
Willie Betschart |
Subject: |
Re: [Rapp-dev] Pixelwise threshold and conditional pixel operations pathes |
Date: |
Thu, 19 May 2016 13:52:54 +0000 |
Hi H-P
Well, the patches with pixelwise threshold and conditional pixop are have been
following the guidelines. Would you like a single patch with both solutions?
The older patch with rapp_validate_buffer was probably not that strict with
whitespaces. I have forgotten that patch, that was intended as a nice to have
feature.
I'm fine with not integrating that. I expected comments or other ideas on that
work before it should considered as finished.
Best regards
Willie
________________________________________
Från: Hans-Peter Nilsson <address@hidden>
Skickat: den 19 maj 2016 14:19
Till: Willie Betschart
Kopia: address@hidden; address@hidden
Ämne: Re: [Rapp-dev] Pixelwise threshold and conditional pixel operations pathes
A 36 times improvement is impressive, thanks.
I'm looking at these and earlier patches right now. It'll take
a little longer than expected to weed out unnecessary
reformatting (only white-space changes) in existing code and
unnecessary inconsistencies in formatting (both regarding rules
and within the submitted code) in your patch. Those just make
it harder to spot typos and review the *intended* changes.
Please make sure to read and follow HACKING.
Your earlier rapp_validate_buffer submission will go in with
this, and we'll make a release in days after that. I'm on the
fence regarding including the vectorization of integral
functions submitted even before that; that'll probably be left
out at this time as it'll have to be cleaned up and ISTR the
performance improvement wasn't a priority.
Thanks also to Johan for the functional review.
brgds, H-P