New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
default $.nl-in on IO::Handle does not correctly work in subclasses #6434
Comments
From @zoffixznetCurrently, this causes breakage in IO::String zoffix@VirtualBox~$ perl6 -e 'class Z is IO::Handle { method x { dd $.nl-in } }.new.x;' zoffix@VirtualBox~$ perl6 -v Above, it should give the two-element array, with "\r\n" as another element. AlexDaniel bisected my original code[^1] to point to https://gist.github.com/Whateverable/b26f4103f5da0809e11749b50fdbabb4 I tried to golf it further, by taking all the relevant bits from IO::Handle into a custom class, but failed to reproduce the issue then. [1] https://gist.github.com/zoffixznet/0c2cbd7acaaf0d3b27d245ad2e2bc737 |
From @lizmatreverted c63c57e9a823303e74c06 for now
|
The RT System itself - Status changed from 'new' to 'open' |
From @dogbert17On Tue, 08 Aug 2017 04:19:24 -0700, elizabeth wrote:
It seems as if everything is still good after the revert |
From @AlexDanielThere's now a regression test in Raku/roast@a7af874 Closing (but please take a look at that roast commit) On 2018-03-11 13:11:21, jan-olof.hendig@bredband.net wrote:
|
@AlexDaniel - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#131858 (status was 'resolved')
Searchable as RT131858$
The text was updated successfully, but these errors were encountered: