Re: [PATCH] [perl #21875] Hash ref transformed as a list

Dave Mitchell wrote:
> 
> Ah yes, a test case - how silly of me to forget ...
> 
> My patch just fixes the
> 
>     { bareword_starting_with_q =>
> 
> case, it doesnt fix the
> 
>     { q =>
> 
> case, since while the first is clearly a bug, the second may (or may
> not) be a feature, since that could legitimately be a q=string= quoting
> construct. Ie it skips to the matching = as being the end of the term,
> then fails to see a following comma or =>, so assumes it's not a hash.  I
> could update the patch to treat /{\s*q[qx]?\s+=>/ as a special case if
> people want. Also, I notice that it doesn't check for qw and qr.  Should I
> add these?

Uh, if it handles qq and qx, I think it should handle qw and qr as well.
(I haven't looked closely at this.)

> Finally, where should the test go? t/op/parser.t ??

Almost, t/comp/parser.t

-- 
Uncomfortable is not *NIX

• [perl #21875] Hash ref transformed as a list by pierre denis
• Re: [perl #21875] Hash ref transformed as a list by Nicholas Clark
• [PATCH] (vaguely connected with) [perl #21875] Hash ref transformed as a list by Dave Mitchell
• Re: [PATCH] (vaguely connected with) [perl #21875] Hash ref transformed as a list by Rafael Garcia-Suarez
• Re: [PATCH] (vaguely connected with) [perl #21875] Hash ref transformed as a list by H.Merijn Brand
• [PATCH] [perl #21875] Hash ref transformed as a list by Dave Mitchell
• Re: [PATCH] [perl #21875] Hash ref transformed as a list by Rafael Garcia-Suarez
• Re: [PATCH] [perl #21875] Hash ref transformed as a list by Dave Mitchell
• Re: [PATCH] [perl #21875] Hash ref transformed as a list by Rafael Garcia-Suarez
• Re: [PATCH] [perl #21875] Hash ref transformed as a list by Dave Mitchell
• Re: [PATCH] [perl #21875] Hash ref transformed as a list by Rafael Garcia-Suarez