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

Fix comments from review

Browse Source
This commit is contained in:
Ian McInerney
2019-08-05 20:51:46 +02:00
parent 585302a69b
commit ff1eb1afcc
4 changed files with 184 additions and 182 deletions
Download Patch File Download Diff File Expand all files Collapse all files

72
src/common/accelcmn.cpp
View File

@@ -45,43 +45,43 @@ static const struct wxKeyName
const char *display_name; const char *display_name;
} wxKeyNames[] = } wxKeyNames[] =
{ {
{ WXK_DELETE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Delete"), /*Display is the same as key name*/ 0 }, { WXK_DELETE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Delete") },
{ WXK_DELETE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Del"), /*Display is the same as key name*/ 0 }, { WXK_DELETE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Del") },
{ WXK_BACK, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Back"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Backspace") }, { WXK_BACK, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Back"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Backspace") },
{ WXK_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Insert"), /*Display is the same as key name*/ 0 }, { WXK_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Insert") },
{ WXK_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Ins"), /*Display is the same as key name*/ 0 }, { WXK_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Ins") },
{ WXK_RETURN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Enter"), /*Display is the same as key name*/ 0 }, { WXK_RETURN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Enter") },
{ WXK_RETURN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Return"), /*Display is the same as key name*/ 0 }, { WXK_RETURN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Return") },
{ WXK_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PageUp"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Page Up") }, { WXK_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PageUp"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Page Up") },
{ WXK_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PageDown"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Page Down") }, { WXK_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PageDown"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Page Down") },
{ WXK_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PgUp"), /*Display is the same as key name*/ 0 }, { WXK_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PgUp") },
{ WXK_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PgDn"), /*Display is the same as key name*/ 0 }, { WXK_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("PgDn") },
{ WXK_LEFT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Left"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Left") }, { WXK_LEFT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Left"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Left") },
{ WXK_RIGHT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Right"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Right") }, { WXK_RIGHT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Right"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Right") },
{ WXK_UP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Up"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Up") }, { WXK_UP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Up"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Up") },
{ WXK_DOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Down"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Down") }, { WXK_DOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Down"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Down") },
{ WXK_HOME, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Home"), /*Display is the same as key name*/ 0 }, { WXK_HOME, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Home") },
{ WXK_END, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("End"), /*Display is the same as key name*/ 0 }, { WXK_END, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("End") },
{ WXK_SPACE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Space"), /*Display is the same as key name*/ 0 }, { WXK_SPACE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Space") },
{ WXK_TAB, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Tab"), /*Display is the same as key name*/ 0 }, { WXK_TAB, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Tab") },
{ WXK_ESCAPE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Esc"), /*Display is the same as key name*/ 0 }, { WXK_ESCAPE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Esc") },
{ WXK_ESCAPE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Escape"), /*Display is the same as key name*/ 0 }, { WXK_ESCAPE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Escape") },
{ WXK_CANCEL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Cancel"), /*Display is the same as key name*/ 0 }, { WXK_CANCEL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Cancel") },
{ WXK_CLEAR, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Clear"), /*Display is the same as key name*/ 0 }, { WXK_CLEAR, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Clear") },
{ WXK_MENU, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Menu"), /*Display is the same as key name*/ 0 }, { WXK_MENU, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Menu") },
{ WXK_PAUSE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Pause"), /*Display is the same as key name*/ 0 }, { WXK_PAUSE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Pause") },
{ WXK_CAPITAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Capital"), /*Display is the same as key name*/ 0 }, { WXK_CAPITAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Capital") },
{ WXK_SELECT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Select"), /*Display is the same as key name*/ 0 }, { WXK_SELECT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Select") },
{ WXK_PRINT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Print"), /*Display is the same as key name*/ 0 }, { WXK_PRINT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Print") },
{ WXK_EXECUTE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Execute"), /*Display is the same as key name*/ 0 }, { WXK_EXECUTE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Execute") },
{ WXK_SNAPSHOT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Snapshot"), /*Display is the same as key name*/ 0 }, { WXK_SNAPSHOT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Snapshot") },
{ WXK_HELP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Help"), /*Display is the same as key name*/ 0 }, { WXK_HELP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Help") },
{ WXK_ADD, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Add"), /*Display is the same as key name*/ 0 }, { WXK_ADD, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Add") },
{ WXK_SEPARATOR, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Separator"), /*Display is the same as key name*/ 0 }, { WXK_SEPARATOR, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Separator") },
{ WXK_SUBTRACT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Subtract"), /*Display is the same as key name*/ 0 }, { WXK_SUBTRACT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Subtract") },
{ WXK_DECIMAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Decimal"), /*Display is the same as key name*/ 0 }, { WXK_DECIMAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Decimal") },
{ WXK_MULTIPLY, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Multiply"), /*Display is the same as key name*/ 0 }, { WXK_MULTIPLY, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Multiply") },
{ WXK_DIVIDE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Divide"), /*Display is the same as key name*/ 0 }, { WXK_DIVIDE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Divide") },
{ WXK_NUMLOCK, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num_lock"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Lock") }, { WXK_NUMLOCK, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num_lock"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Lock") },
{ WXK_SCROLL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Scroll_lock"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Scroll Lock") }, { WXK_SCROLL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Scroll_lock"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Scroll Lock") },
{ WXK_NUMPAD_SPACE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Space"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Space") }, { WXK_NUMPAD_SPACE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Space"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Space") },
@@ -94,8 +94,8 @@ static const struct wxKeyName
{ WXK_NUMPAD_DOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Down"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Down") }, { WXK_NUMPAD_DOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Down"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Down") },
{ WXK_NUMPAD_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_PageUp"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Page Up") }, { WXK_NUMPAD_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_PageUp"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Page Up") },
{ WXK_NUMPAD_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_PageDown"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Page Down") }, { WXK_NUMPAD_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_PageDown"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Page Down") },
{ WXK_NUMPAD_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Prior"), /*Display is the same as key name*/ 0 }, { WXK_NUMPAD_PAGEUP, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Prior") },
{ WXK_NUMPAD_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Next"), /*Display is the same as key name*/ 0 }, { WXK_NUMPAD_PAGEDOWN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Next") },
{ WXK_NUMPAD_END, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_End"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num End") }, { WXK_NUMPAD_END, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_End"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num End") },
{ WXK_NUMPAD_BEGIN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Begin"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Begin") }, { WXK_NUMPAD_BEGIN, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Begin"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Begin") },
{ WXK_NUMPAD_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Insert"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Insert") }, { WXK_NUMPAD_INSERT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Insert"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num Insert") },
@@ -107,10 +107,10 @@ static const struct wxKeyName
{ WXK_NUMPAD_SUBTRACT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Subtract"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num -") }, { WXK_NUMPAD_SUBTRACT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Subtract"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num -") },
{ WXK_NUMPAD_DECIMAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Decimal"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num .") }, { WXK_NUMPAD_DECIMAL, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Decimal"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num .") },
{ WXK_NUMPAD_DIVIDE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Divide"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num /") }, { WXK_NUMPAD_DIVIDE, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("KP_Divide"), /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Num /") },
{ WXK_WINDOWS_LEFT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Left"), /*Display is the same as key name*/ 0 }, { WXK_WINDOWS_LEFT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Left") },
{ WXK_WINDOWS_RIGHT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Right"), /*Display is the same as key name*/ 0 }, { WXK_WINDOWS_RIGHT, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Right") },
{ WXK_WINDOWS_MENU, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Menu"), /*Display is the same as key name*/ 0 }, { WXK_WINDOWS_MENU, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Windows_Menu") },
{ WXK_COMMAND, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Command"), /*Display is the same as key name*/ 0 }, { WXK_COMMAND, /*TRANSLATORS: Name of keyboard key*/ wxTRANSLATE("Command") },
}; };

18
src/gtk/menu.cpp
View File

@@ -674,7 +674,7 @@ void wxMenuItem::SetGtkLabel()
else else
{ {
// Remove the accelerator since it couldn't be made or is invalid // Remove the accelerator since it couldn't be made or is invalid
m_text = m_text.BeforeFirst( wxT( '\t' ) ); m_text = m_text.BeforeFirst( wxS( '\t' ) );
} }
#endif // wxUSE_ACCEL #endif // wxUSE_ACCEL
} }
@@ -1059,25 +1059,25 @@ static wxString GetGtkHotKey( const wxMenuItem& item )
hotkey << wxT("Delete" ); hotkey << wxT("Delete" );
break; break;
case WXK_UP: case WXK_UP:
if( flags ) if ( flags )
hotkey << wxT("Up" ); hotkey << wxT("Up" );
else else
wxFAIL_MSG( wxT("The Up key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The Up key must have modifiers to be an accelerator key") );
break; break;
case WXK_DOWN: case WXK_DOWN:
if( flags ) if ( flags )
hotkey << wxT("Down" ); hotkey << wxT("Down" );
else else
wxFAIL_MSG( wxT("The Down key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The Down key must have modifiers to be an accelerator key") );
break; break;
case WXK_LEFT: case WXK_LEFT:
if( flags ) if ( flags )
hotkey << wxT("Left" ); hotkey << wxT("Left" );
else else
wxFAIL_MSG( wxT("The Left key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The Left key must have modifiers to be an accelerator key") );
break; break;
case WXK_RIGHT: case WXK_RIGHT:
if( flags ) if ( flags )
hotkey << wxT("Right" ); hotkey << wxT("Right" );
else else
wxFAIL_MSG( wxT("The Right key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The Right key must have modifiers to be an accelerator key") );
@@ -1153,25 +1153,25 @@ static wxString GetGtkHotKey( const wxMenuItem& item )
hotkey << wxT("KP_Home" ); hotkey << wxT("KP_Home" );
break; break;
case WXK_NUMPAD_UP: case WXK_NUMPAD_UP:
if( flags ) if ( flags )
hotkey << wxT("KP_Up" ); hotkey << wxT("KP_Up" );
else else
wxFAIL_MSG( wxT("The KP_Up key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The KP_Up key must have modifiers to be an accelerator key") );
break; break;
case WXK_NUMPAD_DOWN: case WXK_NUMPAD_DOWN:
if( flags ) if ( flags )
hotkey << wxT("KP_Down" ); hotkey << wxT("KP_Down" );
else else
wxFAIL_MSG( wxT("The KP_Down key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The KP_Down key must have modifiers to be an accelerator key") );
break; break;
case WXK_NUMPAD_LEFT: case WXK_NUMPAD_LEFT:
if( flags ) if ( flags )
hotkey << wxT("KP_Left" ); hotkey << wxT("KP_Left" );
else else
wxFAIL_MSG( wxT("The KP_Left key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The KP_Left key must have modifiers to be an accelerator key") );
break; break;
case WXK_NUMPAD_RIGHT: case WXK_NUMPAD_RIGHT:
if( flags ) if ( flags )
hotkey << wxT("KP_Right" ); hotkey << wxT("KP_Right" );
else else
wxFAIL_MSG( wxT("The KP_Right key must have modifiers to be an accelerator key") ); wxFAIL_MSG( wxT("The KP_Right key must have modifiers to be an accelerator key") );

22
tests/menu/accelentry.cpp
View File

@@ -26,8 +26,8 @@ namespace
void CheckAccelEntry(const wxAcceleratorEntry& accel, int keycode, int flags) void CheckAccelEntry(const wxAcceleratorEntry& accel, int keycode, int flags)
{ {
REQUIRE( keycode == accel.GetKeyCode() ); CHECK( keycode == accel.GetKeyCode() );
REQUIRE( flags == accel.GetFlags() ); CHECK( flags == accel.GetFlags() );
} }
} // anonymous namespace } // anonymous namespace
@@ -44,8 +44,8 @@ TEST_CASE( "wxAcceleratorEntry::Create", "[accelentry]" )
{ {
pa = wxAcceleratorEntry::Create("Foo\tCtrl+Z"); pa = wxAcceleratorEntry::Create("Foo\tCtrl+Z");
REQUIRE( pa ); CHECK( pa );
REQUIRE( pa->IsOk() ); CHECK( pa->IsOk() );
CheckAccelEntry(*pa, 'Z', wxACCEL_CTRL); CheckAccelEntry(*pa, 'Z', wxACCEL_CTRL);
} }
@@ -53,22 +53,22 @@ TEST_CASE( "wxAcceleratorEntry::Create", "[accelentry]" )
{ {
pa = wxAcceleratorEntry::Create("Shift-Q"); pa = wxAcceleratorEntry::Create("Shift-Q");
REQUIRE( !pa ); CHECK( !pa );
} }
SECTION( "No accelerator key specified" ) SECTION( "No accelerator key specified" )
{ {
pa = wxAcceleratorEntry::Create("bloordyblop"); pa = wxAcceleratorEntry::Create("bloordyblop");
REQUIRE( !pa ); CHECK( !pa );
} }
SECTION( "Display name parsing" ) SECTION( "Display name parsing" )
{ {
pa = wxAcceleratorEntry::Create("Test\tBackSpace"); pa = wxAcceleratorEntry::Create("Test\tBackSpace");
REQUIRE( pa ); CHECK( pa );
REQUIRE( pa->IsOk() ); CHECK( pa->IsOk() );
CheckAccelEntry(*pa, WXK_BACK, wxACCEL_NORMAL); CheckAccelEntry(*pa, WXK_BACK, wxACCEL_NORMAL);
} }
} }
@@ -84,17 +84,17 @@ TEST_CASE( "wxAcceleratorEntry::StringTests", "[accelentry]" )
SECTION( "Create string from key" ) SECTION( "Create string from key" )
{ {
REQUIRE( "Alt+X" == a.ToString() ); CHECK( "Alt+X" == a.ToString() );
} }
SECTION( "Create from valid string" ) SECTION( "Create from valid string" )
{ {
REQUIRE( a.FromString("Alt+Shift+F1") ); CHECK( a.FromString("Alt+Shift+F1") );
CheckAccelEntry(a, WXK_F1, wxACCEL_ALT | wxACCEL_SHIFT); CheckAccelEntry(a, WXK_F1, wxACCEL_ALT | wxACCEL_SHIFT);
} }
SECTION( "Create from invalid string" ) SECTION( "Create from invalid string" )
{ {
REQUIRE( !a.FromString("bloordyblop") ); CHECK( !a.FromString("bloordyblop") );
} }
} }

40
tests/menu/menu.cpp
View File

@@ -611,33 +611,34 @@ void MenuTestCase::Events()
namespace namespace
{ {
void verifyAccelAssigned( wxString labelText, int keycode )
{ void VerifyAccelAssigned( wxString labelText, int keycode )
{
wxAcceleratorEntry* entry = wxAcceleratorEntry::Create( labelText ); wxAcceleratorEntry* entry = wxAcceleratorEntry::Create( labelText );
REQUIRE( entry ); CHECK( entry );
REQUIRE( entry->GetKeyCode() == keycode ); CHECK( entry->GetKeyCode() == keycode );
} }
struct key struct key
{ {
int keycode; int keycode;
wxString name; wxString name;
bool skip; bool skip;
}; };
key modKeys[] = key modKeys[] =
{ {
{ wxACCEL_NORMAL, "Normal", false }, { wxACCEL_NORMAL, "Normal", false },
{ wxACCEL_CTRL, "Ctrl", false }, { wxACCEL_CTRL, "Ctrl", false },
{ wxACCEL_SHIFT, "Shift", false }, { wxACCEL_SHIFT, "Shift", false },
{ wxACCEL_ALT, "Alt", false } { wxACCEL_ALT, "Alt", false }
}; };
/* /*
The keys marked as skip below are not supported as accelerator The keys marked as skip below are not supported as accelerator
keys on GTK. keys on GTK.
*/ */
key specialKeys[] = key specialKeys[] =
{ {
{ WXK_F1, "WXK_F1", false }, { WXK_F1, "WXK_F1", false },
{ WXK_F2, "WXK_F2", false }, { WXK_F2, "WXK_F2", false },
{ WXK_F3, "WXK_F3", false }, { WXK_F3, "WXK_F3", false },
@@ -734,7 +735,8 @@ namespace
{ WXK_WINDOWS_RIGHT, "WXK_WINDOWS_RIGHT", true }, { WXK_WINDOWS_RIGHT, "WXK_WINDOWS_RIGHT", true },
{ WXK_WINDOWS_MENU, "WXK_WINDOWS_MENU", false }, { WXK_WINDOWS_MENU, "WXK_WINDOWS_MENU", false },
{ WXK_COMMAND, "WXK_COMMAND", true } { WXK_COMMAND, "WXK_COMMAND", true }
}; };
} }
TEST_CASE( "wxMenuItemAccelEntry", "[menu][accelentry]" ) TEST_CASE( "wxMenuItemAccelEntry", "[menu][accelentry]" )
@@ -746,7 +748,7 @@ TEST_CASE( "wxMenuItemAccelEntry", "[menu][accelentry]" )
SECTION( "Modifier keys" ) SECTION( "Modifier keys" )
{ {
for( int i = 0; i < WXSIZEOF(modKeys); i++ ) for ( int i = 0; i < WXSIZEOF(modKeys); i++ )
{ {
const key& k = modKeys[i]; const key& k = modKeys[i];
@@ -757,13 +759,13 @@ TEST_CASE( "wxMenuItemAccelEntry", "[menu][accelentry]" )
wxString labelText = item->GetItemLabel(); wxString labelText = item->GetItemLabel();
INFO( wxString::Format( "Label text: %s", labelText ) ); INFO( wxString::Format( "Label text: %s", labelText ) );
verifyAccelAssigned( labelText, 'A' ); VerifyAccelAssigned( labelText, 'A' );
} }
} }
SECTION( "Special keys" ) SECTION( "Special keys" )
{ {
for( int i = 0; i < WXSIZEOF(specialKeys); i++ ) for ( int i = 0; i < WXSIZEOF(specialKeys); i++ )
{ {
const key& k = specialKeys[i]; const key& k = specialKeys[i];
@@ -777,7 +779,7 @@ TEST_CASE( "wxMenuItemAccelEntry", "[menu][accelentry]" )
wxString labelText = item->GetItemLabel(); wxString labelText = item->GetItemLabel();
INFO( wxString::Format( "Label text: %s", labelText ) ); INFO( wxString::Format( "Label text: %s", labelText ) );
verifyAccelAssigned( labelText, k.keycode ); VerifyAccelAssigned( labelText, k.keycode );
} }
} }
} }