Cleanup of dev-time install and debugging, removal of commented code

This commit is contained in:
Marc Durdin 2017-09-20 12:08:46 +07:00
parent 01430bba88
commit 3da89424ed
16 changed files with 36 additions and 49 deletions

View file

@ -25,9 +25,9 @@ BUILD=$(ROOT)\build
DEBUGPATH=$(ROOT)\debug
OUTLIB=$(ROOT)\lib
INSTALLPATH_KEYMANDESKTOP="%ProgramFiles(X86)%\Keyman\Keyman Desktop 10.0"
INSTALLPATH_KEYMANDEVELOPER="%ProgramFiles(X86)%\Keyman\Keyman Developer 10.0"
INSTALLPATH_KEYMANENGINE="%CommonProgramFiles(X86)%\Keyman\Keyman Engine 10.0"
INSTALLPATH_KEYMANDESKTOP=%ProgramFiles(X86)%\Keyman\Keyman Desktop 10.0
INSTALLPATH_KEYMANDEVELOPER=%ProgramFiles(X86)%\Keyman\Keyman Developer 10.0
INSTALLPATH_KEYMANENGINE=%CommonProgramFiles(X86)%\Keyman\Keyman Engine 10.0
!IFDEF DEBUG
MAKEFLAG_DEBUG=-DDEBUG

View file

@ -36,7 +36,7 @@ test-manifest:
$(MT) -nologo -inputresource:$(PROGRAM)\desktop\kmshell.exe -validate_manifest
install:
copy $(ROOT)\bin\desktop\kmshell.exe $(INSTALLPATH_KEYMANDESKTOP)
copy $(ROOT)\bin\desktop\xml\* $(INSTALLPATH_KEYMANDESKTOP)\xml\
copy $(ROOT)\bin\desktop\kmshell.exe "$(INSTALLPATH_KEYMANDESKTOP)"
copy $(ROOT)\bin\desktop\xml\* "$(INSTALLPATH_KEYMANDESKTOP)\xml\"
!include ..\..\Target.mak

View file

@ -40,7 +40,7 @@ backup:
install:
$(COPY) $(PROGRAM)\developer\tike.exe $(INSTALLPATH_KEYMANDEVELOPER)\tike.exe
$(COPY) $(PROGRAM)\developer\tike.exe "$(INSTALLPATH_KEYMANDEVELOPER)\tike.exe"
test-manifest:
# test that linked manifest exists and correct

View file

@ -24,6 +24,6 @@ test-manifest:
@rem This target needed as dependency for TIKE and KMCMPDLL
install:
$(COPY) $(PROGRAM)\developer\kmcmpdll.dll $(INSTALLPATH_KEYMANDEVELOPER)\kmcmpdll.dll
$(COPY) $(PROGRAM)\developer\kmcmpdll.dll "$(INSTALLPATH_KEYMANDEVELOPER)\kmcmpdll.dll"
!include ..\..\Target.mak

View file

@ -28,6 +28,6 @@ test-manifest:
$(MT) -nologo -inputresource:$(PROGRAM)\developer\kmcomp.exe -validate_manifest
install:
$(COPY) $(PROGRAM)\developer\kmcomp.exe $(INSTALLPATH_KEYMANDEVELOPER)\kmcomp.exe
$(COPY) $(PROGRAM)\developer\kmcomp.exe "$(INSTALLPATH_KEYMANDEVELOPER)\kmcomp.exe"
!include ..\..\Target.mak

View file

@ -37,8 +37,8 @@ debug-manifest:
copy std-manifest.xml manifest.xml
install:
$(COPY) $(PROGRAM)\engine\keyman.exe $(INSTALLPATH_KEYMANENGINE)
$(COPY) keyman.rsm $(INSTALLPATH_KEYMANENGINE)
$(COPY) $(PROGRAM)\engine\keyman.exe "$(INSTALLPATH_KEYMANENGINE)"
$(COPY) keyman.rsm "$(INSTALLPATH_KEYMANENGINE)"
test-uiaccess:
grep -c "uiAccess=\"true\"" manifest.xml

View file

@ -402,17 +402,16 @@ void DebugMessage(LPMSG msg, WPARAM wParam) // I2908
char ds[256];
if(msg->message == wm_keymankeydown)
wsprintf(ds, "DebugMessage(%x, wm_keymankeydown: %s %x lParam: %X) [message flags: %x time: %d]", msg->hwnd,
msg->wParam < 256 ? KeyNames[msg->wParam] : "???", msg->wParam, msg->lParam, wParam, msg->time);
wsprintf(ds, "DebugMessage(%x, wm_keymankeydown: %s lParam: %X) [message flags: %x time: %d]", msg->hwnd,
Debug_VirtualKey((WORD) msg->wParam), msg->lParam, wParam, msg->time);
else if(msg->message == wm_keymankeyup)
wsprintf(ds, "DebugMessage(%x, wm_keymankeyup: %s %d lParam: %X) [message flags: %x time: %d]", msg->hwnd,
msg->wParam < 256 ? KeyNames[msg->wParam] : "???", msg->wParam, msg->lParam, wParam, msg->time);
wsprintf(ds, "DebugMessage(%x, wm_keymankeyup: %s lParam: %X) [message flags: %x time: %d]", msg->hwnd,
Debug_VirtualKey((WORD) msg->wParam), msg->lParam, wParam, msg->time);
else if(msg->message == WM_KEYDOWN || msg->message == WM_KEYUP || msg->message == WM_SYSKEYDOWN || msg->message == WM_SYSKEYUP)
wsprintf(ds, "DebugMessage(%x, %s, wParam: '%s' (%d), lParam: %X) [message flags: %x time: %d extra: %x]",
wsprintf(ds, "DebugMessage(%x, %s, wParam: %s, lParam: %X) [message flags: %x time: %d extra: %x]",
msg->hwnd,
msgnames[msg->message-WM_KEYDOWN],
(msg->wParam < 256 ? KeyNames[msg->wParam] : "???"),
msg->wParam,
Debug_VirtualKey((WORD) msg->wParam),
msg->lParam,
wParam,
msg->time,
@ -479,7 +478,7 @@ char *Debug_VirtualKey(WORD vk) {
return "";
}
if (vk < 255) {
if (vk < 256) {
wsprintf(buf, "['%s' 0x%x]", KeyNames[vk], vk);
}
else {

View file

@ -25,6 +25,6 @@ backup:
$(WZZIP) $(BUILD)\engine\keyman32.zip $(BACKUPDEFAULTS) keyman32.dll
install:
$(COPY) $(PROGRAM)\engine\keyman32.dll $(INSTALLPATH_KEYMANENGINE)\keyman32.dll
$(COPY) $(PROGRAM)\engine\keyman32.dll "$(INSTALLPATH_KEYMANENGINE)\keyman32.dll"
!include ..\..\Target.mak

View file

@ -65,7 +65,6 @@ void KeyCapsLockPress(BOOL FIsUp) // I3284 - void // I3529
{
if(FIsUp && !(GetKeyState(VK_CAPITAL) & 1)) // I267 - 24/11/2006 invert GetKeyState test
{
//*Globals::ShiftState() |= CAPITALFLAG; // I4548
keybd_event(VK_CAPITAL, 0xFF, 0, 0);
keybd_event(VK_CAPITAL, 0xFF, KEYEVENTF_KEYUP, 0);
}
@ -74,8 +73,6 @@ void KeyCapsLockPress(BOOL FIsUp) // I3284 - void // I3529
{
if(!FIsUp && (GetKeyState(VK_CAPITAL) & 1))
{ // I267 - 24/11/2006 invert GetKeyState test
//FShouldIgnoreNextKey[vkKey] = 2; // KEY DOWN AND THEN UP
//*Globals::ShiftState() &= ~CAPITALFLAG; // I4548
keybd_event(VK_CAPITAL, 0xFF, KEYEVENTF_KEYUP, 0);
keybd_event(VK_CAPITAL, 0xFF, 0, 0);
}
@ -93,15 +90,10 @@ void KeyShiftPress(BOOL FIsUp) // I3284 - void // I3529
if(_td->lpActiveKeyboard->Keyboard->dwFlags & KF_SHIFTFREESCAPS)
{
//SendDebugMessageFormat(0, sdmDebug, 0, "KeyShiftPress -- KF_SHIFTFRESSCAPS [%s]", FIsUp ? "up" : "down");
if(!FIsUp)
{
//*Globals::IgnoreNextCaps() += 2;
//keybd_event(VK_SHIFT, 0xFF, KEYEVENTF_KEYUP, 0);
//*Globals::ShiftState() &= ~CAPITALFLAG; // I4548
keybd_event(VK_CAPITAL, 0xFF, 0, 0);
keybd_event(VK_CAPITAL, 0xFF, KEYEVENTF_KEYUP, 0);
//keybd_event(VK_SHIFT, 0xFF, 0, 0);
}
}
}

View file

@ -25,6 +25,6 @@ backup:
$(WZZIP) $(BUILD)\engine\keyman64.zip $(BACKUPDEFAULTS) keyman64.dll
install:
$(COPY) $(PROGRAM)\engine\keyman64.dll $(INSTALLPATH_KEYMANENGINE)\keyman64.dll
$(COPY) $(PROGRAM)\engine\keyman64.dll "$(INSTALLPATH_KEYMANENGINE)\keyman64.dll"
!include ..\..\Target.mak

View file

@ -26,7 +26,7 @@ backup:
$(WZZIP) $(BUILD)\engine\keymanx64.exe $(BACKUPDEFAULTS) keymanx64.exe
install:
$(COPY) $(PROGRAM)\engine\keymanx64.exe $(INSTALLPATH_KEYMANENGINE)
$(COPY) $(PROGRAM)\engine\keymanx64.exe "$(INSTALLPATH_KEYMANENGINE)"
test-uiaccess:
grep -c "uiAccess=\"true\"" manifest.xml

View file

@ -23,9 +23,9 @@ signcode:
$(SIGNCODE) /d "Keyman Engine" $(PROGRAM)\engine\kmcomapi.dll
install:
regsvr32 /s /u $(INSTALLPATH_KEYMANENGINE)\kmcomapi.dll
$(COPY) $(PROGRAM)\engine\kmcomapi.dll $(INSTALLPATH_KEYMANENGINE)
regsvr32 $(INSTALLPATH_KEYMANENGINE)\kmcomapi.dll
cmd /c regsvr32 /s /u "$(INSTALLPATH_KEYMANENGINE)\kmcomapi.dll"
$(COPY) $(PROGRAM)\engine\kmcomapi.dll "$(INSTALLPATH_KEYMANENGINE)"
cmd /c regsvr32 /s "$(INSTALLPATH_KEYMANENGINE)\kmcomapi.dll"
backup:
$(WZZIP) $(BUILD)\engine\kmcomapi.zip $(BACKUPDEFAULTS) kmcomapi.dll

View file

@ -25,13 +25,7 @@ backup:
$(WZZIP) $(BUILD)\engine\kmtip64.zip x64\kmtip64.pdb $(BACKUPDEFAULTS) x64\kmtip64.dll
install:
cd $(INSTALLPATH_KEYMANENGINE)
#regsvr32 /s /u kmtip.dll
#regsvr32 /s /u kmtip64.dll
$(COPY) $(PROGRAM)\engine\kmtip.dll $(INSTALLPATH_KEYMANENGINE)\kmtip.dll
$(COPY) $(PROGRAM)\engine\kmtip64.dll $(INSTALLPATH_KEYMANENGINE)\kmtip64.dll
#regsvr32 /s kmtip.dll
#regsvr32 /s kmtip64.dll
cd $(ROOT)\src\engine\kmtip
$(COPY) $(PROGRAM)\engine\kmtip.dll "$(INSTALLPATH_KEYMANENGINE)\kmtip.dll"
$(COPY) $(PROGRAM)\engine\kmtip64.dll "$(INSTALLPATH_KEYMANENGINE)\kmtip64.dll"
!include ..\..\Target.mak

View file

@ -22,7 +22,7 @@ backup:
$(WZZIP) $(BUILD)\engine\mcompile.exe $(BACKUPDEFAULTS) mcompile.exe
install:
$(COPY) $(PROGRAM)\engine\mcompile.exe $(INSTALLPATH_KEYMANENGINE)
$(COPY) $(PROGRAM)\engine\mcompile.exe "$(INSTALLPATH_KEYMANENGINE)"
test-manifest:
# test that linked manifest exists and correct

View file

@ -32,8 +32,8 @@ backup:
$(WZZIP) $(BUILD)\engine\tsysinfo.zip $(BACKUPDEFAULTS) tsysinfo.exe
install:
$(COPY) $(PROGRAM)\engine\tsysinfo.exe $(INSTALLPATH_KEYMANENGINE)
$(COPY) $(ROOT)\src\engine\tsysinfo\*.xsl $(INSTALLPATH_KEYMANENGINE)
$(COPY) $(PROGRAM)\engine\tsysinfo.exe "$(INSTALLPATH_KEYMANENGINE)"
$(COPY) $(ROOT)\src\engine\tsysinfo\*.xsl "$(INSTALLPATH_KEYMANENGINE)"
test-manifest:
# test that linked manifest exists and correct

View file

@ -166,12 +166,14 @@ LPSTR KeyNames[256] = {
"?9D", // &H9D
"?9E", // &H9E
"?9F", // &H9F
"?A0", // &HA0
"?A1", // &HA1
"?A2", // &HA2
"?A3", // &HA3
"?A4", // &HA4
"?A5", // &HA5
"?LShift", // &HA0
"?RShift", // &HA1
"?LControl", // &HA2
"?RControl", // &HA3
"?LAlt", // &HA4
"?RAlt", // &HA5
"?A6", // &HA6
"?A7", // &HA7
"?A8", // &HA8