tbrowder | and i’ve answered my own question about (my $a) vs. my ($a): scopes are the same (duh), just didn’t recognize it as it was written! | 00:05 | |||||||||||||||||||||||||||||||||||||
.ask AlexDaniel can you define a branch of rakudo where the pod slang work can be done so my work can be merged by PR more readily? or should i just do it on my own acc’t in isolation? or? | 00:15 | ||||||||||||||||||||||||||||||||||||||
Geth | nqp/post-release: df45cb8ce9 | (Zoffix Znet)++ | 2 files Fix crashes with native types in conditionals Fixes RT#132718: rt.perl.org/Ticket/Display.html?id=132718 When we put together the MAST for if/unless/while/until/repeat_while /repeat_until ops, we look up which MVM conditional op to use. The compiler did not know about non-fullwidth natives, and complained about ... (13 more lines) |
03:29 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#132718 [open]: rt.perl.org/Ticket/Display.html?id=132718 BUG: Unhandled kind 3 with int32 argument | ||||||||||||||||||||||||||||||||||||||
Geth | roast/post-release: 63fb5d3a95 | (Zoffix Znet)++ | MISC/bug-coverage.t Cover crashes with native types in conditionals RT#132718: rt.perl.org/Ticket/Display.html?id=132718 Rakudo fix: github.com/perl6/nqp/commit/df45cb8ce9 |
||||||||||||||||||||||||||||||||||||||
JimmyZ comments on github.com/perl6/nqp/commit/df45cb8ce9 | 04:58 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo/post-release: ec23031d33 | (Zoffix Znet)++ | t/05-messages/02-errors.t Test detached non-alpha meths give useful error RT#126856: rt.perl.org/Ticket/Display.html?id=126856 |
05:57 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#126856 [new]: rt.perl.org/Ticket/Display.html?id=126856 [LTA] No spaces allowed around .^ and .? (^42 .^methods.say) | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo/post-release: d769f0a62e | (Zoffix Znet)++ | t/05-messages/02-errors.t Minor formatting/line-length adjustments |
06:02 | |||||||||||||||||||||||||||||||||||||
rakudo/post-release: 3745eff13a | (Zoffix Znet)++ | src/Perl6/Actions.nqp Fix postconstraints in `my (...)` being ignored Fixes RT#128935: rt.perl.org/Ticket/Display.html?id=128935 We were just forgetting to pass the post-constraint information to the variable making routines. |
08:55 | ||||||||||||||||||||||||||||||||||||||
synopsebot | RT#128935 [open]: rt.perl.org/Ticket/Display.html?id=128935 [BUG] "where" clauses are parsed, but not enforced, in "my" expressions | ||||||||||||||||||||||||||||||||||||||
Geth | roast/post-release: 4ce0fc56db | (Zoffix Znet)++ | S12-subset/subtypes.t Test postconstraints in `my (...)` RT#128935: rt.perl.org/Ticket/Display.html?id=128935 Rakudo fix: github.com/rakudo/rakudo/commit/3745eff13a Tests for %, &, and @ variables are deliberatelly omited until R#1414 is resolved and will be tested with that Issue. |
09:02 | |||||||||||||||||||||||||||||||||||||
synopsebot | R#1414 [open]: github.com/rakudo/rakudo/issues/1414 Class attribute constraints applied on elements of array attribute, not the attribute itself | ||||||||||||||||||||||||||||||||||||||
rakudo/post-release: dac1951eca | (Zoffix Znet)++ | t/05-messages/02-errors.t We got enough tests in this file; start a new one |
09:22 | ||||||||||||||||||||||||||||||||||||||
timotimo | m: say 4 ** -100 | 09:35 | |||||||||||||||||||||||||||||||||||||
camelia | 6.22301527786114e-61 | ||||||||||||||||||||||||||||||||||||||
timotimo | ^- this is how likely we get a false prime with the parameter we've selected for is-prime by default | 09:36 | |||||||||||||||||||||||||||||||||||||
we might want to drop that number a little bit | |||||||||||||||||||||||||||||||||||||||
m: say 4 ** -50 | |||||||||||||||||||||||||||||||||||||||
camelia | 7.88860905221012e-31 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo/post-release: f3efe5e6b4 | (Zoffix Znet)++ | 2 files Fix optimizer crashes in map with certain args Fixes RT#129790: rt.perl.org/Ticket/Display.html?id=129790 Don't assume the arg is a codeblock; check first. |
09:44 | |||||||||||||||||||||||||||||||||||||
synopsebot | RT#129790 [open]: rt.perl.org/Ticket/Display.html?id=129790 [OPTIMIZER] Giving a sub to a map fails | ||||||||||||||||||||||||||||||||||||||
Geth | nqp: tbrowder++ created pull request #392: change .markdown file suffix to .md |
11:41 | |||||||||||||||||||||||||||||||||||||
rakudo/post-release: 359efef709 | (Zoffix Znet)++ | src/core/IO/Handle.pm Don't block IO::Handle.get on TTYs after we reached EOF Fixes RT#130086: rt.perl.org/Ticket/Display.html?id=130086 We don't block on non-TTYs nor do we block with other method such as .read. While TTYs require an extra read to figure out when they reach EOF, once we know we're at EOF, there's no point trying to read again. |
11:45 | ||||||||||||||||||||||||||||||||||||||
synopsebot | RT#130086 [new]: rt.perl.org/Ticket/Display.html?id=130086 [BUG] multiple eof-s support on $*IN | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | quotable6: ‘.markdown’ | 12:16 | |||||||||||||||||||||||||||||||||||||
quotable6 | AlexDaniel, OK, working on it! This may take up to three minutes (4482999 messages to process) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel, gist.github.com/eb39d5949aa19e3e88...4915bd8c4e | 12:17 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: regarding the branch, feel free to do it the way you like. If your CLA arrived already, then you should have a commit bit, and then you can work in a branch of an upstream repo | ||||||||||||||||||||||||||||||||||||||
tbrowder: I see that the supernova project was supposed to work independently for the initial stages, but the long term plan was to move it to guts | 12:18 | ||||||||||||||||||||||||||||||||||||||
tbrowder | roger | 12:19 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: tbh I know very little about POD and even less so about supernova, so use your own judgement | ||||||||||||||||||||||||||||||||||||||
tbrowder | thnx, i’ll try hard to justify yr faith... | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: also, I think you shouldn't be expecting this to be merged within v6.c because I'm sure that it will result in a lot of changed behavior | 12:20 | |||||||||||||||||||||||||||||||||||||
tbrowder: so the merge will probably happen for the next language version once it's ready. Like v6.e or v6.d it happens that it takes us so much time for v6.d… | 12:21 | ||||||||||||||||||||||||||||||||||||||
if it* | |||||||||||||||||||||||||||||||||||||||
tbrowder: but yeah, I'm very happy that someone is looking in this direction | |||||||||||||||||||||||||||||||||||||||
tbrowder | yep, i expected that. i’m certainly not going to be able to do it anytime soon! | 12:22 | |||||||||||||||||||||||||||||||||||||
but actually i think it may be mostly backwardly compatible. | 12:26 | ||||||||||||||||||||||||||||||||||||||
we have tightened the ascii table requirements almost to hers (more in some cases), and the formatting codes inside tables have just been ignored. | 12:28 | ||||||||||||||||||||||||||||||||||||||
her new fancy unicode tables will be shocking but definitelya show-off | 12:29 | ||||||||||||||||||||||||||||||||||||||
stmuk | I started this github.com/stmuk/rakudo-packages a few months back which maybe should go in docs/website somewhere | 12:39 | |||||||||||||||||||||||||||||||||||||
any idea where though? | |||||||||||||||||||||||||||||||||||||||
psch | stmuk: perl6.org/downloads and rakudo.org/how-to-get-rakudo seem most fitting | 13:02 | |||||||||||||||||||||||||||||||||||||
although the latter already is quite extensive with instructions... | |||||||||||||||||||||||||||||||||||||||
stmuk | both probably have too much information there anyway | ||||||||||||||||||||||||||||||||||||||
psch | maybe a link on perl6.org/downloads under the "Installing from binaries" header? | 13:04 | |||||||||||||||||||||||||||||||||||||
i.e. "Alternative there are [these](...) community-maintained packages for some GNU/Linux distributions" | |||||||||||||||||||||||||||||||||||||||
stmuk | yes that's a good idea | ||||||||||||||||||||||||||||||||||||||
I'll change when I do the R* 2018.01 changes | 13:06 | ||||||||||||||||||||||||||||||||||||||
[Tux] |
|
13:17 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | releasable6: next | 13:18 | |||||||||||||||||||||||||||||||||||||
releasable6 | AlexDaniel, Next release in ≈5 hours. Blockers: github.com/rakudo/rakudo/issues?q=...%9A%A0%22. Unknown changelog format | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | heh releasable6, you're very optimistic! :) | 13:19 | |||||||||||||||||||||||||||||||||||||
psch | hrm, hprof unfortunately seems somewhat useless to find the leak for the EvalServer :l | 13:28 | |||||||||||||||||||||||||||||||||||||
.tell bartolin spectest looks a bit worse than yours at first glance: gist.github.com/anonymous/30ba5ed2...ac2dfe92e2 | 13:33 | ||||||||||||||||||||||||||||||||||||||
yoleaux | psch: I'll pass your message to bartolin. | ||||||||||||||||||||||||||||||||||||||
psch | well, minus the passing TODOs maybe | ||||||||||||||||||||||||||||||||||||||
Geth | nqp: tbrowder++ created pull request #394: add new pod tests |
13:46 | |||||||||||||||||||||||||||||||||||||
nqp: f64e981aa9 | (Tom Browder)++ | t/nqp/005-comments.t add new pod tests + test for robust =end for blocks even though misaligned with its =begin + test to show current state of failure with nested comment blocks that shows parsing of code ends at the first =end comment OR an error is thrown in other circumstances |
13:49 | ||||||||||||||||||||||||||||||||||||||
nqp: 2f362439ad | (Elizabeth Mattijsen)++ (committed using GitHub Web editor) | t/nqp/005-comments.t Merge pull request #394 from tbrowder/pod-tests add new pod tests |
|||||||||||||||||||||||||||||||||||||||
tbrowder | lizmat: thnx! | ||||||||||||||||||||||||||||||||||||||
psch | 9 million lines profiling output /o\ | 14:10 | |||||||||||||||||||||||||||||||||||||
i suppose i shouldn't have put two of those into vimdiff without checking their size | |||||||||||||||||||||||||||||||||||||||
tbrowder | AlexDaniel: i sent the CLA several weeks ago. i tried, through the web interface, to create a rakudo branch, but no recognition of my git user name. i can create a branch in nqp because moritz gave me the perl6 commit bit early on when i was pushing doc PRs. | 14:12 | |||||||||||||||||||||||||||||||||||||
psch | IIRC my CLA approval took around 6 weeks? | 14:13 | |||||||||||||||||||||||||||||||||||||
that's ~4 years ago though | |||||||||||||||||||||||||||||||||||||||
tbrowder | btw, what’s the feeling about adding one’s git user name to a branch name? | 14:14 | |||||||||||||||||||||||||||||||||||||
psch | but github/perl6 isn't bound by the CLA like github/rakudo is i also seem to remember | ||||||||||||||||||||||||||||||||||||||
moritz | that's correct | ||||||||||||||||||||||||||||||||||||||
tbrowder | ok, how will the rakudo owners know about my CLA? | 14:15 | |||||||||||||||||||||||||||||||||||||
is there a list? should i query them directly? | |||||||||||||||||||||||||||||||||||||||
psch | i *think* PerlJam did the correspondence in my case? i.e. bugged TPF people about my CLA and then mentioned it here when it had arrived | 14:16 | |||||||||||||||||||||||||||||||||||||
tbrowder | ok, thnx, i’ll bug TPF then... | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | maybe it's a good idea to ping [Coke] first? | 14:18 | |||||||||||||||||||||||||||||||||||||
tbrowder: using your username as a branch name is just fine | 14:19 | ||||||||||||||||||||||||||||||||||||||
although I'd prefer “new-pod” or something like that :) | |||||||||||||||||||||||||||||||||||||||
tbrowder | i was thinking something like pod-slang-tbrowder | 14:20 | |||||||||||||||||||||||||||||||||||||
psch | +1 on feature-named branches | ||||||||||||||||||||||||||||||||||||||
no real idea whether a username adds anything, but it probably also doesn't take anything away | 14:21 | ||||||||||||||||||||||||||||||||||||||
tbrowder | .ask [Coke] can please check on my CLA application? (Thomas M. Browder, Jr.; aka Tom Browder) | 14:23 | |||||||||||||||||||||||||||||||||||||
yoleaux | tbrowder: I'll pass your message to [Coke]. | ||||||||||||||||||||||||||||||||||||||
tbrowder | ref branch names: i just see other names as part of branches and it seems to provide a quick ref to who “owns” it (i.e., the guilty party) | 14:26 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | but it's also possible to look at the last commits so that's hardly a problem | 14:29 | |||||||||||||||||||||||||||||||||||||
psch | i mean, if tbrowder starts the trend we'll know their branches at a glance at least :) | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | :D | 14:30 | |||||||||||||||||||||||||||||||||||||
DrForr | . o ( CLA application? Something I've missed due to fallout? ) | 14:37 | |||||||||||||||||||||||||||||||||||||
psch | DrForr: contributors license agreement. pushing directly to the rakudo repo needs a waiver for the artistic license | ||||||||||||||||||||||||||||||||||||||
DrForr: and TPF wants to have that in writing, 'cause legal stuff probably | |||||||||||||||||||||||||||||||||||||||
DrForr | Ah. | 14:38 | |||||||||||||||||||||||||||||||||||||
psch | hm, pretty much everything seems to just straight double on the heap with two vs one EvalServer calls using &run | 14:45 | |||||||||||||||||||||||||||||||||||||
i'm not sure that's surprising or to be expected :S | |||||||||||||||||||||||||||||||||||||||
travis-ci | Rakudo build passed. Zoffix Znet 'Merge pull request #1423 from W4anD0eR96/patch-1 | 14:57 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/rakudo/rakudo/builds/331046454 github.com/rakudo/rakudo/compare/9...b5279aa5b1 | |||||||||||||||||||||||||||||||||||||||
timotimo | huh, nqp has pod? | 15:06 | |||||||||||||||||||||||||||||||||||||
Geth | nqp/master: 6 commits pushed by pmurias++ | 15:07 | |||||||||||||||||||||||||||||||||||||
travis-ci | NQP build failed. pmurias 'Add a few tests that grammars match graphemes' | 15:26 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/331210300 github.com/perl6/nqp/compare/2f362...aedda0b194 | |||||||||||||||||||||||||||||||||||||||
Geth | nqp: 594dc7abf1 | pmurias++ | t/nqp/031-grammar.t Fix number of skipped tests |
15:29 | |||||||||||||||||||||||||||||||||||||
psch | neat, InvocationTargetException when trying to heap dump :l | 15:41 | |||||||||||||||||||||||||||||||||||||
in the heap dump tool too | |||||||||||||||||||||||||||||||||||||||
travis-ci | NQP build passed. pmurias 'Fix number of skipped tests' | 15:46 | |||||||||||||||||||||||||||||||||||||
travis-ci.org/perl6/nqp/builds/331216163 github.com/perl6/nqp/compare/98aed...4dc7abf16e | |||||||||||||||||||||||||||||||||||||||
Geth | rakudo: efed4105da | (Elizabeth Mattijsen)++ | src/core/Buf.pm Fix stray \ Apparently we don't test assigning to a buf with a native integer index :-( Spotted while playing with sieve of erastothenes. |
16:04 | |||||||||||||||||||||||||||||||||||||
psch | heh, we keep 360k P6intInstances around *after* the GlobalContext that we ran the provided code in finished | 16:12 | |||||||||||||||||||||||||||||||||||||
though the 150k SixModelObjects probably have more memory impact... | |||||||||||||||||||||||||||||||||||||||
AlexDaniel | weird. Every time I run toaster it just feels slower | 17:07 | |||||||||||||||||||||||||||||||||||||
timotimo | less ram free than last time? | ||||||||||||||||||||||||||||||||||||||
more diverse files in the CUR that makes finding stuff slower? | 17:08 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | more than 60G RAM free, seems to be bound by the CPU | 17:09 | |||||||||||||||||||||||||||||||||||||
or something | |||||||||||||||||||||||||||||||||||||||
I guess I'll just let it crunch with turtle speeds | 17:10 | ||||||||||||||||||||||||||||||||||||||
Geth | nqp: 4d3db7f008 | (Aleks-Daniel Jakimenko-Aleksejev)++ | tools/build/MOAR_REVISION [MoarVM Bump] Brings 25 commits MoarVM bump brought: github.com/MoarVM/MoarVM/compare/2...g85b2c5f82 85b2c5f82 Pay attention to return op in cross-HLL inlines 473344916 Mark sp_rebless as :noinline a74869e37 Merge pull request #781 from samcv/ucd-critic 9d540bfdf ucd2c.pl: Cleanup part 18, eliminate 7 global variables ... (21 more lines) |
17:27 | |||||||||||||||||||||||||||||||||||||
¦ nqp: version bump brought these changes: github.com/MoarVM/MoarVM/compare/2...g85b2c5f82 | |||||||||||||||||||||||||||||||||||||||
rakudo: d987bf8243 | (Aleks-Daniel Jakimenko-Aleksejev)++ | tools/build/NQP_REVISION [NQP Bump] Brings 16 commits NQP bump brought: github.com/perl6/nqp/compare/2017....g4d3db7f00 4d3db7f00 [MoarVM Bump] Brings 25 commits 594dc7abf Fix number of skipped tests 98aedda0b Add a few tests that grammars match graphemes 3f1d425b3 [js] Make cclass in regexes match full graphemes ... (12 more lines) |
|||||||||||||||||||||||||||||||||||||||
AlexDaniel | Zoffix++ | ||||||||||||||||||||||||||||||||||||||
¦ rakudo: version bump brought these changes: github.com/perl6/nqp/compare/2017....g4d3db7f00 | |||||||||||||||||||||||||||||||||||||||
roast: 906c688aba | (Aleks-Daniel Jakimenko-Aleksejev)++ | 2 files Revert "Follow semantic change of github.com/rakudo/rakudo/commit/6cca27669f" This reverts commit 5670acf676d65167e892515a42b71c81984f39f2. See rakudo/rakudo#1425 |
17:45 | ||||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1425 [open]: github.com/rakudo/rakudo/issues/1425 [RFC] Semantic change with List.WHICH | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: b41a9176a6 | (Aleks-Daniel Jakimenko-Aleksejev)++ | 3 files Revert "Make List a value type if it only has value types" This reverts commit 6cca27669f15ddc3629bfe9071649a1da322a9d6. See issue #1425 |
||||||||||||||||||||||||||||||||||||||
AlexDaniel goes for a nap | 18:26 | ||||||||||||||||||||||||||||||||||||||
possibly a very long nap, see you all tomorrow o/ | |||||||||||||||||||||||||||||||||||||||
lizmat | AlexDaniel++ # see you! | 18:28 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | hmmm I think something is wrong with WHICH | 19:43 | |||||||||||||||||||||||||||||||||||||
releasable complains | |||||||||||||||||||||||||||||||||||||||
AlexDaniel runs again | 19:52 | ||||||||||||||||||||||||||||||||||||||
fwiw I'm seeing some weird fresh failures in toaster, tomorrow will be trying to figure it out | 22:16 | ||||||||||||||||||||||||||||||||||||||
maybe a non-issue because things look… weird | 22:17 | ||||||||||||||||||||||||||||||||||||||
6c: say X::Proc::Async::TapBeforeSpawn.WHICH.WHAT.perl | 22:21 | ||||||||||||||||||||||||||||||||||||||
6c: say X::Proc::Async::TapBeforeSpawn.WHICH.WHAT.perl | 22:22 | ||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦2015.12,2016.01.1,2016.02,2016.03,2016.04,2016.05,2016.06,2016.07.1,2016.08.1,2016.09,2016.10,2016.11,2016.12,2017.01,2017.02,2017.03,2017.04.3,2017.05,2017.06,2017.07,2017.08,2017.09,2017.10,2017.11,2017.12: «ObjAt» ¦HEAD(b41a917): «ValueObjAt» | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | bisect: say X::Proc::Async::TapBeforeSpawn.WHICH.WHAT.perl | ||||||||||||||||||||||||||||||||||||||
bisectable6 | AlexDaniel, Bisecting by output (old=2015.12 new=b41a917) because on both starting points the exit code is 0 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel, bisect log: gist.github.com/71dc6e479d40c3b8f1...fe982ef829 | 22:23 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel, (2017-12-21) github.com/rakudo/rakudo/commit/20...15d52ac3d4 | |||||||||||||||||||||||||||||||||||||||
AlexDaniel looks at github.com/perl6/roast/commit/904a...e91e1e7b8c | 22:24 | ||||||||||||||||||||||||||||||||||||||
m: 2017.12,HEAD X::Proc::Async::TapBeforeSpawn.WHICH | |||||||||||||||||||||||||||||||||||||||
camelia | 5===SORRY!5=== Error while compiling <tmp> Undeclared name: HEAD used at line 1 |
||||||||||||||||||||||||||||||||||||||
AlexDaniel | c: 2017.12,HEAD X::Proc::Async::TapBeforeSpawn.WHICH | ||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦2017.12,HEAD(b41a917): «» | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | c: 2017.12,HEAD say X::Proc::Async::TapBeforeSpawn.WHICH | ||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦2017.12: «X::Proc::Async::TapBeforeSpawn|U94293319722984» ¦HEAD(b41a917): «X::Proc::Async::TapBeforeSpawn|U94669681789072» | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | 6c: is NaN.Rat, NaN, "NaN.Rat == NaN"; | 22:43 | |||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, ¦6c (26 commits): «04===SORRY!04=== Error while compiling /tmp/cvEGsZQ0RMUndeclared routine: is used at line 1 «exit code = 1»» | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | 6c: use Test; is NaN.Rat, NaN, "NaN.Rat == NaN"; | ||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, gist.github.com/f61e644c75dff7db2b...6ba0603ca3 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | hm | 22:44 | |||||||||||||||||||||||||||||||||||||
6c: say NaN.Rat | |||||||||||||||||||||||||||||||||||||||
committable6 | AlexDaniel, gist.github.com/17c428ef80f077e146...4661a40b9a | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | .tell Zoffix “is NaN.Rat, NaN, "NaN.Rat == NaN";” needs a similar change in 6.c-errata branch, I'm assuming there's a justification for that? | 22:49 | |||||||||||||||||||||||||||||||||||||
yoleaux | AlexDaniel: I'll pass your message to Zoffix. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | .tell Zoffix github.com/perl6/roast/commit/a257...757ca9f4a2 | 22:53 | |||||||||||||||||||||||||||||||||||||
yoleaux | AlexDaniel: I'll pass your message to Zoffix. | ||||||||||||||||||||||||||||||||||||||
Zoffix | AlexDaniel: it's weird these are popping for you, 'cause I thought I fixed all 6.c-errata breakage other than the list type object.. | 23:23 | |||||||||||||||||||||||||||||||||||||
yoleaux | 22:49Z <AlexDaniel> Zoffix: “is NaN.Rat, NaN, "NaN.Rat == NaN";” needs a similar change in 6.c-errata branch, I'm assuming there's a justification for that? | ||||||||||||||||||||||||||||||||||||||
22:53Z <AlexDaniel> Zoffix: github.com/perl6/roast/commit/a257...757ca9f4a2 | |||||||||||||||||||||||||||||||||||||||
Zoffix | Perhaps I just lost these failures under the sea of TODO passage | ||||||||||||||||||||||||||||||||||||||
AlexDaniel: but yeah, those tests are incorrect. They're using string equality (due to use of `is` routine) to test numeric equality of two objects. | 23:25 | ||||||||||||||||||||||||||||||||||||||
m: dd NaN.Rat.isNaN | |||||||||||||||||||||||||||||||||||||||
camelia | Bool::True | ||||||||||||||||||||||||||||||||||||||
Zoffix | That NaN fudge was removed in later commit | ||||||||||||||||||||||||||||||||||||||
6c: say NaN.Rat.Num | 23:31 | ||||||||||||||||||||||||||||||||||||||
committable6 | Zoffix, ¦6c (26 commits): «NaN» | ||||||||||||||||||||||||||||||||||||||
Zoffix | Str view of Rat involves division and only in Num view is division by zero is not an explosion | ||||||||||||||||||||||||||||||||||||||
As for the change itself. In 2017.12, we had NaN.Rat return a special non-Rat object that knew how to convert back to NaN and did not explode with stringification. Now, we simply return an effective inverse operation: <0/0>.Num => NaN; NaN.Rat => <0/0>. This fixed 3 bugs with the old system and my log digging showed me that the original special-object thing was a misinterpretation. Full description is in | 23:57 | ||||||||||||||||||||||||||||||||||||||
the commit: github.com/rakudo/rakudo/commit/04...ec1cd3dfdc | |||||||||||||||||||||||||||||||||||||||
.tell AlexDaniel irclog.perlgeek.de/perl6-dev/2018-...i_15716502 | |||||||||||||||||||||||||||||||||||||||
yoleaux | Zoffix: I'll pass your message to AlexDaniel. |