Move the cursor after putting a char (dhylands#7)

mcauser · dhylands · commit 4a4ecfcd8bcf · 2017-02-19T23:02:09.000-08:00
Looks good to me. I tested it on a 4x20 LCD and it works the same both before and after the patch.
diff --git a/lcd/lcd_api.py b/lcd/lcd_api.py
@@ -134,15 +134,15 @@ def putchar(self, char):
         """Writes the indicated character to the LCD at the current cursor
         position, and advances the cursor by one position.
         """
+        if char != '\n':
+            self.hal_write_data(ord(char))
+            self.cursor_x += 1
         if self.cursor_x >= self.num_columns or char == '\n':
             self.cursor_x = 0
             self.cursor_y += 1
             if self.cursor_y >= self.num_lines:
                 self.cursor_y = 0
             self.move_to(self.cursor_x, self.cursor_y)
-        if char != '\n':
-            self.hal_write_data(ord(char))
-            self.cursor_x += 1
 
     def putstr(self, string):
         """Write the indicated string to the LCD at the current cursor
