Skip to content
This repository has been archived by the owner on Jun 23, 2020. It is now read-only.

Added a way to turn ON and OFF Logs, just define BF_DEBUG on the prefix ... #9

Closed
wants to merge 2 commits into from

Conversation

Julioacarrettoni
Copy link
Contributor

Added a way to turn ON and OFF Logs, just define BF_DEBUG on the prefix file or add it on the compile macro to turn it ON.

In the demo project I added it on the prefix file.

…delaying the set of the image to the UIImageView until the very last moment
if (![self needsBetterFace]) {
return;
[self setBetterFaceImage:image];
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, the setBetterFaceImage method should be moved at the first line in this function, because whatever you need BF or not, you should call setImage to let the UIImageView has the image instance to do other necessary things (such as showing the whole picture or saving it to your disk). You may hold the original image in the imageView instance to let it be easy to find.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems that I mixed the pull request, I was trying a little "hack" to avoid the "blink" problem, the image starts normal then blinks to the proper ace AFTER the face was detected.

I'll fix this pull request and after that I'll send you a link for you to try my fix.

Do you have another example to stress test the library? Preferable one with a UITableView.

Sent from my iPhone

On 17/01/2014, at 22:09, Croath Liu notifications@github.com wrote:

In bf/bf/BetterFaceClass/UIImageView+BetterFace.m:

 if (![self needsBetterFace]) {
  •    return;
    
  •    [self setBetterFaceImage:image];
    
    Actually, the setBetterFaceImage method should be moved at the first line in this function, because whatever you need BF or not, you should call setImage to let the UIImageView has the image instance to do other necessary things (such as showing the whole picture or saving it to your disk). You may hold the original image in the imageView instance to let it be easy to find.


Reply to this email directly or view it on GitHub.

@Julioacarrettoni
Copy link
Contributor Author

I'm back on my mac!
Yes, I just wanted to make a pull request for 214b038, let me fix that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants