[vlc-devel] commit: Force new not to throw. Also drop a separate declaration of a static function that' s called exactly once, and mark it inline instead. (JP Dinger )

Derk-Jan Hartman hartman at videolan.org
Mon Jun 22 19:48:29 CEST 2009


On 22 jun 2009, at 19:38, Rémi Denis-Courmont wrote:
> Le lundi 22 juin 2009 20:13:20 jpd at videolan.org, vous avez écrit :
>> On Mon, Jun 22, 2009 at 07:57:04PM +0300, R?mi Denis-Courmont wrote:
>>> You're visibly leaking a lock here. Maybe some other stuff.
>>
>> Fixed in the very next commit. The standing alternative was a throw
>> back into C code, though. Take your pick.
>
> I don't care... I just happen to review commits in chronological  
> order. My
> short-term memory stack space is very finite, so that recursively  
> unwinding
> review is not really an option.

ROFL, nerd :D

DJ


More information about the vlc-devel mailing list