NuRegex isEqual: was incorrectly assuming its argument was a NuRegex.…

… Fixed.
solakv · Jan 13, 2011 · 4f8a2d8 · 4f8a2d8
1 parent e79c2c1
commit 4f8a2d8
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/objc/NuRegex.m b/objc/NuRegex.m
@@ -387,7 +387,7 @@ - (const pcre *)pcre { return regex; }
 
 - (BOOL)isEqual:(NuRegex *)other
 {
-    return (([pattern isEqual: [other pattern]]) && (options == [other options]));
+    return ([other isKindOfClass:[NuRegex class]] && [pattern isEqual:[other pattern]] && (options == [other options]));
 }
 
 @end

diff --git a/test/test_regex.nu b/test/test_regex.nu
@@ -51,7 +51,9 @@
         (assert_equal /extended/x /extended/x)
         (assert_equal /a(.*)z/ /a(.*)z/)
         (assert_not_equal /hello/ /goodbye/)
-        (assert_not_equal /extended/x /extended/)))
+        (assert_not_equal /extended/x /extended/)
+        (assert_not_equal /foo/ nil)
+        (assert_not_equal /foo/ "foo")))