[vlc-devel] [Patches] Dash: mostly cosmetics

Tobias Güntner fatbull at web.de
Tue Nov 29 13:13:39 CET 2011


Am 29.11.2011 11:55, schrieb Hugo Beauzée-Luyssen:
> On Tue, Nov 29, 2011 at 7:02 AM, Tobias Güntner<fatbull at web.de>  wrote:
>> Am 28.11.2011 15:12, schrieb Hugo Beauzée-Luyssen:
>>>
>>> I agree, but my plan was to remove all the exception, and therefore
>>> the exceptions specifications.
>>
>> If you are going to remove *all* exceptions, you cannot use "new" or STL
>> containers any more because those things can throw exceptions. You would
>> have to rewrite almost everything. Is this really what you want?
>>
>> Probably not. You cannot ignore exceptions either, because unhandled
>> exceptions will terminate() the program. So you *have* to catch them. And
>> since you have to pay for exception handling anyway, you might as well use
>> it.
>>
>> Regards,
>> Tobias
>>
>
> I disagree

Well, I don't see an alternative. Did I miss anything?

> but I don't think there much room for discussion since this
> is your code. If you don't want it to be changed, then it won't be,
> end of the story.

You're mistaking me for someone else. It's not my code. ;) I'm just 
arguing for proper exception handling.

> Regards,

Regards,
Tobias




More information about the vlc-devel mailing list