Crestez and I talked about this patch on #winehackers, and there are a few things that need to be changed (let's ignore the formatting changes here):
- There was code to avoid a crash when the desktop value read from the registry doesn't have an x in it for some reason (yes this does happen) but this patch deletes it - mode_to_label is for translated, UI strings, not for stuff to be put in the registry. It does say this in a comment but I can appreciate that it looks at first glance like pointlessly duplicated code. We should fix this, especially as right now winecfgs translation support is weak. But for now the replacement of a case statement with a mode_to_label call is incorrect.
thanks -mike
On Fri, 07 Jan 2005 16:44:11 +0200, Crestez Leonard wrote:
This patch makes a bunch of small winecfg fixes. Since most of them are just a few lines in length I don't think it's worth splitting them.
Sorry for forgetting the attachment.
Changelog: winecfg: fix add application (memory error) winecfg: fix apply button lighting up in audio tab winecfg: fix video tab(crash when checking desktop, apply button not lighting up winecfg: fix apply button in libraries tab, fix memory leak in parse_override, remove horrible abuse of macros in set_dllmode