[Bug 19368] New: lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Summary: lpfnWndProc can't catch Up/Down/Left/Right and CTRL+
                    other Button
           Product: Wine
           Version: 1.1.26
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Hello

I want to use "sakura editor".
(a Japanese Open Souce text editor for MS Windows)
(http://sakura-editor.sourceforge.net/)

But this application is not work well in wine(1.0.1 or 1.1.26 Ubuntu GNU/Linux
9.0.4)
- Not work Up/Down/Left/Right button in text area
- Not work CTRL+C or CTRL+A and other in text area
- Not work Backspace in text area
- Not work Function Key in text area

normal text button( a-z, 0-9 ) is work well,
and Up/Down... is work well in menu .

"sakura editor" catch Up/Down/Ctrl event from lpfnWndProc() callback.

but wine's lpfnWndProc seem not work when push Up/Down... key.
( I insert log in sakura editor source and check.)

I have 2 question. please tell me.

(1) Can lpfnWndProc() catch Up/Down/Left/Right CTRL+A,X,C and Function key in
Wine ?
(2) "lpfnWndProc() can catch Up/Down/Left/Right ..." is WindowsAPI
specification?

Thank you.

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Jeff Zaroyko  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download, source

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Vitaliy Margolen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                URL|                            |http://sakura-editor.source
                   |                            |forge.net/
     Ever Confirmed|0                           |1

--- Comment #1 from Vitaliy Margolen   2009-07-19 12:19:31 ---
Confirming.

The code looks soo bad it's impossible to see what it's doing and how it's
doing it. It would be much easier for a dev from sakura-editor project to find
the problem. Or at least point to exact place where they are expecting
left/right/up/down arrow keys.

The ctrl,shift,alt + hotkeys don't work because of bug 1348.
> "sakura editor" catch Up/Down/Ctrl event from lpfnWndProc() callback.

Where exactly? Which file? Which function?

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Hidetaka Sakai  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Hidetaka Sakai   2009-07-19 20:44:07 ---
Thank you for reply
> The code looks soo bad it's impossible to see what it's doing and how it's
> doing it. It would be much easier for a dev from sakura-editor project to find
> the problem. Or at least point to exact place where they are expecting
> left/right/up/down arrow keys.

Ohoo. sakura-editor is not good code?!
> The ctrl,shift,alt + hotkeys don't work because of bug 1348.

It's already reported bug.
But I can't find 1348's patch (id=3668) in wine(1.1.26).
(not find RegisterHotKey keyword in dlls/user32/driver.c )
> > "sakura editor" catch Up/Down/Ctrl event from lpfnWndProc() callback.
>
> Where exactly? Which file? Which function?

CEDditWndProc() (CEditWnd.cpp)  <= This is lpfnWndProc callback function.
  ->  DispatchEvent() (CEditWnd.cpp)
   ->  OnCommand() () (CEditWnd.cpp)

thank you.

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

--- Comment #3 from Vitaliy Margolen   2009-07-20 07:49:17 ---
Why did you marked fixed?

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Hidetaka Sakai  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |WORKSFORME

--- Comment #4 from Hidetaka Sakai   2009-07-20 16:15:24 ---
> Why did you marked fixed?

Sory. I made a mistake.
But I don't know to select how status.

I makes it to WORKSFORME temporarily.
Is this OK ?

Thank you

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|WORKSFORME                  |

--- Comment #5 from Austin English   2009-07-20 17:54:09 ---
(In reply to comment #4)
> > Why did you marked fixed?
>
> Sory. I made a mistake.
> But I don't know to select how status.
>
> I makes it to WORKSFORME temporarily.
> Is this OK ?
>
> Thank you

There's still a bug to fix in wine.

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

--- Comment #6 from Hidetaka Sakai   2009-08-05 11:43:33 ---
Hello.

I corrected the sakura-editor and report it.

I seem "The problem was in Accelarator Table".

sakura-editor has Accelarator Table (HACCEL) in Shared Memory.
Accelarator Table is shared in two or more processes.

I tried change sakura-editor that it use StackMemory.
Then, sakura-editor became operated normally. (UP/Down/Ctrl... OK)
(http://sakaihdt.gozaru.jp/sakura_en.html )

I seem " Windows has Accelarator Table in Shared Memory "
But Wine has Accelarator Table in HeepArea.
( dlls/user32/resource.c CreateAcceleratorTableA() use
  GlobalAlloc16 function .)

thank you.

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |NEW

--- Comment #7 from Dmitry Timoshkov   2009-08-05 23:08:35 ---
(In reply to comment #6)
> I seem " Windows has Accelarator Table in Shared Memory "
> But Wine has Accelarator Table in HeepArea.
> ( dlls/user32/resource.c CreateAcceleratorTableA() use
>   GlobalAlloc16 function .)

In Windows an accelerator is a user32 kernel object, like windows and
menus, and an accelerator created in one process is visible to another
one. Wine implements this only for window handles.

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

http://bugs.winehq.org/show_bug.cgi?id=19368

Bruno Jesus  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #8 from Bruno Jesus  2011-10-07 17:21:25 CDT ---
Still present in 1.3.29.

When the arrows or CTRL key is pressed this is printed in the console for each
attempt:

fixme:resource:CopyAcceleratorTableW other process handle 0x2003a?

[Bug 19368] lpfnWndProc can't catch Up/Down/Left/Right and CTRL+ other Button

https://bugs.winehq.org/show_bug.cgi?id=19368

--- Comment #9 from Ken Sharp  ---
Is this still an issue in Wine 1.7.45 or later?