Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Rewrite a chain of "if"s as a switch

Browse Source 
No real changes, just make the code a bit more maintainable and easier
to read.
This commit is contained in:
Vadim Zeitlin
2019-09-27 19:29:42 +02:00
parent 11f24369fd
commit 35d9b0f056
1 changed files with 21 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

45
src/common/intl.cpp
View File

@@ -877,37 +877,34 @@ wxString wxLocale::GetSystemEncodingName()
wxFontEncoding wxLocale::GetSystemEncoding()
{
#if defined(__WIN32__)
UINT codepage = ::GetACP();
const UINT codepage = ::GetACP();
// wxWidgets only knows about CP1250-1257, 874, 932, 936, 949, 950
if ( codepage >= 1250 && codepage <= 1257 )
switch ( codepage )
{
return (wxFontEncoding)(wxFONTENCODING_CP1250 + codepage - 1250);
}
case 1250:
case 1251:
case 1252:
case 1253:
case 1254:
case 1255:
case 1256:
case 1257:
return (wxFontEncoding)(wxFONTENCODING_CP1250 + codepage - 1250);
if ( codepage == 874 )
{
return wxFONTENCODING_CP874;
}
case 874:
return wxFONTENCODING_CP874;
if ( codepage == 932 )
{
return wxFONTENCODING_CP932;
}
case 932:
return wxFONTENCODING_CP932;
if ( codepage == 936 )
{
return wxFONTENCODING_CP936;
}
case 936:
return wxFONTENCODING_CP936;
if ( codepage == 949 )
{
return wxFONTENCODING_CP949;
}
case 949:
return wxFONTENCODING_CP949;
if ( codepage == 950 )
{
return wxFONTENCODING_CP950;
case 950:
return wxFONTENCODING_CP950;
}
#elif defined(__WXMAC__)
CFStringEncoding encoding = 0 ;