Don't use >>.grep if .grep is enough #4
Conversation
The use of `>>.grep` in combination with stringification is something that feels strange. In any case, somehow the recent speedup of `>>.` (aka `nodemap`) something subtly changed so that that expression generates a `Seq` with a lot of empty `Seq`s in them, each of which stringifies to a space, and thus breaks the test. Since I don't see a need for the `>>.` there, I'd thought it safe to remove.
|
Thanks @lizmat ! |
|
Hi, @pmqs! In another run of testing I see there is still an upload on CPAN which does not include this fix:
Can you please do a version bump and do a CPAN upload? |
Will do |
|
Hi @Altai-man - new version uploaded to CPAN. Ping if there are any issues |
The use of
>>.grepin combination with stringification is something that feels strange. In any case, somehow the recent speedup of>>.(akanodemap) something subtly changed so that that expression generates aSeqwith a lot of emptySeqs in them, each of which stringifies to a space, and thus breaks the test.Since I don't see a need for the
>>.there, I'd thought it safe to remove.