Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[perl #38294] [PATCH] Fix :optional => :slurpy state transition

0 views
Skip to first unread message

Bob Rogers

unread,
Jan 20, 2006, 11:19:22 PM1/20/06
to bugs-bi...@rt.perl.org
# New Ticket Created by Bob Rogers
# Please include the string: [perl #38294]
# in the subject line of all future correspondence about this issue.
# <URL: https://rt.perl.org/rt3/Ticket/Display.html?id=38294 >


From: Leopold Toetsch <l...@toetsch.at>
Date: Sat, 21 Jan 2006 01:40:38 +0100

As mentioned earlier today, I've rewritten the guts of the argument
passing code. It's now one function 'process_args', which is mostly a
state machine with currently ~12 cases that handles the various states
or should eventually handle . . .

leo

The state transition from :optional to :slurpy results in an "Unhandled
process_args state 0x83" error. The attached fix works, though it may
not be complete. Also, it assumes that CALL_STATE_SLURP and
CALL_STATE_OPT are mutually exclusive; seems to me they ought to be, but
I may not have read your spec carefully enough.

-- Bob Rogers
http://rgrjr.dyndns.org/

opt-plus-slurp.patch

Leopold Toetsch

unread,
Jan 21, 2006, 7:37:27 AM1/21/06
to perl6-i...@perl.org, bugs-bi...@netlabs.develooper.com

On Jan 21, 2006, at 5:19, Bob Rogers (via RT) wrote:

> The state transition from :optional to :slurpy results in an "Unhandled
> process_args state 0x83" error. The attached fix works, though it may
> not be complete.

Thanks, applied - r11278.

> Also, it assumes that CALL_STATE_SLURP and
> CALL_STATE_OPT are mutually exclusive; seems to me they ought to be,
> but
> I may not have read your spec carefully enough.

pdd03 doesn't specify this AFAIK, but providing an :optional :slurpy
would be ambiguous, so the current state seems to be ok.

leo

0 new messages