[vlc-devel] [PATCH] macosx: add verbosity setting to messages window

[Felix Paul Kühne]

On 22.04.2012, at 21:20, Brendon Justin wrote:

> Analogous to the verbosity chooser in the Qt intf messages window.

Thanks for your patch. Regrettably, it doesn't apply for me no matter what I do. Additionally, I need to kvetch about the coding style: the integer (why no NSInteger?) misses the i_ prefix, which we still use throughout the macosx interface module to keep the code more readable.

Anyway, why do you need or want this feature? Just to match the Qt4 interface? [1]

I don't really see the use case here, since if a user opens the messages window (most aren't aware of its existence), it is fair to assume that s/he would like to see all the messages. To see the errors, you don't need the messages, since most of them are presented through dialogs and most warnings are rather ambiguous without the debug messages.

Best regards,

Felix


[1] j-b is going to remove these GUI elements from the Qt4 interface shortly btw.