Fix tests and clarify first character access

katlings · katlings · commit eb3cb4d99bc3 · 2019-05-07T09:30:55.000-07:00
diff --git a/py/repl.c b/py/repl.c
@@ -189,7 +189,7 @@ size_t mp_repl_autocomplete(const char *str, size_t len, const mp_print_t *print
                     if (dest[0] != MP_OBJ_NULL) {
                         // special case; filter out words that begin with underscore
                         // unless there's already a partial match
-                        if (s_len == 0 && *d_str == '_') {
+                        if (s_len == 0 && d_str[0] == '_') {
                             continue;
                         }
                         if (match_str == NULL) {
diff --git a/tests/cmdline/repl_autocomplete.py b/tests/cmdline/repl_autocomplete.py
@@ -7,6 +7,6 @@
 i = str
 i.lowe	('ABC')
 x = 5
-x.	
+x.	
 x._	
 None.	
diff --git a/tests/cmdline/repl_autocomplete.py.exp b/tests/cmdline/repl_autocomplete.py.exp
@@ -12,7 +12,8 @@ Use \.\+
 'abc'
 >>> x = 5
 >>> x.
-from_bytes      to_bytes[K[K
+from_bytes      to_bytes
+>>> x.[K[K
 >>> x.__class__
 <class 'int'>
 >>> None.[K[K[K[K[K

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,7 @@ size_t mp_repl_autocomplete(const char str, size_t len, const mp_print_t print`
`189`	`189`	`if (dest[0] != MP_OBJ_NULL) {`
`190`	`190`	`// special case; filter out words that begin with underscore`
`191`	`191`	`// unless there's already a partial match`
`192`		`- if (s_len == 0 && *d_str == '_') {`
	`192`	`+ if (s_len == 0 && d_str[0] == '_') {`
`193`	`193`	`continue;`
`194`	`194`	`}`
`195`	`195`	`if (match_str == NULL) {`
-Original file line number
+Diff line change
 i = str
 i.lowe	('ABC')
 x = 5
 -x.
 +x.
 x._
 None.