> It certainly is an API change. At least it makes the API inconsistent
> since now:
>
> $r->print("");
>
> and
>
> print "";
>
> are not the same under modperl.
um, how do you figure? the new tests show that those two calls are exactly
the same - they both return '0E0' (at least when print() is using STDOUT,
which is all we could ever hope to accomplish).
> Moreover, failures are not the same as perl's print. so it's really only
> a partial fix. (even though the manpage doesn't say what happens on
> failure, but we know that it returns 0).
yeah, but that's not likely to cause userland troubles.
the issue here, as I see it is mostly that the return value of
print()/$r->print() under mp1 is different than under mp2 when printing zero
bytes. and it's one that is easily fixed in a way that is unlikely to catch
anyone by surprise.
> I'm not against patches, I just want us to decide first what's to be
> done (if at all) and then move on with implementation, rather than the
> other way around.
well, I'm out of time at the moment. if you want to incorporate the idea
then you have a patch to start with. otherwise, don't worry about it.
--Geoff
--
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html
