Stas Bekman <[EMAIL PROTECTED]> writes:

> Joe Schaefer wrote:
>> Cool;  I'm wondering why not just use fallback => 1 and remove the '=='
>> overload?
>
> it doesn't seem to work. I did as you've suggested (to the current cvs):
>
>    'fallback' => 1,
>
> instead of:
>
>    '==' => \&num_cmp;
>
> and t/error/runtime.t now fails:

You have to remove "nomethod" also; all current tests pass for
me with this patch:

Index: xs/APR/Error/Error_pm
===================================================================
--- xs/APR/Error/Error_pm       (revision 156118)
+++ xs/APR/Error/Error_pm       (working copy)
@@ -4,10 +4,11 @@
 use APR::Util ();
 
 use overload
-    nomethod => \&fatal,
+     fallback => 1,
+#    nomethod => \&fatal,
     'bool'   => \&str,
-    '=='     => \&num_cmp,
-    '!='     => \&num_cmp_not,
+#    '=='     => \&num_cmp,
+#    '!='     => \&num_cmp_not,
     '0+'     => \&num,
     '""'     => \&str;
 


-- 
Joe Schaefer


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to