Manual Merge change 3d38805

ID Author Date Message
3d38805 Ray Hayes <[email protected]>
9/20/2016 11:11:06 AM -07:00 Minor fixes for color handling and newline
handling.

Author: manojampalam

contrib/win32/win32compat/ansiprsr.c

@@ -250,8 +250,8 @@ unsigned char* ParseBuffer(unsigned char* pszBuffer, unsigned char* pszBufferEnd
 		case 10:
 			pszBuffer++;
 			AutoWrap = 1;
-		    bAtEOLN = TRUE;
-		    break;
+            GoToNextLine();
+            break;
 
 		case 12:
 		    pszBuffer++;
@@ -510,6 +510,8 @@ unsigned char * ParseANSI(unsigned char * pszBuffer, unsigned char * pszBufferEn
 				else if (bMode & MODE_BRK)
 				{
 					// Cursor UP
+                    if (iParam[0] == 0)
+                        iParam[0] = 1;
 					ConMoveCursorPosition(0, -iParam[0]);
 				}
 				fcompletion = 1;
@@ -523,6 +525,8 @@ unsigned char * ParseANSI(unsigned char * pszBuffer, unsigned char * pszBufferEn
 				else if (bMode & MODE_BRK)
 				{
 					// Cursor DOWN
+                    if (iParam[0] == 0)
+                        iParam[0] = 1;
 					ConMoveCursorPosition(0, iParam[0]);
 				}
 				fcompletion = 1;
@@ -536,6 +540,8 @@ unsigned char * ParseANSI(unsigned char * pszBuffer, unsigned char * pszBufferEn
 				else if (bMode & MODE_BRK)
 				{
 					// Cursor right
+                    if (iParam[0] == 0)
+                        iParam[0] = 1;
 					ConMoveCursorPosition(iParam[0], 0);
 
                 }
@@ -863,7 +869,7 @@ unsigned char * ParseANSI(unsigned char * pszBuffer, unsigned char * pszBufferEn
 		bCS1 = 0;
 		bBkMode = 0;
 		bCharMode = 0;
-		return pszCurrent;
+        return pszCurrent;
 	}
 	else
 		return pszBuffer;
@@ -906,9 +912,9 @@ unsigned char * ParseVT52(unsigned char * pszBuffer, unsigned char * pszBufferEn
 			break;
 
 		case 'H':  // Cursor Home
-			ConSetCursorPosition(1, 1);
+			ConSetCursorPosition(0, 0);
 			pszCurrent++;
-				bAtEOLN = FALSE;
+			bAtEOLN = FALSE;
 			break;
 		case 'I':  // Reverse Line Feed
 			pszCurrent++;

contrib/win32/win32compat/console.c

@@ -104,7 +104,12 @@ int ConInit( DWORD OutputHandle, BOOL fSmartInit )
 
     if ( os.dwPlatformId == VER_PLATFORM_WIN32_NT )
     {
-        dwAttributes = (DWORD)ENABLE_PROCESSED_OUTPUT | ENABLE_VIRTUAL_TERMINAL_PROCESSING;  // PERFECT in NT
+        char *term = getenv("TERM");
+        dwAttributes = (DWORD)ENABLE_PROCESSED_OUTPUT;  // PERFECT in NT
+
+        if (term != NULL && (_stricmp(term, "ansi") == 0 || _stricmp(term, "passthru")))
+            dwAttributes |= (DWORD)ENABLE_VIRTUAL_TERMINAL_PROCESSING;
+
         SetConsoleMode(hOutputConsole, dwAttributes); // Windows NT
     }
     else
@@ -285,19 +290,34 @@ BOOL ConSetScreenSize( int xSize, int ySize )
 /* ************************************************************ */
 void ConSetAttribute(int *iParam, int iParamCount)
 {
-    int		iAttr = 0;
+    static int	iAttr = 0;
     int		i = 0;
+    BOOL    bRet = TRUE;
 
     if (iParamCount < 1)
-        SetConsoleTextAttribute(hOutputConsole, FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE);
+    {
+        iAttr |= FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE;
+
+        iAttr = iAttr & ~BACKGROUND_INTENSITY;
+        iAttr = iAttr & ~FOREGROUND_INTENSITY;
+        iAttr = iAttr & ~COMMON_LVB_UNDERSCORE;
+        iAttr = iAttr & ~COMMON_LVB_REVERSE_VIDEO;
+
+        SetConsoleTextAttribute(hOutputConsole, (WORD)iAttr);
+    }
     else
     {
         for (i=0;i<iParamCount;i++)
         {
             switch (iParam[i])
             {
                 case ANSI_ATTR_RESET:
-                    SetConsoleTextAttribute(hOutputConsole, FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE);
+                    iAttr |= FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE;
+
+                    iAttr = iAttr & ~BACKGROUND_INTENSITY;
+                    iAttr = iAttr & ~FOREGROUND_INTENSITY;
+                    iAttr = iAttr & ~COMMON_LVB_UNDERSCORE;
+                    iAttr = iAttr & ~COMMON_LVB_REVERSE_VIDEO;
                     break;
                 case ANSI_BRIGHT:
                     iAttr |= FOREGROUND_INTENSITY;
@@ -315,7 +335,7 @@ void ConSetAttribute(int *iParam, int iParamCount)
                 case ANSI_HIDDEN:
                     break;
                 case ANSI_NOREVERSE:
-                    iAttr |= FOREGROUND_RED | FOREGROUND_GREEN | FOREGROUND_BLUE;
+                    iAttr = iAttr & ~COMMON_LVB_REVERSE_VIDEO;
                     break;
                 case ANSI_DEFAULT_FOREGROUND:
                     // White
@@ -359,6 +379,9 @@ void ConSetAttribute(int *iParam, int iParamCount)
                     break;
                 case ANSI_DEFAULT_BACKGROUND:
                     //Black
+                    iAttr = iAttr & ~BACKGROUND_RED;
+                    iAttr = iAttr & ~BACKGROUND_BLUE;
+                    iAttr = iAttr & ~BACKGROUND_GREEN;
                     iAttr |= 0;
                     break;
                 case ANSI_BACKGROUND_BLACK:
@@ -400,11 +423,13 @@ void ConSetAttribute(int *iParam, int iParamCount)
                 case ANSI_BACKGROUND_BRIGHT:
                     iAttr |= BACKGROUND_INTENSITY;
                     break;
+                default:
+                    continue;
             }
         }
 
         if (iAttr)
-            SetConsoleTextAttribute(hOutputConsole, (WORD)iAttr);
+            bRet = SetConsoleTextAttribute(hOutputConsole, (WORD)iAttr);
     }
 } // End procedure