Windows MFC migration framework: qmfcapp::wineventFilter calls PreTranslateMessage unnecessarily

When the main window of an MFC application has shortcuts and you open a modal Qt dialog on top of it from the menu, the shortcuts get evaluated and executed, which is not what you want when a modal dialog is open.

This is caused by the call to PreTranslateMessage in wineventFilter (coming from the event-loop of the modal dialog, which is unnecessary when the message is sent in a Qt-only context.

I have provided a patch that fixes this to the best of my knowledge.

Hint for reproduction:
It's necessary that the Qt dialog be opened from the menu and not a button or shortcut, because otherwise the main event-loop will already go through wineventFilter and it won't go recursively because it has a recursion guard.