fix bug of opencv memory leak, remove logs

muesli / smartcrop

smartcrop finds good image crops for arbitrary crop sizes

MIT License

1.82k stars 114 forks source link

fix bug of opencv memory leak, remove logs #25

Closed chideat closed 7 years ago

chideat commented 7 years ago

Two place used opencv, but not release the object. And the FindBestCrop func outputs a log useless logs, so I cleaned them all.

muesli commented 7 years ago

Thanks for the PR! I feel like this should have been two separate pull requests, though. Any chance you can open one for the opencv fixes and one for the logging (see my extra comment on that).

chideat commented 7 years ago

Sorry, I'll resubmit two requests.