View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001027||OpenMPT||[All Projects] General||public||2017-09-21 12:28||2018-02-16 21:37|
|Product Version||OpenMPT 1.27.01.00 / libopenmpt 0.3.1 (upgrade first)|
|Target Version||OpenMPT 1.28.01.00 (goals)||Fixed in Version|
|Summary||0001027: Remove various WCHAR hacks for !UNICODE|
There are various hacks in the codebase that were only needed for unicode filename support in !unicode builds. Those can be removed now, as well as all explicit call too FooW WinAPI functions.
|Tags||No tags attached.|
|Has the bug occurred in previous versions?|
|Tested code revision (in case you know it)|
r9154 removes filename UTF8 tunneling in document handling
We need to decide if we want TCHAR PathString or not; if we don't, modifying CTreeCtrl.h doesn't make much sense, as it's used in PathString-related contexts only. So it's either both or none.
Another option might of course be to just completely remove support for ANSI builds. This would cost us some compile-time string type checking though.
Removing explicit WCHAR support in FileDIalog would allow us to use CFileDialog again, which has the advantage of automatically using IFileDialog when available rather than the classic GetOpenFileName/GetSaveFileName. This in return has the advantage that specifying a hook procedure for selection changes (like we do when "preview samples in file dialogs" is checked) does not enforce the "classic" pre-Vista file dialogs.
|2017-09-21 12:28||manx||New Issue|
|2017-09-21 12:28||manx||Status||new => assigned|
|2017-09-21 12:28||manx||Assigned To||=> manx|
|2017-09-21 12:29||manx||Relationship added||related to 0000570|
|2017-10-28 08:20||manx||Additional Information Updated||View Revisions|
|2017-10-28 08:21||manx||Additional Information Updated||View Revisions|
|2017-10-28 08:22||manx||Note Added: 0003316|
|2017-10-28 08:22||manx||Additional Information Updated||View Revisions|
|2018-02-11 22:55||Saga Musix||Note Added: 0003415|
|2018-02-11 22:56||Saga Musix||Note Edited: 0003415||View Revisions|
|2018-02-12 08:23||manx||Note Added: 0003416|
|2018-02-12 12:51||manx||Note Added: 0003417|
|2018-02-12 16:25||Saga Musix||Note Added: 0003418|
|2018-02-16 21:37||Saga Musix||Additional Information Updated||View Revisions|