A reasonable fix for mobile-shell#582

Big kudos to @ridiculousfish for https://github.com/ridiculousfish/widecharwidth !
jdrouhard · Jun 28, 2021 · 29408c7 · 29408c7
1 parent 03087e7
commit 29408c7
Show file tree

Hide file tree

Showing 3 changed files with 562 additions and 3 deletions.
diff --git a/src/frontend/terminaloverlay.cc b/src/frontend/terminaloverlay.cc
@@ -37,6 +37,7 @@
 #include <limits.h>
 
 #include "terminaloverlay.h"
+#include "widechar_width.h"
 
 using namespace Overlay;
 
@@ -261,7 +262,7 @@ void NotificationEngine::apply( Framebuffer &fb ) const
     }
 
     wchar_t ch = *i;
-    int chwidth = ch == L'\0' ? -1 : wcwidth( ch );
+    int chwidth = ch == L'\0' ? -1 : widechar_wcwidth( ch );
     Cell *this_cell = 0;
 
     switch ( chwidth ) {
@@ -743,7 +744,7 @@ void PredictionEngine::new_user_byte( char the_byte, const Framebuffer &fb )
 	    }
 	  }
 	}
-      } else if ( (ch < 0x20) || (wcwidth( ch ) != 1) ) {
+      } else if ( (ch < 0x20) || (widechar_wcwidth( ch ) != 1) ) {
 	/* unknown print */
 	become_tentative();
 	//	fprintf( stderr, "Unknown print 0x%x\n", ch );

diff --git a/src/terminal/terminal.cc b/src/terminal/terminal.cc
@@ -37,6 +37,7 @@
 #include <typeinfo>
 
 #include "terminal.h"
+#include "widechar_width.h"
 
 using namespace Terminal;
 
@@ -66,7 +67,7 @@ void Emulator::print( const Parser::Print *act )
    * Check for printing ISO 8859-1 first, it's a cheap way to detect
    * some common narrow characters.
    */
-  const int chwidth = ch == L'\0' ? -1 : ( Cell::isprint_iso8859_1( ch ) ? 1 : wcwidth( ch ));
+  const int chwidth = ch == L'\0' ? -1 : ( Cell::isprint_iso8859_1( ch ) ? 1 : widechar_wcwidth( ch ));
 
   Cell *this_cell = fb.get_mutable_cell();