#6 CToolsMenuManager is a mess

open
nobody
None
5
2001-11-23
2001-11-23
Nick Marley
No

Looks like it completely lost the plot:

The list of macros is coded in two places
The need for a CWnd derived class to get at CWnd::GetCurrentMessage()
No encapsulation of the macro substitution system
Reliance on a range of menu items for macros

Discussion

  • Nick Marley
    Nick Marley
    2002-01-06

    Logged In: YES
    user_id=342518

    >
    >The need for a CWnd derived class to get at
    >CWnd::GetCurrentMessage()
    >
    Removed! Use the menu ID passed in the command range
    handler member function. This also fixes a crash in
    the release build - since I didn't have a parameter in
    the member function the stack was screwed.