Connect Ctrl key in WinBoard
[xboard.git] / winboard / wengineoutput.c
index d05b8ff..47bca87 100644 (file)
@@ -1,26 +1,30 @@
 /*\r
- * Engine output (PV)\r
+ * wengineoutput.c - split-off front-end of Engine output (PV) by HGM\r
  *\r
  * Author: Alessandro Scotti (Dec 2005)\r
  *\r
+ * Copyright 2005 Alessandro Scotti\r
+ *\r
+ * Enhancements Copyright 2009, 2010, 2011, 2012, 2013 Free Software Foundation, Inc.\r
+ *\r
  * ------------------------------------------------------------------------\r
- *
- * GNU XBoard is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or (at
- * your option) any later version.
- *
- * GNU XBoard is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program. If not, see http://www.gnu.org/licenses/.
- *
- *------------------------------------------------------------------------
- ** See the file ChangeLog for a revision history.  */
-
+ *\r
+ * GNU XBoard is free software: you can redistribute it and/or modify\r
+ * it under the terms of the GNU General Public License as published by\r
+ * the Free Software Foundation, either version 3 of the License, or (at\r
+ * your option) any later version.\r
+ *\r
+ * GNU XBoard is distributed in the hope that it will be useful, but\r
+ * WITHOUT ANY WARRANTY; without even the implied warranty of\r
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU\r
+ * General Public License for more details.\r
+ *\r
+ * You should have received a copy of the GNU General Public License\r
+ * along with this program. If not, see http://www.gnu.org/licenses/.  *\r
+ *\r
+ *------------------------------------------------------------------------\r
+ ** See the file ChangeLog for a revision history.  */\r
+\r
 #include "config.h"\r
 \r
 #include <windows.h> /* required for all Windows applications */\r
 #include <dlgs.h>\r
 \r
 #include "common.h"\r
-#include "winboard.h"\r
 #include "frontend.h"\r
 #include "backend.h"\r
+#include "winboard.h"\r
 \r
 #include "wsnap.h"\r
-\r
-VOID EngineOutputPopUp();\r
-VOID EngineOutputPopDown();\r
-BOOL EngineOutputIsUp();\r
-\r
-#define SHOW_PONDERING\r
-\r
-/* Imports from backend.c */\r
-char * SavePart(char *str);\r
-\r
-/* Imports from winboard.c */\r
-extern HWND engineOutputDialog;\r
-extern BOOLEAN engineOutputDialogUp;\r
-\r
-extern HINSTANCE hInst;\r
-extern HWND hwndMain;\r
-\r
-extern WindowPlacement wpEngineOutput;\r
+#include "engineoutput.h"\r
 \r
 /* Module variables */\r
-#define H_MARGIN            2\r
-#define V_MARGIN            2\r
-#define LABEL_V_DISTANCE    1   /* Distance between label and memo */\r
-#define SPLITTER_SIZE       4   /* Distance between first memo and second label */\r
-\r
-#define ICON_SIZE           14\r
-\r
-#define STATE_UNKNOWN   -1\r
-#define STATE_THINKING   0\r
-#define STATE_IDLE       1\r
-#define STATE_PONDERING  2\r
-#define STATE_ANALYZING  3\r
-\r
-static int  windowMode = 1;\r
-\r
-static BOOL needInit = TRUE;\r
-\r
-static HICON hiColorBlack = NULL;\r
-static HICON hiColorWhite = NULL;\r
-static HICON hiColorUnknown = NULL;\r
-static HICON hiClear = NULL;\r
-static HICON hiPondering = NULL;\r
-static HICON hiThinking = NULL;\r
-static HICON hiAnalyzing = NULL;\r
-\r
-static int  lastDepth[2] = { -1, -1 };\r
-static int  lastForwardMostMove[2] = { -1, -1 };\r
-static int  engineState[2] = { -1, -1 };\r
-\r
-typedef struct {\r
-    HWND hColorIcon;\r
-    HWND hLabel;\r
-    HWND hStateIcon;\r
-    HWND hStateData;\r
-    HWND hLabelNPS;\r
-    HWND hMemo;\r
-    char * name;\r
-    int which;\r
-    int depth;\r
-    unsigned long nodes;\r
-    int score;\r
-    int time;\r
-    char * pv;\r
-    char * hint;\r
-    int an_move_index;\r
-    int an_move_count;\r
-} EngineOutputData;\r
+int  windowMode = 1;\r
+static BOOLEAN engineOutputDialogUp = FALSE;\r
+HICON icons[8]; // [HGM] this front-end array translates back-end icon indicator to handle\r
+HWND outputField[2][7]; // [HGM] front-end array to translate output field to window handle\r
 \r
+// front end\r
 static HICON LoadIconEx( int id )\r
 {\r
     return LoadImage( hInst, MAKEINTRESOURCE(id), IMAGE_ICON, ICON_SIZE, ICON_SIZE, 0 );\r
 }\r
 \r
-static VOID InitializeEngineOutput()\r
+// [HGM] the platform-dependent way of indicating where output should go is now all\r
+// concentrated here, where a table of platform-dependent handles are initialized.\r
+// This cleanses most other routines of front-end stuff, so they can go into the back end.\r
+static void InitializeEngineOutput()\r
 {\r
-    if( needInit ) {\r
-        hiColorBlack = LoadIconEx( IDI_BLACK_14 );\r
-        hiColorWhite = LoadIconEx( IDI_WHITE_14 );\r
-        hiColorUnknown = LoadIconEx( IDI_UNKNOWN_14 );\r
-        hiClear = LoadIconEx( IDI_TRANS_14 );\r
-        hiPondering = LoadIconEx( IDI_PONDER_14 );\r
-        hiThinking = LoadIconEx( IDI_CLOCK_14 );\r
-        hiAnalyzing = LoadIconEx( IDI_ANALYZE2_14 );\r
-        needInit = FALSE;\r
-    }\r
+       // [HGM] made this into a table, rather than separate global variables\r
+        icons[nColorBlack]   = LoadIconEx( IDI_BLACK_14 );\r
+        icons[nColorWhite]   = LoadIconEx( IDI_WHITE_14 );\r
+        icons[nColorUnknown] = LoadIconEx( IDI_UNKNOWN_14 );\r
+        icons[nClear]        = LoadIconEx( IDI_TRANS_14 );\r
+        icons[nPondering]    = LoadIconEx( IDI_PONDER_14 );\r
+        icons[nThinking]     = LoadIconEx( IDI_CLOCK_14 );\r
+        icons[nAnalyzing]    = LoadIconEx( IDI_ANALYZE2_14 );\r
+\r
+       // [HGM] also make a table of handles to output controls\r
+       // Note that engineOutputDialog must be defined first!\r
+        outputField[0][nColorIcon] = GetDlgItem( engineOutputDialog, IDC_Color1 );\r
+        outputField[0][nLabel]     = GetDlgItem( engineOutputDialog, IDC_EngineLabel1 );\r
+        outputField[0][nStateIcon] = GetDlgItem( engineOutputDialog, IDC_StateIcon1 );\r
+        outputField[0][nStateData] = GetDlgItem( engineOutputDialog, IDC_StateData1 );\r
+        outputField[0][nLabelNPS]  = GetDlgItem( engineOutputDialog, IDC_Engine1_NPS );\r
+        outputField[0][nMemo]      = GetDlgItem( engineOutputDialog, IDC_EngineMemo1 );\r
+\r
+        outputField[1][nColorIcon] = GetDlgItem( engineOutputDialog, IDC_Color2 );\r
+        outputField[1][nLabel]     = GetDlgItem( engineOutputDialog, IDC_EngineLabel2 );\r
+        outputField[1][nStateIcon] = GetDlgItem( engineOutputDialog, IDC_StateIcon2 );\r
+        outputField[1][nStateData] = GetDlgItem( engineOutputDialog, IDC_StateData2 );\r
+        outputField[1][nLabelNPS]  = GetDlgItem( engineOutputDialog, IDC_Engine2_NPS );\r
+        outputField[1][nMemo]      = GetDlgItem( engineOutputDialog, IDC_EngineMemo2 );\r
 }\r
 \r
-static VOID SetControlPos( HWND hDlg, int id, int x, int y, int width, int height )\r
+// front end\r
+static void SetControlPos( HWND hDlg, int id, int x, int y, int width, int height )\r
 {\r
     HWND hControl = GetDlgItem( hDlg, id );\r
 \r
@@ -134,7 +97,8 @@ static VOID SetControlPos( HWND hDlg, int id, int x, int y, int width, int heigh
 #define HIDDEN_X    20000\r
 #define HIDDEN_Y    20000\r
 \r
-static VOID HideControl( HWND hDlg, int id )\r
+// front end\r
+static void HideControl( HWND hDlg, int id )\r
 {\r
     HWND hControl = GetDlgItem( hDlg, id );\r
     RECT rc;\r
@@ -150,6 +114,7 @@ static VOID HideControl( HWND hDlg, int id )
     }\r
 }\r
 \r
+// front end, although we might make GetWindowRect front end instead\r
 static int GetControlWidth( HWND hDlg, int id )\r
 {\r
     RECT rc;\r
@@ -159,6 +124,7 @@ static int GetControlWidth( HWND hDlg, int id )
     return rc.right - rc.left;\r
 }\r
 \r
+// front end?\r
 static int GetControlHeight( HWND hDlg, int id )\r
 {\r
     RECT rc;\r
@@ -177,76 +143,9 @@ static int GetHeaderHeight()
     return result;\r
 }\r
 \r
-#define ENGINE_COLOR_WHITE      'w'\r
-#define ENGINE_COLOR_BLACK      'b'\r
-#define ENGINE_COLOR_UNKNOWN    ' '\r
-\r
-char GetEngineColor( int which )\r
-{\r
-    char result = ENGINE_COLOR_UNKNOWN;\r
-\r
-    if( which == 0 || which == 1 ) {\r
-        ChessProgramState * cps;\r
-\r
-        switch (gameMode) {\r
-        case MachinePlaysBlack:\r
-        case IcsPlayingBlack:\r
-            result = ENGINE_COLOR_BLACK;\r
-            break;\r
-        case MachinePlaysWhite:\r
-        case IcsPlayingWhite:\r
-            result = ENGINE_COLOR_WHITE;\r
-            break;\r
-        case AnalyzeMode:\r
-        case AnalyzeFile:\r
-            result = WhiteOnMove(forwardMostMove) ? ENGINE_COLOR_WHITE : ENGINE_COLOR_BLACK;\r
-            break;\r
-        case TwoMachinesPlay:\r
-            cps = (which == 0) ? &first : &second;\r
-            result = cps->twoMachinesColor[0];\r
-            result = result == 'w' ? ENGINE_COLOR_WHITE : ENGINE_COLOR_BLACK;\r
-            break;\r
-        }\r
-    }\r
-\r
-    return result;\r
-}\r
-\r
-char GetActiveEngineColor()\r
-{\r
-    char result = ENGINE_COLOR_UNKNOWN;\r
-\r
-    if( gameMode == TwoMachinesPlay ) {\r
-        result = WhiteOnMove(forwardMostMove) ? ENGINE_COLOR_WHITE : ENGINE_COLOR_BLACK;\r
-    }\r
-\r
-    return result;\r
-}\r
-\r
-static int IsEnginePondering( int which )\r
-{\r
-    int result = FALSE;\r
-\r
-    switch (gameMode) {\r
-    case MachinePlaysBlack:\r
-    case IcsPlayingBlack:\r
-        if( WhiteOnMove(forwardMostMove) ) result = TRUE;\r
-        break;\r
-    case MachinePlaysWhite:\r
-    case IcsPlayingWhite:\r
-        if( ! WhiteOnMove(forwardMostMove) ) result = TRUE;\r
-        break;\r
-    case TwoMachinesPlay:\r
-        if( GetActiveEngineColor() != ENGINE_COLOR_UNKNOWN ) {\r
-            if( GetEngineColor( which ) != GetActiveEngineColor() ) result = TRUE;\r
-        }\r
-        break;\r
-    }\r
-\r
-    return result;\r
-}\r
-\r
-static VOID PositionControlSet( HWND hDlg, int x, int y, int clientWidth, int memoHeight, int idColor, int idEngineLabel, int idNPS, int idMemo, int idStateIcon, int idStateData )\r
+// The size calculations should be backend? If setControlPos is a platform-dependent way of doing things,\r
+// a platform-independent wrapper for it should be supplied.\r
+static void PositionControlSet( HWND hDlg, int x, int y, int clientWidth, int memoHeight, int idColor, int idEngineLabel, int idNPS, int idMemo, int idStateIcon, int idStateData )\r
 {\r
     int label_x = x + ICON_SIZE + H_MARGIN;\r
     int label_h = GetControlHeight( hDlg, IDC_EngineLabel1 );\r
@@ -267,13 +166,15 @@ static VOID PositionControlSet( HWND hDlg, int x, int y, int clientWidth, int me
     SetControlPos( hDlg, idMemo, x, memo_y, max_w, memoHeight );\r
 }\r
 \r
-static VOID ResizeWindowControls( HWND hDlg, int mode )\r
+// Also here some of the size calculations should go to the back end, and their actual application to a front-end routine\r
+void ResizeWindowControls( int mode )\r
 {\r
+    HWND hDlg = engineOutputDialog; // [HGM] used to be parameter, but routine is called from back-end\r
     RECT rc;\r
     int headerHeight = GetHeaderHeight();\r
-    int labelHeight = GetControlHeight( hDlg, IDC_EngineLabel1 );\r
-    int labelOffset = H_MARGIN + ICON_SIZE + H_MARGIN;\r
-    int labelDeltaY = ICON_SIZE - labelHeight;\r
+//    int labelHeight = GetControlHeight( hDlg, IDC_EngineLabel1 );\r
+//    int labelOffset = H_MARGIN + ICON_SIZE + H_MARGIN;\r
+//    int labelDeltaY = ICON_SIZE - labelHeight;\r
     int clientWidth;\r
     int clientHeight;\r
     int maxControlWidth;\r
@@ -324,237 +225,63 @@ static VOID ResizeWindowControls( HWND hDlg, int mode )
     InvalidateRect( GetDlgItem(hDlg,IDC_EngineMemo2), NULL, FALSE );\r
 }\r
 \r
-static VOID SetDisplayMode( int mode )\r
-{\r
-    if( windowMode != mode ) {\r
-        windowMode = mode;\r
-\r
-        ResizeWindowControls( engineOutputDialog, mode );\r
-    }\r
+static int currentPV, highTextStart[2], highTextEnd[2];\r
+extern RECT boardRect;\r
+\r
+VOID\r
+GetMemoLine(HWND hDlg, int x, int y)\r
+{      // [HGM] pv: translate click to PV line, and load it for display\r
+       char buf[10000];\r
+       int index, start, end, memo;\r
+       POINT pt;\r
+\r
+       pt.x = x; pt.y = y;\r
+       memo = currentPV ? IDC_EngineMemo2 : IDC_EngineMemo1;\r
+       index = SendDlgItemMessage( hDlg, memo, EM_CHARFROMPOS, 0, (LPARAM) &pt );\r
+       GetDlgItemText(hDlg, memo, buf, sizeof(buf));\r
+       if(LoadMultiPV(x, y, buf, index, &start, &end, currentPV)) {\r
+           SetCapture(hDlg);\r
+           SendMessage( outputField[currentPV][nMemo], EM_SETSEL, (WPARAM)start, (LPARAM)end );\r
+           highTextStart[currentPV] = start; highTextEnd[currentPV] = end;\r
+           SetFocus(outputField[currentPV][nMemo]);\r
+       }\r
 }\r
 \r
-static VOID VerifyDisplayMode()\r
+// front end. Actual printing of PV lines into the output field\r
+void InsertIntoMemo( int which, char * text, int where )\r
 {\r
-    int mode;\r
-\r
-    /* Get proper mode for current game */\r
-    switch( gameMode ) {\r
-    case AnalyzeMode:\r
-    case AnalyzeFile:\r
-    case MachinePlaysWhite:\r
-    case MachinePlaysBlack:\r
-    case IcsPlayingWhite:\r
-    case IcsPlayingBlack:\r
-        mode = 0;\r
-        break;\r
-    case TwoMachinesPlay:\r
-        mode = 1;\r
-        break;\r
-    default:\r
-        /* Do not change */\r
-        return;\r
-    }\r
+    SendMessage( outputField[which][nMemo], EM_SETSEL, where, where ); // [HGM] multivar: choose insertion point\r
 \r
-    SetDisplayMode( mode );\r
-}\r
-\r
-static VOID InsertIntoMemo( HWND hMemo, char * text )\r
-{\r
-    SendMessage( hMemo, EM_SETSEL, 0, 0 );\r
-\r
-    SendMessage( hMemo, EM_REPLACESEL, (WPARAM) FALSE, (LPARAM) text );\r
-}\r
-\r
-static VOID SetIcon( HWND hControl, HICON hIcon )\r
-{\r
-    if( hIcon != NULL ) {\r
-        SendMessage( hControl, STM_SETICON, (WPARAM) hIcon, 0 );\r
+    SendMessage( outputField[which][nMemo], EM_REPLACESEL, (WPARAM) FALSE, (LPARAM) text );\r
+    if(where < highTextStart[which]) { // [HGM] multiPVdisplay: move highlighting\r
+       int len = strlen(text);\r
+       highTextStart[which] += len; highTextEnd[which] += len;\r
+       SendMessage( outputField[which][nMemo], EM_SETSEL, highTextStart[which], highTextEnd[which] );\r
     }\r
 }\r
 \r
-static VOID SetEngineColorIcon( HWND hControl, int which )\r
+// front end. Associates an icon with an output field ("control" in Windows jargon).\r
+// [HGM] let it find out the output field from the 'which' number by itself\r
+void SetIcon( int which, int field, int nIcon )\r
 {\r
-    char color = GetEngineColor(which);\r
-    HICON hicon = NULL;\r
 \r
-    if( color == ENGINE_COLOR_BLACK )\r
-        hicon = hiColorBlack;\r
-    else if( color == ENGINE_COLOR_WHITE )\r
-        hicon = hiColorWhite;\r
-    else\r
-        hicon = hiColorUnknown;\r
-\r
-    SetIcon( hControl, hicon );\r
+    if( nIcon != 0 ) {\r
+        SendMessage( outputField[which][field], STM_SETICON, (WPARAM) icons[nIcon], 0 );\r
+    }\r
 }\r
 \r
-static SetEngineState( int which, int state, char * state_data )\r
+// front end wrapper for SetWindowText, taking control number in stead of handle\r
+void DoSetWindowText(int which, int field, char *s_label)\r
 {\r
-    int x_which = 1 - which;\r
-    HWND hStateIcon = GetDlgItem( engineOutputDialog, which == 0 ? IDC_StateIcon1 : IDC_StateIcon2 );\r
-    HWND hStateData = GetDlgItem( engineOutputDialog, which == 0 ? IDC_StateData1 : IDC_StateData2 );\r
-\r
-    if( engineState[ which ] != state ) {\r
-        engineState[ which ] = state;\r
-\r
-        switch( state ) {\r
-        case STATE_THINKING:\r
-            SetIcon( hStateIcon, hiThinking );\r
-            if( engineState[ x_which ] == STATE_THINKING ) {\r
-                SetEngineState( x_which, STATE_IDLE, "" );\r
-            }\r
-            break;\r
-        case STATE_PONDERING:\r
-            SetIcon( hStateIcon, hiPondering );\r
-            break;\r
-        case STATE_ANALYZING:\r
-            SetIcon( hStateIcon, hiAnalyzing );\r
-            break;\r
-        default:\r
-            SetIcon( hStateIcon, hiClear );\r
-            break;\r
-        }\r
-    }\r
-\r
-    if( state_data != 0 ) {\r
-        SetWindowText( hStateData, state_data );\r
-    }\r
+    SetWindowText( outputField[which][field], s_label );\r
 }\r
 \r
-#define MAX_NAME_LENGTH 32\r
-\r
-static VOID UpdateControls( EngineOutputData * ed )\r
+void SetEngineOutputTitle(char *title)\r
 {\r
-    BOOL isPondering = FALSE;\r
-\r
-    char s_label[MAX_NAME_LENGTH + 32];\r
-    \r
-    char * name = ed->name;\r
-\r
-    /* Label */\r
-    if( name == 0 || *name == '\0' ) {\r
-        name = "?";\r
-    }\r
-\r
-    strncpy( s_label, name, MAX_NAME_LENGTH );\r
-    s_label[ MAX_NAME_LENGTH-1 ] = '\0';\r
-\r
-#ifdef SHOW_PONDERING\r
-    if( IsEnginePondering( ed->which ) ) {\r
-        char buf[8];\r
-\r
-        buf[0] = '\0';\r
-\r
-        if( ed->hint != 0 && *ed->hint != '\0' ) {\r
-            strncpy( buf, ed->hint, sizeof(buf) );\r
-            buf[sizeof(buf)-1] = '\0';\r
-        }\r
-        else if( ed->pv != 0 && *ed->pv != '\0' ) {\r
-            char * sep = strchr( ed->pv, ' ' );\r
-            int buflen = sizeof(buf);\r
-\r
-            if( sep != NULL ) {\r
-                buflen = sep - ed->pv + 1;\r
-                if( buflen > sizeof(buf) ) buflen = sizeof(buf);\r
-            }\r
-\r
-            strncpy( buf, ed->pv, buflen );\r
-            buf[ buflen-1 ] = '\0';\r
-        }\r
-\r
-        SetEngineState( ed->which, STATE_PONDERING, buf );\r
-    }\r
-    else if( gameMode == TwoMachinesPlay ) {\r
-        SetEngineState( ed->which, STATE_THINKING, "" );\r
-    }\r
-    else if( gameMode == AnalyzeMode || gameMode == AnalyzeFile ) {\r
-        char buf[64];\r
-        int time_secs = ed->time / 100;\r
-        int time_mins = time_secs / 60;\r
-\r
-        buf[0] = '\0';\r
-\r
-        if( ed->an_move_index != 0 && ed->an_move_count != 0 && *ed->hint != '\0' ) {\r
-            char mov[16];\r
-\r
-            strncpy( mov, ed->hint, sizeof(mov) );\r
-            mov[ sizeof(mov)-1 ] = '\0';\r
-\r
-            sprintf( buf, "%d/%d: %s [%02d:%02d:%02d]", ed->an_move_index, ed->an_move_count, mov, time_mins / 60, time_mins % 60, time_secs % 60 );\r
-        }\r
-\r
-        SetEngineState( ed->which, STATE_ANALYZING, buf );\r
-    }\r
-    else {\r
-        SetEngineState( ed->which, STATE_IDLE, "" );\r
-    }\r
-#endif\r
-\r
-    SetWindowText( ed->hLabel, s_label );\r
-\r
-    s_label[0] = '\0';\r
-\r
-    if( ed->time > 0 && ed->nodes > 0 ) {\r
-        unsigned long nps_100 = ed->nodes / ed->time;\r
-\r
-        if( nps_100 < 100000 ) {\r
-            sprintf( s_label, "NPS: %lu", nps_100 * 100 );\r
-        }\r
-        else {\r
-            sprintf( s_label, "NPS: %.1fk", nps_100 / 10.0 );\r
-        }\r
-    }\r
-\r
-    SetWindowText( ed->hLabelNPS, s_label );\r
-\r
-    /* Memo */\r
-    if( ed->pv != 0 && *ed->pv != '\0' ) {\r
-        char s_nodes[24];\r
-        char s_score[16];\r
-        char s_time[24];\r
-        char buf[256];\r
-        int buflen;\r
-        int time_secs = ed->time / 100;\r
-        int time_cent = ed->time % 100;\r
-\r
-        /* Nodes */\r
-        if( ed->nodes < 1000000 ) {\r
-            sprintf( s_nodes, "%lu", ed->nodes );\r
-        }\r
-        else {\r
-            sprintf( s_nodes, "%.1fM", ed->nodes / 1000000.0 );\r
-        }\r
-\r
-        /* Score */\r
-        if( ed->score > 0 ) {\r
-            sprintf( s_score, "+%.2f", ed->score / 100.0 );\r
-        }\r
-        else {\r
-            sprintf( s_score, "%.2f", ed->score / 100.0 );\r
-        }\r
-\r
-        /* Time */\r
-        sprintf( s_time, "%d:%02d.%02d", time_secs / 60, time_secs % 60, time_cent );\r
-\r
-        /* Put all together... */\r
-        sprintf( buf, "%3d\t%s\t%s\t%s\t", ed->depth, s_score, s_nodes, s_time );\r
-\r
-        /* Add PV */\r
-        buflen = strlen(buf);\r
-\r
-        strncpy( buf + buflen, ed->pv, sizeof(buf) - buflen );\r
-\r
-        buf[ sizeof(buf) - 3 ] = '\0';\r
-\r
-        strcat( buf + buflen, "\r\n" );\r
-\r
-        /* Update memo */\r
-        InsertIntoMemo( ed->hMemo, buf );\r
-    }\r
-\r
-    /* Colors */\r
-    SetEngineColorIcon( ed->hColorIcon, ed->which );\r
+    SetWindowText( engineOutputDialog, title );\r
 }\r
 \r
+// This seems pure front end\r
 LRESULT CALLBACK EngineOutputProc( HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam )\r
 {\r
     static SnapData sd;\r
@@ -564,9 +291,17 @@ LRESULT CALLBACK EngineOutputProc( HWND hDlg, UINT message, WPARAM wParam, LPARA
         if( engineOutputDialog == NULL ) {\r
             engineOutputDialog = hDlg;\r
 \r
+            Translate(hDlg, DLG_EngineOutput);\r
             RestoreWindowPlacement( hDlg, &wpEngineOutput ); /* Restore window placement */\r
 \r
-            ResizeWindowControls( hDlg, windowMode );\r
+            ResizeWindowControls( windowMode );\r
+\r
+            SendDlgItemMessage( hDlg, IDC_EngineMemo1, EM_SETEVENTMASK, 0, ENM_MOUSEEVENTS );\r
+            SendDlgItemMessage( hDlg, IDC_EngineMemo2, EM_SETEVENTMASK, 0, ENM_MOUSEEVENTS );\r
+\r
+           /* Set font */\r
+           SendDlgItemMessage( engineOutputDialog, IDC_EngineMemo1, WM_SETFONT, (WPARAM)font[boardSize][MOVEHISTORY_FONT]->hf, MAKELPARAM(TRUE, 0 ));\r
+           SendDlgItemMessage( engineOutputDialog, IDC_EngineMemo2, WM_SETFONT, (WPARAM)font[boardSize][MOVEHISTORY_FONT]->hf, MAKELPARAM(TRUE, 0 ));\r
 \r
             SetEngineState( 0, STATE_IDLE, "" );\r
             SetEngineState( 1, STATE_IDLE, "" );\r
@@ -590,6 +325,28 @@ LRESULT CALLBACK EngineOutputProc( HWND hDlg, UINT message, WPARAM wParam, LPARA
 \r
         break;\r
 \r
+    case WM_MOUSEMOVE:\r
+        MovePV(LOWORD(lParam) - boardRect.left, HIWORD(lParam) - boardRect.top, boardRect.bottom - boardRect.top);\r
+        break;\r
+\r
+    case WM_RBUTTONUP:\r
+        ReleaseCapture();\r
+        SendMessage( outputField[currentPV][nMemo], EM_SETSEL, 0, 0 );\r
+        highTextStart[currentPV] = highTextEnd[currentPV] = 0;\r
+        UnLoadPV();\r
+        break;\r
+\r
+    case WM_NOTIFY:\r
+        if( wParam == IDC_EngineMemo1 || wParam == IDC_EngineMemo2 ) {\r
+            MSGFILTER * lpMF = (MSGFILTER *) lParam;\r
+            if( lpMF->msg == WM_RBUTTONDOWN && (lpMF->wParam & (MK_CONTROL)) == 0 ) {\r
+               shiftKey = (lpMF->wParam & MK_SHIFT) != 0; // [HGM] remember last shift status\r
+                currentPV = (wParam == IDC_EngineMemo2);\r
+                GetMemoLine(hDlg, LOWORD(lpMF->lParam), HIWORD(lpMF->lParam));\r
+            }\r
+        }\r
+        break;\r
+\r
     case WM_GETMINMAXINFO:\r
         {\r
             MINMAXINFO * mmi = (MINMAXINFO *) lParam;\r
@@ -604,7 +361,7 @@ LRESULT CALLBACK EngineOutputProc( HWND hDlg, UINT message, WPARAM wParam, LPARA
         break;\r
 \r
     case WM_SIZE:\r
-        ResizeWindowControls( hDlg, windowMode );\r
+        ResizeWindowControls( windowMode );\r
         break;\r
 \r
     case WM_ENTERSIZEMOVE:\r
@@ -623,13 +380,11 @@ LRESULT CALLBACK EngineOutputProc( HWND hDlg, UINT message, WPARAM wParam, LPARA
     return FALSE;\r
 }\r
 \r
-VOID EngineOutputPopUp()\r
+// front end\r
+void EngineOutputPopUp()\r
 {\r
   FARPROC lpProc;\r
-\r
-  if( needInit ) {\r
-      InitializeEngineOutput();\r
-  }\r
+  static int  needInit = TRUE;\r
   \r
   CheckMenuItem(GetMenu(hwndMain), IDM_ShowEngineOutput, MF_CHECKED);\r
 \r
@@ -649,11 +404,17 @@ VOID EngineOutputPopUp()
     FreeProcInstance(lpProc);\r
   }\r
 \r
+  // [HGM] displaced to after creation of dialog, to allow initialization of output fields\r
+  if( needInit ) {\r
+      InitializeEngineOutput();\r
+      needInit = FALSE;\r
+  }\r
+\r
   engineOutputDialogUp = TRUE;\r
-  ShowThinkingEvent(); // [HGM] thinking: might need to prompt engine for thinking output
 }\r
 \r
-VOID EngineOutputPopDown()\r
+// front end\r
+void EngineOutputPopDown()\r
 {\r
   CheckMenuItem(GetMenu(hwndMain), IDM_ShowEngineOutput, MF_UNCHECKED);\r
 \r
@@ -662,92 +423,22 @@ VOID EngineOutputPopDown()
   }\r
 \r
   engineOutputDialogUp = FALSE;\r
-  ShowThinkingEvent(); // [HGM] thinking: might need to shut off thinking output
 }\r
 \r
-BOOL EngineOutputIsUp()\r
+// front end. [HGM] Takes handle of output control from table, so only number is passed\r
+void DoClearMemo(int which)\r
 {\r
-    return engineOutputDialogUp;\r
+        SendMessage( outputField[which][nMemo], WM_SETTEXT, 0, (LPARAM) "" );\r
 }\r
 \r
-VOID EngineOutputUpdate( FrontEndProgramStats * stats )\r
+// front end (because only other front-end wants to know)\r
+int EngineOutputIsUp()\r
 {\r
-    EngineOutputData ed;\r
-    BOOL clearMemo = FALSE;\r
-    int which;\r
-    int depth;\r
-\r
-    if( stats == 0 ) {\r
-        SetEngineState( 0, STATE_IDLE, "" );\r
-        SetEngineState( 1, STATE_IDLE, "" );\r
-        return;\r
-    }\r
-\r
-    which = stats->which;\r
-    depth = stats->depth;\r
-\r
-    if( which < 0 || which > 1 || depth < 0 || stats->time < 0 || stats->pv == 0 ) {\r
-        return;\r
-    }\r
-\r
-    if( engineOutputDialog == NULL ) {\r
-        return;\r
-    }\r
-\r
-    VerifyDisplayMode();\r
-\r
-    ed.which = which;\r
-    ed.depth = depth;\r
-    ed.nodes = stats->nodes;\r
-    ed.score = stats->score;\r
-    ed.time = stats->time;\r
-    ed.pv = stats->pv;\r
-    ed.hint = stats->hint;\r
-    ed.an_move_index = stats->an_move_index;\r
-    ed.an_move_count = stats->an_move_count;\r
-\r
-    /* Get target control */\r
-    if( which == 0 ) {\r
-        ed.hColorIcon = GetDlgItem( engineOutputDialog, IDC_Color1 );\r
-        ed.hLabel = GetDlgItem( engineOutputDialog, IDC_EngineLabel1 );\r
-        ed.hStateIcon = GetDlgItem( engineOutputDialog, IDC_StateIcon1 );\r
-        ed.hStateData = GetDlgItem( engineOutputDialog, IDC_StateData1 );\r
-        ed.hLabelNPS = GetDlgItem( engineOutputDialog, IDC_Engine1_NPS );\r
-        ed.hMemo  = GetDlgItem( engineOutputDialog, IDC_EngineMemo1 );\r
-        ed.name = first.tidy;\r
-    }\r
-    else {\r
-        ed.hColorIcon = GetDlgItem( engineOutputDialog, IDC_Color2 );\r
-        ed.hLabel = GetDlgItem( engineOutputDialog, IDC_EngineLabel2 );\r
-        ed.hStateIcon = GetDlgItem( engineOutputDialog, IDC_StateIcon2 );\r
-        ed.hStateData = GetDlgItem( engineOutputDialog, IDC_StateData2 );\r
-        ed.hLabelNPS = GetDlgItem( engineOutputDialog, IDC_Engine2_NPS );\r
-        ed.hMemo  = GetDlgItem( engineOutputDialog, IDC_EngineMemo2 );\r
-        ed.name = second.tidy;\r
-    }\r
-\r
-    /* Clear memo if needed */\r
-    if( lastDepth[which] > depth || (lastDepth[which] == depth && depth <= 1) ) {\r
-        clearMemo = TRUE;\r
-    }\r
-\r
-    if( lastForwardMostMove[which] != forwardMostMove ) {\r
-        clearMemo = TRUE;\r
-    }\r
-\r
-    if( clearMemo ) {\r
-        SendMessage( ed.hMemo, WM_SETTEXT, 0, (LPARAM) "" );\r
-    }\r
-\r
-    /* Update */\r
-    lastDepth[which] = depth;\r
-    lastForwardMostMove[which] = forwardMostMove;\r
-\r
-    if( ed.pv != 0 && ed.pv[0] == ' ' ) {\r
-        if( strncmp( ed.pv, " no PV", 6 ) == 0 ) { /* Hack on hack! :-O */\r
-            ed.pv = "";\r
-        }\r
-    }\r
+    return engineOutputDialogUp;\r
+}\r
 \r
-    UpdateControls( &ed );\r
+// front end, to give back-end access to engineOutputDialog\r
+int EngineOutputDialogExists()\r
+{\r
+    return engineOutputDialog != NULL;\r
 }\r