bool IsOk() const;
/**
- Returns a wxString for this accelerator.
+ Returns a textual representation of this accelerator.
- This function formats it using the @c "flags-keycode" format
- where @c flags maybe a hyphen-separed list of @c "shift|alt|ctrl".
+ The returned string is of the form <code>[Alt+][Ctrl+][Shift+]Key</code>
+ where the modifier keys are present only if the corresponding flag is
+ set.
*/
wxString ToString() const;
Parses the given string and sets the accelerator accordingly.
@param str
- Should be a string in the form "flags-keycode"
+ This string may be either in the same format as returned by
+ ToString(), i.e. contain the accelerator itself only, or have the
+ format of a full menu item text with i.e. <code>Label TAB
+ Accelerator</code>. In the latter case, the part of the string
+ before the TAB is ignored.
@return @true if the given string correctly initialized this object
(i.e. if IsOk() returns true after this call)
wxString label = text;
label.Trim(true); // the initial \t must be preserved so don't strip leading whitespaces
+ // If we're passed the entire menu item label instead of just the
+ // accelerator, skip the label part and only look after the TAB.
// check for accelerators: they are given after '\t'
int posTab = label.Find(wxT('\t'));
if ( posTab == wxNOT_FOUND )
- {
- return false;
- }
+ posTab = 0;
+ else
+ posTab++;
// parse the accelerator string
int accelFlags = wxACCEL_NORMAL;
wxString current;
- for ( size_t n = (size_t)posTab + 1; n < label.length(); n++ )
+ for ( size_t n = (size_t)posTab; n < label.length(); n++ )
{
if ( (label[n] == '+') || (label[n] == '-') )
{
test_gui_image.o \
test_gui_rawbmp.o \
test_gui_htmlwindow.o \
+ test_gui_accelentry.o \
test_gui_menu.o \
test_gui_guifuncs.o \
test_gui_selstoretest.o \
test_gui_htmlwindow.o: $(srcdir)/html/htmlwindow.cpp $(TEST_GUI_ODEP)
$(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/html/htmlwindow.cpp
+test_gui_accelentry.o: $(srcdir)/menu/accelentry.cpp $(TEST_GUI_ODEP)
+ $(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/menu/accelentry.cpp
+
test_gui_menu.o: $(srcdir)/menu/menu.cpp $(TEST_GUI_ODEP)
$(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/menu/menu.cpp
$(OBJS)\test_gui_gaugetest.obj \\r
$(OBJS)\test_gui_gridtest.obj \\r
$(OBJS)\test_gui_headerctrltest.obj \\r
- $(OBJS)\test_gui_htmllboxtest.obj \
+ $(OBJS)\test_gui_htmllboxtest.obj \\r
$(OBJS)\test_gui_hyperlinkctrltest.obj \\r
$(OBJS)\test_gui_itemcontainertest.obj \\r
$(OBJS)\test_gui_label.obj \\r
$(OBJS)\test_gui_image.obj \\r
$(OBJS)\test_gui_rawbmp.obj \\r
$(OBJS)\test_gui_htmlwindow.obj \\r
+ $(OBJS)\test_gui_accelentry.obj \\r
$(OBJS)\test_gui_menu.obj \\r
$(OBJS)\test_gui_guifuncs.obj \\r
$(OBJS)\test_gui_selstoretest.obj \\r
$(OBJS)\test_gui_headerctrltest.obj: .\controls\headerctrltest.cpp\r
$(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\controls\headerctrltest.cpp\r
\r
-$(OBJS)\test_gui_htmllboxtest.obj: .\controls\htmllboxtest.cpp
- $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\controls\htmllboxtest.cpp
-
+$(OBJS)\test_gui_htmllboxtest.obj: .\controls\htmllboxtest.cpp\r
+ $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\controls\htmllboxtest.cpp\r
+\r
$(OBJS)\test_gui_hyperlinkctrltest.obj: .\controls\hyperlinkctrltest.cpp\r
$(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\controls\hyperlinkctrltest.cpp\r
\r
$(OBJS)\test_gui_htmlwindow.obj: .\html\htmlwindow.cpp\r
$(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\html\htmlwindow.cpp\r
\r
+$(OBJS)\test_gui_accelentry.obj: .\menu\accelentry.cpp\r
+ $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\menu\accelentry.cpp\r
+\r
$(OBJS)\test_gui_menu.obj: .\menu\menu.cpp\r
$(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\menu\menu.cpp\r
\r
$(OBJS)\test_gui_gaugetest.o \\r
$(OBJS)\test_gui_gridtest.o \\r
$(OBJS)\test_gui_headerctrltest.o \\r
- $(OBJS)\test_gui_htmllboxtest.o \
+ $(OBJS)\test_gui_htmllboxtest.o \\r
$(OBJS)\test_gui_hyperlinkctrltest.o \\r
$(OBJS)\test_gui_itemcontainertest.o \\r
$(OBJS)\test_gui_label.o \\r
$(OBJS)\test_gui_image.o \\r
$(OBJS)\test_gui_rawbmp.o \\r
$(OBJS)\test_gui_htmlwindow.o \\r
+ $(OBJS)\test_gui_accelentry.o \\r
$(OBJS)\test_gui_menu.o \\r
$(OBJS)\test_gui_guifuncs.o \\r
$(OBJS)\test_gui_selstoretest.o \\r
$(OBJS)\test_gui_headerctrltest.o: ./controls/headerctrltest.cpp\r
$(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
\r
-$(OBJS)\test_gui_htmllboxtest.o: ./controls/htmllboxtest.cpp
- $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<
-
+$(OBJS)\test_gui_htmllboxtest.o: ./controls/htmllboxtest.cpp\r
+ $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
+\r
$(OBJS)\test_gui_hyperlinkctrltest.o: ./controls/hyperlinkctrltest.cpp\r
$(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
\r
$(OBJS)\test_gui_htmlwindow.o: ./html/htmlwindow.cpp\r
$(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
\r
+$(OBJS)\test_gui_accelentry.o: ./menu/accelentry.cpp\r
+ $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
+\r
$(OBJS)\test_gui_menu.o: ./menu/menu.cpp\r
$(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
\r
$(OBJS)\test_gui_gaugetest.obj \\r
$(OBJS)\test_gui_gridtest.obj \\r
$(OBJS)\test_gui_headerctrltest.obj \\r
- $(OBJS)\test_gui_htmllboxtest.obj \
+ $(OBJS)\test_gui_htmllboxtest.obj \\r
$(OBJS)\test_gui_hyperlinkctrltest.obj \\r
$(OBJS)\test_gui_itemcontainertest.obj \\r
$(OBJS)\test_gui_label.obj \\r
$(OBJS)\test_gui_image.obj \\r
$(OBJS)\test_gui_rawbmp.obj \\r
$(OBJS)\test_gui_htmlwindow.obj \\r
+ $(OBJS)\test_gui_accelentry.obj \\r
$(OBJS)\test_gui_menu.obj \\r
$(OBJS)\test_gui_guifuncs.obj \\r
$(OBJS)\test_gui_selstoretest.obj \\r
$(OBJS)\test_gui_headerctrltest.obj: .\controls\headerctrltest.cpp\r
$(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\controls\headerctrltest.cpp\r
\r
-$(OBJS)\test_gui_htmllboxtest.obj: .\controls\htmllboxtest.cpp
- $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\controls\htmllboxtest.cpp
-
+$(OBJS)\test_gui_htmllboxtest.obj: .\controls\htmllboxtest.cpp\r
+ $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\controls\htmllboxtest.cpp\r
+\r
$(OBJS)\test_gui_hyperlinkctrltest.obj: .\controls\hyperlinkctrltest.cpp\r
$(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\controls\hyperlinkctrltest.cpp\r
\r
$(OBJS)\test_gui_htmlwindow.obj: .\html\htmlwindow.cpp\r
$(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\html\htmlwindow.cpp\r
\r
+$(OBJS)\test_gui_accelentry.obj: .\menu\accelentry.cpp\r
+ $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\menu\accelentry.cpp\r
+\r
$(OBJS)\test_gui_menu.obj: .\menu\menu.cpp\r
$(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\menu\menu.cpp\r
\r
$(OBJS)\test_gui_gaugetest.obj &\r
$(OBJS)\test_gui_gridtest.obj &\r
$(OBJS)\test_gui_headerctrltest.obj &\r
- $(OBJS)\test_gui_htmllboxtest.obj &
+ $(OBJS)\test_gui_htmllboxtest.obj &\r
$(OBJS)\test_gui_hyperlinkctrltest.obj &\r
$(OBJS)\test_gui_itemcontainertest.obj &\r
$(OBJS)\test_gui_label.obj &\r
$(OBJS)\test_gui_image.obj &\r
$(OBJS)\test_gui_rawbmp.obj &\r
$(OBJS)\test_gui_htmlwindow.obj &\r
+ $(OBJS)\test_gui_accelentry.obj &\r
$(OBJS)\test_gui_menu.obj &\r
$(OBJS)\test_gui_guifuncs.obj &\r
$(OBJS)\test_gui_selstoretest.obj &\r
$(OBJS)\test_gui_headerctrltest.obj : .AUTODEPEND .\controls\headerctrltest.cpp\r
$(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
\r
-$(OBJS)\test_gui_htmllboxtest.obj : .AUTODEPEND .\controls\htmllboxtest.cpp
- $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<
-
+$(OBJS)\test_gui_htmllboxtest.obj : .AUTODEPEND .\controls\htmllboxtest.cpp\r
+ $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
+\r
$(OBJS)\test_gui_hyperlinkctrltest.obj : .AUTODEPEND .\controls\hyperlinkctrltest.cpp\r
$(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
\r
$(OBJS)\test_gui_htmlwindow.obj : .AUTODEPEND .\html\htmlwindow.cpp\r
$(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
\r
+$(OBJS)\test_gui_accelentry.obj : .AUTODEPEND .\menu\accelentry.cpp\r
+ $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
+\r
$(OBJS)\test_gui_menu.obj : .AUTODEPEND .\menu\menu.cpp\r
$(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
\r
--- /dev/null
+///////////////////////////////////////////////////////////////////////////////
+// Name: tests/menu/accelentry.cpp
+// Purpose: wxAcceleratorEntry unit test
+// Author: Vadim Zeitlin
+// Created: 2010-12-03
+// RCS-ID: $Id$
+// Copyright: (c) 2010 Vadim Zeitlin
+///////////////////////////////////////////////////////////////////////////////
+
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+
+#include "testprec.h"
+
+#ifdef __BORLANDC__
+ #pragma hdrstop
+#endif
+
+#ifndef WX_PRECOMP
+#endif // WX_PRECOMP
+
+#include "wx/accel.h"
+#include "wx/scopedptr.h"
+
+class AccelEntryTestCase : public CppUnit::TestCase
+{
+public:
+ AccelEntryTestCase() {}
+
+private:
+ CPPUNIT_TEST_SUITE( AccelEntryTestCase );
+ CPPUNIT_TEST( Create );
+ CPPUNIT_TEST( ToFromString );
+ CPPUNIT_TEST_SUITE_END();
+
+ void Create();
+ void ToFromString();
+
+ wxDECLARE_NO_COPY_CLASS(AccelEntryTestCase);
+};
+
+// register in the unnamed registry so that these tests are run by default
+CPPUNIT_TEST_SUITE_REGISTRATION( AccelEntryTestCase );
+
+// also include in it's own registry so that these tests can be run alone
+CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( AccelEntryTestCase, "AccelEntryTestCase" );
+
+namespace
+{
+
+void CheckAccelEntry(const wxAcceleratorEntry& accel, int keycode, int flags)
+{
+ CPPUNIT_ASSERT_EQUAL( keycode, accel.GetKeyCode() );
+ CPPUNIT_ASSERT_EQUAL( flags, accel.GetFlags() );
+}
+
+} // anonymous namespace
+
+void AccelEntryTestCase::Create()
+{
+ wxScopedPtr<wxAcceleratorEntry>
+ pa(wxAcceleratorEntry::Create("Foo\tCtrl+Z"));
+ CPPUNIT_ASSERT( pa );
+ CPPUNIT_ASSERT( pa->IsOk() );
+
+ CheckAccelEntry(*pa, 'Z', wxACCEL_CTRL);
+
+
+ pa.reset(wxAcceleratorEntry::Create("Shift-Q"));
+ CPPUNIT_ASSERT( pa );
+ CPPUNIT_ASSERT( pa->IsOk() );
+
+ CheckAccelEntry(*pa, 'Q', wxACCEL_SHIFT);
+
+
+ pa.reset(wxAcceleratorEntry::Create("bloordyblop"));
+ CPPUNIT_ASSERT( !pa );
+}
+
+void AccelEntryTestCase::ToFromString()
+{
+ wxAcceleratorEntry a(wxACCEL_ALT, 'X');
+ CPPUNIT_ASSERT_EQUAL( "Alt+X", a.ToString() );
+
+ CPPUNIT_ASSERT( a.FromString("Alt+Shift+F1") );
+ CheckAccelEntry(a, WXK_F1, wxACCEL_ALT | wxACCEL_SHIFT);
+
+ CPPUNIT_ASSERT( !a.FromString("bloordyblop") );
+}
image/image.cpp
image/rawbmp.cpp
html/htmlwindow.cpp
+ menu/accelentry.cpp
menu/menu.cpp
misc/guifuncs.cpp
misc/selstoretest.cpp
# PROP Default_Filter ""\r
# Begin Source File\r
\r
+SOURCE=.\menu\accelentry.cpp\r
+# End Source File\r
+# Begin Source File\r
+\r
SOURCE=.\asserthelper.cpp\r
# End Source File\r
# Begin Source File\r
# End Source File\r
# Begin Source File\r
\r
-SOURCE=.\controls\htmllboxtest.cpp
-# End Source File
-# Begin Source File
-
+SOURCE=.\controls\htmllboxtest.cpp\r
+# End Source File\r
+# Begin Source File\r
+\r
SOURCE=.\html\htmlwindow.cpp\r
# End Source File\r
# Begin Source File\r
Name="Source Files"\r
Filter="cpp;c;cc;cxx;def;odl;idl;hpj;bat;asm;asmx"\r
UniqueIdentifier="{4FC737F1-C7A5-4376-A066-2A32D752A2FF}">\r
+ <File\r
+ RelativePath=".\menu\accelentry.cpp">\r
+ </File>\r
<File\r
RelativePath=".\asserthelper.cpp">\r
</File>\r
RelativePath=".\controls\headerctrltest.cpp">\r
</File>\r
<File\r
- RelativePath=".\controls\htmllboxtest.cpp">
- </File>
- <File
+ RelativePath=".\controls\htmllboxtest.cpp">\r
+ </File>\r
+ <File\r
RelativePath=".\html\htmlwindow.cpp">\r
</File>\r
<File\r
Filter="cpp;c;cc;cxx;def;odl;idl;hpj;bat;asm;asmx"\r
UniqueIdentifier="{4FC737F1-C7A5-4376-A066-2A32D752A2FF}"\r
>\r
+ <File\r
+ RelativePath=".\menu\accelentry.cpp"\r
+ >\r
+ </File>\r
<File\r
RelativePath=".\asserthelper.cpp"\r
>\r
>\r
</File>\r
<File\r
- RelativePath=".\controls\htmllboxtest.cpp"
- >
- </File>
- <File
+ RelativePath=".\controls\htmllboxtest.cpp"\r
+ >\r
+ </File>\r
+ <File\r
RelativePath=".\html\htmlwindow.cpp"\r
>\r
</File>\r
Filter="cpp;c;cc;cxx;def;odl;idl;hpj;bat;asm;asmx"\r
UniqueIdentifier="{4FC737F1-C7A5-4376-A066-2A32D752A2FF}"\r
>\r
+ <File\r
+ RelativePath=".\menu\accelentry.cpp"\r
+ >\r
+ </File>\r
<File\r
RelativePath=".\asserthelper.cpp"\r
>\r
>\r
</File>\r
<File\r
- RelativePath=".\controls\htmllboxtest.cpp"
- >
- </File>
- <File
+ RelativePath=".\controls\htmllboxtest.cpp"\r
+ >\r
+ </File>\r
+ <File\r
RelativePath=".\html\htmlwindow.cpp"\r
>\r
</File>\r