[CinCV TNG] [PATCH] close_all should consistently return 0, not 1.

Nicola Ferralis feranick at hotmail.com
Tue Apr 7 22:28:26 CEST 2015


It's a matter of consistency across similar calls. My bad that it should 
have been the same since the beginning.

Nicola

On 4/7/15 4:25 PM, Johannes Sixt wrote:
> Am 07.04.2015 um 22:00 schrieb Nicola Ferralis:
>> Hi, I was wondering if there are any comments on this patch.
>>
>> Thanks.
>> Nicola
>>
>> On 3/31/15 4:52 PM, Nicola Ferralis wrote:
>>> The non-zero return value of close_all in audioesound and audioalsa
>>> is not consistent with similar calls in other audio*.C and video*.C
>>> files. close_all is called upon by parent functions that then return
>>> 0 themselves. If close_all returns non-zero, there may be issues
>>> where the event is consumed within close_all. This patch sets the
>>> returns to zero.
>>>
>>> This was a mistake I introduced in commit b7e5b1ae3a, based on wrong
>>> consideration of the role of close_all as an event.
>
> I don't see where the return value of close_all() is relevant. Where
> exactly is it inspected?
>
> At this point, it would be much better to convert all incarnations of
> the function to void rather than fixing a small consistency error that
> has no consequence.
>
> -- Hannes
>
> _______________________________________________
> Cinelerra mailing list
> Cinelerra at lists.cinelerra-cv.org
> http://lists.cinelerra-cv.org/cgi-bin/mailman/listinfo/cinelerra



More information about the Cinelerra mailing list