[haiku-commits] Re: haiku: hrev50886 - in src: add-ons/media/media-add-ons/multi_audio kits/media/experimental
- From: Axel Dörfler <axeld@xxxxxxxxxxxxxxxx>
- To: haiku-commits@xxxxxxxxxxxxx
- Date: Fri, 20 Jan 2017 23:18:46 +0100
Am 20/01/2017 um 23:02 schrieb Dario Casalinuovo:
> Yes, the semaphore is not needed at this place, and it's a good idea
> and a good optimization to remove it.
Why asking to remove a "bogus" commit then?
Because you didn't do that correctly, maybe?
I've been going to commit a patch to fix both issues BTW, but as long as
it works that way...
Fixes are always welcome, commits that potentially break things, and
make code worse aren't. Revert, fix, recommit -- if you can fix it
quickly, you can save the revert.
Bye,
Axel.
Other related posts: