EventGhostPlus (1 Viewer)

michael_t

Portal Pro
November 30, 2008
1,258
813
Home Country
Germany Germany
Hmmm that maybe @michael_t :

Removed code :
Code:
      // TODO: should not call return here, no process plugins should be allowed to abort windows message processing
      // forward message to process plugins
      if (PluginManager.WndProc(ref msg))
      {
        // msg.Result = new IntPtr(0); <-- do plugins really set it on their own?
        return;
      }

What do you think ?
Oops, the PluginManager.WndProc(ref msg) is not called on every message type, so obviously the message the event ghost plugin is waiting for is omitted. There must be a "default:" clause added to the switch statement...

Michael
 
Last edited:

Sebastiii

Development Group
  • Team MediaPortal
  • November 12, 2007
    16,583
    10,403
    France
    Home Country
    France France
    Hmmm that maybe @michael_t :

    Removed code :
    Code:
          // TODO: should not call return here, no process plugins should be allowed to abort windows message processing
          // forward message to process plugins
          if (PluginManager.WndProc(ref msg))
          {
            // msg.Result = new IntPtr(0); <-- do plugins really set it on their own?
            return;
          }

    What do you think ?
    Oops, the PluginManager.WndProc(ref msg) is not called on every message type, so obviously the message the event ghost plugin is waiting for is omitted. There must be a "default:" clause added to the switch statement...

    Michael

    :)

    I can share the code but i will push it in a branch with code review, like this you can confirm if it's ok or needed change :)
     

    Users who are viewing this thread

    Top Bottom