Meson/wren
1
0
Fork 0
forked from Mirror/wren
Code Pull Requests Activity

Make String#contains return a bool.

Browse Source
This commit is contained in:
Bob Nystrom
2013-11-17 10:04:02 -08:00
parent 0181f69dfb
commit 0c1ce91e80
2 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/primitives.c
View File

@ -176,10 +176,9 @@ DEF_PRIMITIVE(string_contains)
const char* search = AS_CSTRING(args[1]);
// Corner case, the empty string contains the empty string.
if (strlen(string) == 0 && strlen(search) == 0) return NUM_VAL(1);
if (strlen(string) == 0 && strlen(search) == 0) return TRUE_VAL;
// TODO(bob): Return bool.
return NUM_VAL(strstr(string, search) != NULL);
return BOOL_VAL(strstr(string, search) != NULL);
}
DEF_PRIMITIVE(string_count)

12
test/string_contains.wren
View File

@ -1,8 +1,8 @@
io.write("".contains("")) // expect: 1
io.write("anything".contains("")) // expect: 1
io.write("something".contains("meth")) // expect: 1
io.write("something".contains("some")) // expect: 1
io.write("something".contains("ing")) // expect: 1
io.write("something".contains("math")) // expect: 0
io.write("".contains("")) // expect: true
io.write("anything".contains("")) // expect: true
io.write("something".contains("meth")) // expect: true
io.write("something".contains("some")) // expect: true
io.write("something".contains("ing")) // expect: true
io.write("something".contains("math")) // expect: false
// TODO(bob): Passing non-string as argument.