Skip to content
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

"Octal" digits warn twice. #3211

Closed
p6rt opened this issue Aug 17, 2013 · 8 comments
Closed

"Octal" digits warn twice. #3211

p6rt opened this issue Aug 17, 2013 · 8 comments

Comments

@p6rt
Copy link

p6rt commented Aug 17, 2013

Migrated from rt.perl.org#119339 (status was 'resolved')

Searchable as RT119339$

@p6rt
Copy link
Author

p6rt commented Aug 17, 2013

From glitchmr@myopera.com

If I write something like `say 09`, Rakudo warns about octal numbers
twice. Also, "octal" numbers don't work in REPL.

@p6rt
Copy link
Author

p6rt commented Aug 17, 2013

From @masak

<masak> GlitchMr​: the phrase "don't work" is generally taboo in bug
reports.
<GlitchMr> sorry
<masak> GlitchMr​: you should be talking about what you did, what you
observe, and what you expected.
<GlitchMr> Null PMC access in get_string()
<masak> right. that should be in there.
<masak> GlitchMr​: the goal is to have enough information that the thing
can be checked again later.
* masak adds this to the ticket

@p6rt
Copy link
Author

p6rt commented Aug 17, 2013

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Aug 19, 2013

From glitchmr@myopera.com

By REPL not working, I mean that every time octal number appears, "Null
PMC access in get_string()" appears. Sorry for not mentioning that
initially.

@p6rt
Copy link
Author

p6rt commented Feb 1, 2015

From @skids

Fresher test results​: no duplicate warnings, NULL pointers in REPL
under parrot and jvm but not moar. Probably LTA to be telling the
user to use '0o8' or '0o9'.

NOTE​: my perl6-p is Star.2014.12 and my perl6-j/m are built from star-daily
Cologne-256-gd78c678.

$ perl6-p -e 'say 09'
Potential difficulties​:
  Leading 0 does not indicate octal in Perl 6; please use 0o9 if you mean that
  at -e​:1
  ------> say 09⏏<EOL>
9
$ perl6-m -e 'say 09'
Potential difficulties​:
  Leading 0 does not indicate octal in Perl 6; please use 0o9 if you mean that
  at -e​:1
  ------> say 09⏏<EOL>
9
$ perl6-j -e 'say 09'
Potential difficulties​:
  Leading 0 does not indicate octal in Perl 6; please use 0o9 if you mean that
  at -e​:1
  ------> say 09⏏<EOL>
9
$ perl6-p

say 09
Null PMC access in get_string()
say 09;
Null PMC access in get_string()
say 8
8
exit
$ perl6-m
say 09
Potential difficulties​:
  Leading 0 does not indicate octal in Perl 6; please use 0o9 if you mean that
  at <unknown file>​:1
  ------> say 09⏏<EOL>
9
say 9
9
exit
$ perl6-j
say 09
java.lang.NullPointerException

@p6rt
Copy link
Author

p6rt commented Aug 29, 2015

From @coke

On Sun Feb 01 12​:33​:27 2015, bri@​abrij.org wrote​:

Fresher test results​: no duplicate warnings, NULL pointers in REPL
under parrot and jvm but not moar. Probably LTA to be telling the
user to use '0o8' or '0o9'.

NOTE​: my perl6-p is Star.2014.12 and my perl6-j/m are built from star-
daily
Cologne-256-gd78c678.

$ perl6-p -e 'say 09'
Potential difficulties​:
Leading 0 does not indicate octal in Perl 6; please use 0o9 if you
mean that
at -e​:1
------> say 09⏏<EOL>
9
$ perl6-m -e 'say 09'
Potential difficulties​:
Leading 0 does not indicate octal in Perl 6; please use 0o9 if you
mean that
at -e​:1
------> say 09⏏<EOL>
9
$ perl6-j -e 'say 09'
Potential difficulties​:
Leading 0 does not indicate octal in Perl 6; please use 0o9 if you
mean that
at -e​:1
------> say 09⏏<EOL>
9
$ perl6-p

say 09
Null PMC access in get_string()
say 09;
Null PMC access in get_string()
say 8
8
exit
$ perl6-m
say 09
Potential difficulties​:
Leading 0 does not indicate octal in Perl 6; please use 0o9 if you
mean that
at <unknown file>​:1
------> say 09⏏<EOL>
9
say 9
9
exit
$ perl6-j
say 09
java.lang.NullPointerException

This works as expected in the REPL and out. Can be closed with tests.

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Jul 7, 2016

From @zoffixznet

LTA warning fixed in rakudo/rakudo@6d0cc07

Warning tests added in Raku/roast@e669003e22
REPL tests added in Raku/roast@7dce301ae7

@p6rt p6rt closed this as completed Jul 7, 2016
@p6rt
Copy link
Author

p6rt commented Jul 7, 2016

@zoffixznet - Status changed from 'open' to 'resolved'

@p6rt p6rt added the testneeded label Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant