MasterDuke | github.com/rakudo/rakudo/pull/814, if anybody has an hour or so for the in-depth analysis required to validate this PR i promise it will be a thrilling ride | 00:08 | |
Zoffix | m: say min +'a' | 00:14 | |
camelia | rakudo-moar 29a110: OUTPUT«Inf» | ||
Zoffix | m: say minmax +'a' | ||
camelia | rakudo-moar 29a110: OUTPUT«Cannot convert string to number: base-10 number must begin with valid digits or '.' in '⏏a' (indicated by ⏏) in block <unit> at <tmp> line 1Actually thrown at: in any at gen/moar/m-Metamodel.nqp line 3055 in block <unit> at <tmp…» | ||
Zoffix | hm | ||
Zoffix will fix | 00:15 | ||
m: say List.^can('min')[0].package | 00:18 | ||
camelia | rakudo-moar 29a110: OUTPUT«(Any)» | ||
Zoffix | I don't see it defined in Any.pm tho? | ||
Hm, Any-iterable-methods OK | 00:19 | ||
m: sub foo { fail }; foo // say 'meow' | 00:21 | ||
camelia | rakudo-moar 29a110: OUTPUT«===SORRY!=== Error while compiling <tmp>Null regex not allowedat <tmp>:1------> sub foo { fail }; foo //⏏ say 'meow'» | ||
Zoffix | PR fixing the min/max stuff: github.com/rakudo/rakudo/pull/815 | 01:01 | |
sortiz | Zoffix++ # Failure propagation | 01:06 | |
Zoffix | It's a shitty solution, now that I think of it. | ||
Well, not consistent, rather. | 01:07 | ||
zoffix@VirtualBox:~/CPANPRC/rakudo$ ./perl6-m -e 'my $x = min +"a"; say 42' | |||
42 | |||
zoffix@VirtualBox:~/CPANPRC/rakudo$ ./perl6-m -e 'my $x = min +"a", +"a"; say 42' | |||
Cannot convert string to number: base-10 number must begin with valid digits or ' | |||
m: say min +'a', +'a' | 01:09 | ||
camelia | rakudo-moar 29a110: OUTPUT«Cannot convert string to number: base-10 number must begin with valid digits or '.' in '⏏a' (indicated by ⏏) in block <unit> at <tmp> line 1Actually thrown at: in block <unit> at <tmp> line 1» | ||
Zoffix | m: say min +'a' | ||
camelia | rakudo-moar 29a110: OUTPUT«Inf» | ||
Zoffix | So, the PR is a mild improvement over the current situation :) | ||
MasterDuke | Zoffix += .5? | 01:11 | |
sortiz | :) | ||
Well, Failure handling is tricky. | 01:13 | ||
MasterDuke | i started thinking "what's half of '++', but more interesting than '+'?" and nearly said Zoffix-- (a '-' being half of a '+''), but thought that could be mis-interpreted | ||
sortiz | Zoffix, thinking about it, in those cases if a Failure is detected is better to throw it, imo | 01:42 | |
geekosaur | ⁺⁺ :p | 01:46 | |
sortiz | i.e, nqp::istype($min, Failure) ?? $min.throw !! $min // Inf | ||
Zoffix | sortiz, updated, thanks, but I see it spilling guts in the throw message "in any at gen/moar/m-Metamodel.nqp line 3055": gist.github.com/zoffixznet/cb2ec41...f3552e20ec | 01:59 | |
The second one is the current rakudo message, for reference | |||
sortiz | Zoffix, btw, I think that the test for Failure can be moved to !first-concrete, and may be better handled there. | 02:04 | |
Zoffix | sortiz, yeah, makes more sense to put it there. Any way to make the thrown exception look the same as for the case with more than one arg? | 02:14 | |
sortiz | Zoffix, I'm looking into... | 02:15 | |
Zoffix | Actually, not sure how to do it in !first-concrete... self.throw if nqp::istype(self, Failure); doesn't work | 02:25 | |
sortiz | You don't need to test in self, test in $value. | 02:27 | |
self is the Iterable. | 02:28 | ||
Zoffix, something like gist.github.com/salortiz/56ed771aa...5baadeef92 | 02:44 | ||
Zoffix | sortiz, yeah, that's my current version I'm spectesting. | 02:45 | |
It still got gen/moar/m-Metamodel.nqp in the error message, but at least it's consistent now between 1- and many- arg versions :) | |||
sortiz | ¯\_(ツ)_/¯ , as Zoffix says | 02:46 | |
Zoffix | :) I got a key bound to type that :) | 02:47 | |
sortiz | Yep, I suppose that | ||
:) | |||
Zoffix, seems that nqp::istype($value, Failure) ?? $value.exception.throw !! $value avoids the Metamodel issue. | 02:56 | ||
sortiz likes Pair programming | 03:03 | ||
Zoffix | \o/ | 03:09 | |
sortiz++ great! | |||
spectested and PR updated. | 03:12 | ||
Zoffix goes to bed o | |||
sortiz | Zoffix++ | ||
\o | 03:13 | ||
dalek | ast: 683aeea | (Zoffix Znet)++ | S03-operators/minmax.t: min/max with Failures throw RT#125334 RT#128573 (currently fudged; awaiting merge of github.com/rakudo/rakudo/pull/815) |
04:05 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125334 | ||
Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128573 | |||
nine | MasterDuke: I kinda hate that we slow down every use of .chop for a completely unrealistic edge case :/ | 07:47 | |
psch | what's the completely unrealistic edge case? $chars <= 0? | 07:49 | |
nine | $chars being 9999999999999999999999999999 | ||
psch | oh, right, because substr takes an int | 07:50 | |
well, nqp::substr | 07:51 | ||
nine | I wonder why we have a coercer in the signature there. We usually don't? | 07:52 | |
Because I really think it should just take an int | 07:53 | ||
psch | m: say "foo".chop("1") | ||
camelia | rakudo-moar 29a110: OUTPUT«fo» | ||
nine | The string's length cannot exceed int characters anyway | ||
psch | yeah, the coercer is a bit weird | ||
nine | The commit doesn't really explain it either github.com/rakudo/rakudo/commit/0c0eb135 | 08:00 | |
psch | yeah, the coercer still comes out of nowhere | 08:02 | |
the message seems to talk about the parameter being propagated to the sub form and Cool candidate, if anything | |||
grr | 08:26 | ||
gist.github.com/peschwa/f0b3c39e9c...0e237a9b2d | |||
what a supremely useless backtrace :| | |||
nine | We apparently even have a spec test for the coercion, but again the commit message does not tell why :/ | 08:34 | |
psch | hrm, i suppose i could try to reach for lexical_handler_not_found_error in invokeDirect where we normally throw the UnwindException | 08:39 | |
but i'm pretty sure it's not only things that want to throw X::ControlFlow::Return that could reach there | |||
nine | psch: at least your useless backtrace tought me about "catch uncought" :) | 08:47 | |
psch | yeah, it's pretty neat | 08:49 | |
doesn't always seem to work though | |||
as in, i remember that RTE still always get caught | |||
but i might be mistaken, 'cause UnwindException extends ControlException extends RuntimeException | |||
so, yeah, it's probably generally useful :P | |||
dalek | kudo/nom: 94b52c1 | (Daniel Green)++ | src/core/Str.pm: Fix RT #125814 |
09:03 | |
kudo/nom: 98d472c | lizmat++ | src/core/Str.pm: Merge pull request #814 from MasterDuke17/RT125814 Fix RT #125814 |
|||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125814 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125814 | ||
ast: a222a04 | (Daniel Green)++ | S32-str/chop.t: Tests for RT #125814 |
|||
ast: d9c3d2c | lizmat++ | S32-str/chop.t: Merge pull request #133 from MasterDuke17/RT125814 Tests for RT #125814 |
|||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125814 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125814 | ||
lizmat | nine psch re chop coercing to Int() | 09:05 | |
I thought it would be wise to the behaviour be consistent with this: | 09:06 | ||
m: my @a = ^10; say @a["4"] | |||
camelia | rakudo-moar 29a110: OUTPUT«4» | ||
nine | lizmat: that looks kinda surprising, too :) | ||
I thought we don't coerce in general? | 09:07 | ||
lizmat | not sure how general that is :-) | 09:08 | |
psch | i'm mostly wondering why it has to be a coercer instead of Cool and just use it Int-y | 09:15 | |
so, yeah, it's probably fine | 09:16 | ||
nine | Even with Cool, we'd have to support Int. But I wonder if a multi could safe performance in the common case here? | 09:17 | |
psch | uhh, what. i got a ThreadContext without a curFrame..? | 09:21 | |
well, i guess i have an idea why that is | 09:49 | ||
'cause it happens in invokeDirect | |||
which means we are between frames, i think..? | |||
hrm, this is somewhat terrible | 10:28 | ||
because we don't really have way to figure out if we've missed all possible handlers | |||
(still talking about the 'sub f { ^3 .map({ return }) }; f' thing jnthn mentioned yesterday) | 10:29 | ||
dalek | kudo/nom: 5b0a9e4 | lizmat++ | src/core/Rakudo/Internals.pm: Make sure we know about the new leap-second datacenter.iers.org/web/guest/eop/.../latest/16 |
||
psch | the weird bit is that it apparently doesn't go through handlerDynamic either | 10:30 | |
'cause i already tried that and it didn't change anything | |||
and i don't think we can reliably install a handler during code gen either | 10:32 | ||
jnthn | psch: All MoarVM does there is check that the frame that it got is also in the caller chain relative to tc->cur_frame | 10:40 | |
psch: Though it has some short-cuts to avoid the scan | 10:41 | ||
(In some cases) | |||
(The common ones... :)) | |||
But a scan along the dynamic chain should be correct | |||
github.com/MoarVM/MoarVM/blob/mast...ons.c#L154 | 10:42 | ||
Just realized that we have an even simpler way to figure it out so never need to scan :) | |||
psch | jnthn: so that's handlerDynamic and handlerLexical rolled in one? | 10:44 | |
Zoffix | m: my $a = +'a'; say 42 | 10:48 | |
camelia | rakudo-moar 98d472: OUTPUT«42» | ||
Zoffix | m: my $a = +'a'; my $y = $a cmp 42; say 42 | ||
camelia | rakudo-moar 98d472: OUTPUT«Cannot convert string to number: base-10 number must begin with valid digits or '.' in '⏏a' (indicated by ⏏) in block <unit> at <tmp> line 1Actually thrown at: in block <unit> at <tmp> line 1» | ||
Zoffix | So my very first version of #815 should've been the more correct one? min/max with single arg that is Failure should return it instead of throwing? RE: github.com/rakudo/rakudo/pull/815#...-231308852 | 10:49 | |
s/should've/were/; | |||
jnthn | psch: Yeah, the way it's factored in Moar we switch move `mode` | 10:53 | |
*switch over | |||
dalek | kudo/nom: ea8c42a | lizmat++ | src/core/Array.pm: Make Array.unshift|prepend a few % faster - lose the ensure-allocated - also make sure it can handle "bare" Array objects without $!reified |
||
lizmat | Zoffix: yeah, that feels righter | 11:09 | |
gfldex | m: enum Foo { 'Bar' => 1}; say 'Bar'; | 11:15 | |
camelia | rakudo-moar ea8c42: OUTPUT«===SORRY!=== Error while compiling <tmp>Undeclared name: Foo used at line 1Undeclared routine: enum used at line 1» | ||
gfldex | m: enum Foo ( 'Bar' => 1 ); say 'Bar'; | ||
camelia | rakudo-moar ea8c42: OUTPUT«Bar» | ||
gfldex | m: enum Foo ( 'Bar' => 1 ); say "{'Bar'}"; | 11:16 | |
camelia | rakudo-moar ea8c42: OUTPUT«Bar» | ||
tbrowder | mornin' #perl6-dev | 11:23 | |
[Tux] | This is Rakudo version 2016.06-169-gea8c42a built on MoarVM version 2016.06-9-g8fc21d5 | 11:24 | |
test 15.808 | |||
test-t 9.202 | |||
csv-parser 16.541 | |||
tbrowder | ref table pod: I'm trying to incrementally add some severe changes as gradually as I can to eliminate problems, so I've added my proposed DebugPod.nqp as a single PR. The current spec table test has been run against it successfully. Please consider PR github.com/rakudo/rakudo/pull/816 | 11:26 | |
dalek | kudo/nom: c98a6e6 | (Tom Browder)++ | src/Perl6/DebugPod.nqp: add new file with pod debugging subroutines |
11:39 | |
kudo/nom: 1fb74d5 | (Tom Browder)++ | tools/build/Makefile- (2 files): mod build tools to compile the new debugging file |
|||
kudo/nom: 30cb7d3 | lizmat++ | / (3 files): Merge pull request #816 from tbrowder/add-pod-debug-file Add pod debug file |
|||
lizmat | tbrowder++ | 11:47 | |
dalek | kudo/nom: 94a31c6 | lizmat++ | src/core/List.pm: Make List.EXISTS-POS about 2.5x as fast - no longer allocates unless necessary - first checks validity of index before doing anything else |
12:10 | |
lizmat | afk& | 12:20 | |
BrokenRobot | Oh neat. Google Compute Engine has an API to start/stop instances: cloud.google.com/compute/docs/refe...nces/start Should be fairly easy to cook up a script fire up an instance and copy local work to it for fast spectesting | 12:25 | |
Sounds like a good project for my weekend :) | |||
roast fails now with: github.com/perl6/roast/commit/d9c3...52318b1ccd | 12:28 | ||
psch | humm, my current approach complains about modifying the grammar, which i really don't do /o\ | 12:29 | |
soo it's probably wrong :S | |||
BrokenRobot | And I second nine's sentiments about this. If you're attempting to .chomp an exabyte of data you already have a bug | ||
s/this/slowdown for unrelistic edge case/; | 12:30 | ||
psch wonders how long Str we can have | 12:32 | ||
probably depends on how we get them, too | |||
m: my $x = [~] ("a" x 1073741824) xx 2**16 # works, 2**32 doesn't | |||
camelia | ( no output ) | ||
psch | so somewhere in between for that method vOv | ||
m: my $x = [~] ("a" x 1073741824) xx 2**16; say $x.chars | |||
camelia | rakudo-moar 94a31c: OUTPUT«0» | ||
psch | aha | ||
m: my $x = [~] (("a" x 1073741824) xx 2**16); say $x.chars | 12:33 | ||
camelia | rakudo-moar 94a31c: OUTPUT«0» | ||
psch | m: my $x = [~] (("a" x 1073741824)); say $x.chars | ||
camelia | rakudo-moar 94a31c: OUTPUT«1073741824» | ||
psch | m: my $x = [~] (("a" x 1073741824) xx 4); say $x.chars | ||
camelia | rakudo-moar 94a31c: OUTPUT«0» | ||
psch | m: say (("a" x 1073741824) xx 4).map: *.chars | ||
camelia | rakudo-moar 94a31c: OUTPUT«(1073741824 1073741824 1073741824 1073741824)» | ||
psch | m: say (("a" x 1073741824) ~ ("a" x 1073741824)).chars | 12:34 | |
camelia | rakudo-moar 94a31c: OUTPUT«2147483648» | ||
BrokenRobot | k, I updated the PR to return a Failure on single-arg: github.com/rakudo/rakudo/pull/815#...-231347953 | 12:35 | |
m: say 2**16 | 12:37 | ||
camelia | rakudo-moar 94a31c: OUTPUT«65536» | ||
BrokenRobot | What's the magicalness of 1073741824? | 12:38 | |
psch | m: say "a" xx 2**64 | ||
camelia | rakudo-moar 94a31c: OUTPUT«Cannot unbox 65 bit wide bigint into native integer in block <unit> at <tmp> line 1» | ||
psch | m: say "a" x 2**64 | ||
camelia | rakudo-moar 94a31c: OUTPUT«Cannot unbox 65 bit wide bigint into native integer in block <unit> at <tmp> line 1» | ||
psch | m: say "a" x 2**63 | ||
camelia | rakudo-moar 94a31c: OUTPUT«repeat count (-9223372036854775808) cannot be negative in block <unit> at <tmp> line 1» | ||
psch | m: say "a" x 2**32 | ||
camelia | rakudo-moar 94a31c: OUTPUT«repeat count > 1073741824 arbitrarily unsupported... in block <unit> at <tmp> line 1» | ||
psch | there | ||
BrokenRobot | Ah | ||
psch | also, interesting escalation of broken tbh | ||
gfldex | m: printf("%x", 1073741824); | ||
camelia | rakudo-moar 94a31c: OUTPUT«40000000» | ||
BrokenRobot | m: say 1073741824 * 2**16 | 12:39 | |
camelia | rakudo-moar 94a31c: OUTPUT«70368744177664» | ||
psch | from "we don't do that", to overflow, to "this is too much" :S | ||
gfldex | m: printf("%x", 1073741824); say 4 * 0x10000000 | ||
camelia | rakudo-moar 94a31c: OUTPUT«400000001073741824» | ||
gfldex | m: printf("%x\n", 1073741824); say 4 * 0x10000000 | 12:40 | |
camelia | rakudo-moar 94a31c: OUTPUT«400000001073741824» | ||
BrokenRobot | m: 9999999999999999999/70368744177664 | ||
camelia | rakudo-moar 94a31c: OUTPUT«WARNINGS for <tmp>:Useless use of "/" in expression "9999999999999999999/70368744177664" in sink context (line 1)» | ||
BrokenRobot | m: say 9999999999999999999/70368744177664 | ||
camelia | rakudo-moar 94a31c: OUTPUT«142108.547152020037160» | ||
dalek | p/optimize-with-hoopl: 6ffee97 | (Pawel Murias)++ | src/vm/js/ (4 files): [js] -Ofun actually optimizes nqp::say(nqp::add_i(100, 11)) into nqp::say(111). |
12:52 | |
BrokenRobot | :/ well, that's new t/spec/S17-procasync/no-runaway-file-limit.t hanged | 13:08 | |
hung | 13:09 | ||
dalek | ast: 3e099c1 | (Zoffix Znet)++ | S03-operators/minmax.t: Fix incorrect fudge |
13:11 | |
psch | so, the UnwindException leaks when we invoke a CR that doesn't have a handler but invokes throwcatdyn_c | 13:17 | |
and we can blanket catch those in invokeDirect, because invokeDirect might *want* to leak the because the handler is outside | |||
dalek | ast: 1956de4 | (Zoffix Znet)++ | S32-str/chop.t: Add more tests for .chop with large values RT#125814 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=125814 | ||
psch | but that doesn't seem to run through handlerDynamic, but instead just up the JavaClass method invokation chain | ||
or, well, it at least doesn't seem to go through handlerDynamic is there isn't a handler | 13:19 | ||
...actually i can only confidently say that the CX for the return in the map example doesn't go through handlerDynamic | |||
nor does it go through handlerLexical, 'cause then it would already be fixed :| | 13:20 | ||
tbrowder | ref table pod handling: next set of changes, builds okay, table tests pass okay, please see PR github.com/rakudo/rakudo/pull/817 | 13:57 | |
dalek | ast: dbb6a58 | (Zoffix Znet)++ | S19-command-line/repl.t: Pressing CTRL+D in REPL produces correct output on exit RT#70297 |
14:01 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...l?id=70297 | ||
dalek | p: a9380c3 | (Pawel Murias)++ | t/nqp/067-container.t: Test that the container is passed as an invocant to a method. |
14:14 | |
psch | grr | 14:39 | |
third time today i get a digit-off-by-one error when trying to check for the category of the UnwindException >_> | |||
i had 66536 once and 65535 twice now | |||
i mean, i just mistyped so it's kinda ok, except 13 minutes compile cycle :| | 14:40 | ||
that's probably an argument for constants on the jvm level :P | 14:47 | ||
well, using them, as we actually already have them | |||
dalek | p: d90ea08 | peschwa++ | src/vm/jvm/runtime/org/perl6/nqp/runtime/ (2 files): Minor clean-up and perf improvement. We can just keep one int int CallSite around instead of allocating a new one every time we want to call into lexicalHandlerNotFoundError. |
14:53 | |
ast: d9781be | (Zoffix Znet)++ | S19-command-line/repl.t: Exceptions from lazy-evaluated things do not crash REPL RT#128470 |
14:55 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128470 | ||
b2gills | An improvement for method chop could be 「my int $len = nqp::chars($str); if $chopping > $len { ... } else { ... }」 That way the only case where you have to deal with a large Int you don't actually use it ( because str is limited to less than that anyway ) | 15:04 | |
BrokenRobot | Is debugging unspace problems hard? Thinking of taking a crack at rt.perl.org/Ticket/Display.html?id=128462 | 15:18 | |
timotimo | could be as easy as putting an <.allow_unsp> (or whatever it's called) in the right place | 15:19 | |
BrokenRobot | sweet, I'll try (/me sees such place) | 15:20 | |
gfldex | timotimo: you where real good at updating Pod::To::HTML on docs.perl6.org. Could you do it again please? | 15:26 | |
timotimo | done | 15:27 | |
gfldex | thanks | 15:28 | |
BrokenRobot | Nope, that was not the right place. The try-and-see programming approach doesn't really work with rakudo due to the build times involved :) | 15:29 | |
And searching online for "What are some things to do while waiting for your code to compile?" really gives terrible results. | 15:31 | ||
Most common answer: "make your compiling faster" ~_~ | 15:32 | ||
"Look for more add-ons to add to Chrome, Firefox, your mobile phone, your tablet, or whatever." really? | 15:37 | ||
gfldex | "make your compiling faster" could send lizmat into an infinite loop | 15:41 | |
timotimo | liz mostly works on the core setting, which doesn't factor into the speed of the compiler, only its run time | 15:43 | |
(because that's the input data) | |||
psch | BrokenRobot: don't forget about --target=parse | 16:37 | |
timotimo | ah, right, when you change the parser you don't actually have to build the whole core setting | ||
psch | BrokenRobot: and, honestly, the remaining unspace problems might not really be that easy - i think i caught the easy ones already in my early contributions | ||
BrokenRobot | Well, I got this: gist.github.com/zoffixznet/6a1a698...65f52fc540 | 16:38 | |
I tried sticking <.unsp> at various places in <identifier> and <longname> but the only change I produced is it was complaining about no such sub "term\" | |||
psch | hrm | 16:39 | |
yeah, that one does look somewhat hard from here | |||
i mean, i never looked at the low-prec methodop | |||
so idk what TimToady++ did there to make that work :) | |||
BrokenRobot | k, I'll leave it. Plenty of other fruits | ||
psch | #78188 looks closeable with tests, if that's to your liking | 16:42 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...l?id=78188 | ||
psch | m: say &[Xxx] | ||
camelia | rakudo-moar 94a31c: OUTPUT«-> + is raw { #`(Block|58035688) ... }» | ||
psch | m: say &infix:<Xxx> | ||
camelia | rakudo-moar 94a31c: OUTPUT«-> + is raw { #`(Block|66025872) ... }» | ||
psch | m: say &infix:<Xxx>(<a b>, (2,3)) | ||
camelia | rakudo-moar 94a31c: OUTPUT«((a a) (a a a) (b b) (b b b))» | ||
psch | BrokenRobot: what kind of bugs are you looking for? LTA errors, grammar..? | 16:48 | |
#123903 looks pretty fun, fwiw | 16:51 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=123903 | ||
BrokenRobot | Well, I just spotted that unspace one and it looked easy at the first look, so I tried it. | 16:53 | |
But it was suggested there was work to be done on the profiler output, so I think I'll hack on that on the weekend. | 16:54 | ||
sortiz | \o #perl6-dev | ||
BrokenRobot | \o | ||
psch | o/ | ||
timotimo | if it complains about term\, you'll have to make sure the thing that matches term mustn't try to nom a bare \ | 16:55 | |
BrokenRobot | Well, this is the only thing I changed, so I'm not even sure how I'd go about noming bare anythings :) gist.github.com/zoffixznet/7ec794b...fa2bdf77fc | 16:57 | |
psch | well, that means an unspace can belong to an identifier | ||
which is why it complains about "term\" | |||
timotimo | right | 17:00 | |
that's problematic | |||
BrokenRobot | sub term:<term\\> { say 42 }; term\ | 17:01 | |
m: sub term:<term\\> { say 42 }; term\ | |||
camelia | rakudo-moar 94a31c: OUTPUT«42» | ||
psch | m: sub term:<term\\> { say 42 }; term \ | 17:02 | |
camelia | rakudo-moar 94a31c: OUTPUT«===SORRY!=== Error while compiling <tmp>You can't backslash thatat <tmp>:1------> sub term:<term\\> { say 42 }; term \⏏<EOL> expecting any of: argument list term» | ||
psch | m: sub term:<term\\> { say 42 }; term \ .Str | ||
camelia | rakudo-moar 94a31c: OUTPUT«===SORRY!=== Error while compiling <tmp>Undeclared routine: term used at line 1» | ||
psch | those two probably parse with your local change | ||
but, well, it's not an unspace in the CORE sense anymore, but a really weird variable name :P | |||
or term, rather | |||
...or i'm misremembering unspace | 17:03 | ||
BrokenRobot | 7 [Coke] RT: 1338; CONC: 7; GLR: 6; JVM: 69; LHF: 1; LTA: 74; NYI: 28; OSX: 6; PERF: 16; POD: 3; PRECOMP: 4; RFC: 22; SEGV: 22; STAR: 1; TESTNEEDED: 12; TODO: 8; UNI: 5; UNTAGGED: 675; WEIRD: 3 | 17:05 | |
[Coke]: what's "RT" in that list? The total number of tickets? | |||
'cause on the site it says "Found 1,364 tickets" seems off. | |||
probabilistically, there's a whole ton of "testsneeded" tickets that aren't tagged | 17:12 | ||
[Coke] | BrokenRobot: skipping Stalled tickets | 17:40 | |
which right now is primarily macro-related stuff. | 17:41 | ||
BrokenRobot: sure, I can only report on what's tagged. | |||
my hope with this occasional ping is for devs to go "oh, maybe I'll go fix some segfaults today". :| | |||
(or whatever.) | |||
BrokenRobot | I was just curious why there was a difference. | 17:42 | |
[Coke] | which reminds me: | 17:45 | |
RT: 1335; CONC: 7; GLR: 6; JVM: 69; LHF: 1; LTA: 75; NYI: 28; OSX: 6; PERF: 16; POD: 3; PRECOMP: 4; RFC: 22; SEGV: 22; STAR: 1; TESTNEEDED: 15; TODO: 8; UNI: 5; UNTAGGED: 674; WEIRD: 3 | |||
PRECOMP: 5 - one was mistagged | 17:46 | ||
BrokenRobot | Do we have a protocol for when 6.c roast has faulty tests? | 18:03 | |
Is that was 6.c-errata for? | 18:04 | ||
s/was/what./; | |||
Specifically this: rt.perl.org/Ticket/Display.html?id=128203 the failures were due to a faulty test. If we can amend the 6.c testsuite, we can revert hoelzro++'s temporary workaround in Test::subtest | 18:05 | ||
[Coke] | where should a doc for "how to deal with tickets for Rakudo Perl 6" go ? | 18:16 | |
Given that we need a doc that covers things for 'how to report a bug' to 'how to deal with tickets' and "you know, there are really like 8 different bug queues' | 18:17 | ||
[Coke] guesses github.com/rakudo/rakudo/wiki is a fine place to start. | |||
some of these wiki pages are (old) and written in .textile. is there a compelling reason to not switch them to .md as I touch things? | 18:25 | ||
ah, no, since github is dropping textile support. | |||
gfldex | m: my class C:ver<1.2.3>:auth<me> {}; say [ C.^ver(), C.^auth ]; | 18:49 | |
camelia | rakudo-moar 94a31c: OUTPUT«[v1.2.3 me]» | ||
gfldex | m: my package P:ver<1.2.3>:auth<me> {}; say [ C.^ver(), C.^auth ]; | 18:50 | |
camelia | rakudo-moar 94a31c: OUTPUT«===SORRY!=== Error while compiling <tmp>Undeclared name: C used at line 1» | ||
gfldex | m: my package P:ver<1.2.3>:auth<me> {}; say [ P.^ver(), P.^auth ]; | ||
camelia | rakudo-moar 94a31c: OUTPUT«Method 'ver' not found for invocant of class 'Perl6::Metamodel::PackageHOW' in block <unit> at <tmp> line 1» | ||
dalek | ast: feb3c86 | (Zoffix Znet)++ | S12-introspection/meta-class.t: Test basic metamethods on a module |
19:19 | |
ast: c22897b | (Zoffix Znet)++ | S12-introspection/meta-class.t: Add missing plan to subtest |
19:20 | ||
[Coke] | anyone mind if I kill this page? github.com/rakudo/rakudo/wiki/NG-issues | 19:24 | |
also, might be nice if we got updates here if someone updated rakudo.wiki | |||
dalek | ast/RT128579: 95088d8 | (Zoffix Znet)++ | S12-introspection/meta-class.t: Test basic metamethods for a Package RT#128579 |
19:26 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128579 | ||
dalek | ast: a83ede0 | usev6++ | S32-exceptions/misc.t: Fudge test for JVM (UnwindException) |
19:30 | |
kudo/nom: 26817b3 | (Zoffix Znet)++ | src/Perl6/Metamodel/PackageHOW.nqp: Make .ver and .auth MetaModel methods available on Packages Fixes RT#128579 |
19:33 | ||
kudo/nom: ae36a14 | moritz++ | src/Perl6/Metamodel/PackageHOW.nqp: Merge pull request #818 from zoffixznet/fix-RT128579-package-verf Make .ver and .auth MetaModel methods available on Packages |
|||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128579 | ||
[Coke] | .seen skids | ||
yoleaux2 | I saw skids 7 Jul 2016 16:03Z in #perl6-dev: <skids> m: sub a (int() $f) { }; a(9) # maybe if that was brought up to speed it would help with chr | ||
skids | o/ | ||
dalek | ast: fe1d544 | (Zoffix Znet)++ | S12-introspection/meta-class.t: Test basic metamethods for a Package (#134) RT#128579 |
||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128579 | ||
[Coke] | skids: removing github.com/rakudo/rakudo/wiki/tasks, ok? | ||
most of it is parrot related, the rest is probably outdated/already in RT. | 19:34 | ||
skids | Yeah, way old, go for it. | ||
gfldex | m: enum P:ver<4.2.3>:auth<me@here.local>; | 19:44 | |
camelia | rakudo-moar 94a31c: OUTPUT«===SORRY!=== Error while compiling <tmp>Undeclared name: P:ver<4.2.3>:auth<me@here.local> used at line 1Undeclared routine: enum used at line 1» | ||
gfldex | m: my $capture = (2, 3, apples => 2).capture; say $capture.Str; | 19:55 | |
camelia | rakudo-moar ae36a1: OUTPUT«Method 'capture' not found for invocant of class 'List' in block <unit> at <tmp> line 1» | ||
gfldex | m: my $capture = (2, 3, apples => 2).Capture; say $capture.Str; | ||
camelia | rakudo-moar ae36a1: OUTPUT«This representation (VMIter) cannot unbox to a native string (for type BOOTIter) in block <unit> at <tmp> line 1» | ||
gfldex | m: my $capture = (2, 3, apples => 2).Capture; say $capture.perl; | ||
camelia | rakudo-moar ae36a1: OUTPUT«\(2, 3, :apples(2))» | ||
[Coke] | we have 44 unresolved pull requests on rakudo. | 19:56 | |
(going back to 2015) | 19:57 | ||
dalek | ast: 5a156d9 | usev6++ | S32-str/comb.t: Unfudge some tests for JVM |
20:03 | |
gfldex | m: my $l = (1,2, :2c); | 20:06 | |
camelia | ( no output ) | ||
gfldex | m: my $l = (1,2, :2c); $l.Str.put; | 20:07 | |
camelia | rakudo-moar ae36a1: OUTPUT«1 2 c 2» | ||
dalek | ast: f09ad45 | usev6++ | S32-str/comb.t: Fudge (skip) some tests for JVM |
20:10 | |
bartolin | psch++ # we're back to a manageable number of spectest failures on r-j | 20:12 | |
psch | \o/ | ||
still, many things just hidden and not really fixed :| | |||
dalek | kudo/nom: 5745195 | peschwa++ | src/core/Capture.pm: Unbreak Capture.Str. VMIter needs the nqp ops to work. |
20:16 | |
[Coke] | rt.perl.org/Ticket/Display.html?id=127050 - this appears to be a DIHWIDT - the role is applied to the literal 1, making a new object; when you ++ you're getting a new instance, a (fresh) 2. No? | 20:45 | |
jnthn | [Coke]: It's correct behavior. The ticket is confused. | 20:49 | |
(So yes, you're correct. :)) | |||
[Coke] | jnthn: ok, I'll reject. | 20:50 | |
jnthn | Just to add a little, $i++ is taking the Scalar container $i and changing the value it references. | ||
And the role was, like you said, on the value 1, and now it contains the value 2. | 20:51 | ||
jnthn didn't manage to look into the latest MMD ticket today, as hoped. :( | 20:52 | ||
Will see if I find a moment at the weekend. | |||
Prolly gone for the evening :) | 20:53 | ||
[Coke] | reminder, rt.perl.org/Ticket/Display.html?id=128423 - rakudo.org/ is broken. | ||
perlpilot | [Coke]: might mention that to pmichaud on #perl6 since he isn't here | 20:57 | |
dalek | rakudo/nom: a44cdd4 | (Tom Browder)++ | src/Perl6/Pod.nqp: | ||
rakudo/nom: preparing for upcoming changes: | |||
rakudo/nom: | |||
rakudo/nom: + add vars for future use | |||
rakudo/nom: for pattern matching | |||
rakudo/nom: for adding exception handling info | |||
rakudo/nom: | |||
rakudo/nom: + rename 'sub formatted_text' to 'normalize_text' | |||
rakudo/nom: suggested better name | |||
rakudo/nom: | |||
rakudo/nom: + denote two code sections that may be refactored | |||
rakudo/nom: mental prep for more changes | |||
rakudo/nom: | |||
rakudo/nom: + add comments | |||
ast: 407e8c1 | usev6++ | S32-str/comb.t: Mention ticket RT #128580 for skipped tests |
21:01 | ||
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=128580 | ||
dalek | ast: 37f9e40 | usev6++ | S32-exceptions/misc.t: Fudge tests with NPE for rakudo-j |
21:16 | |
psch | all those CX handling bugs :( | 21:19 | |
it's literally dozends | 21:20 | ||
bartolin | psch: :-/ but on a related note: didn't you intend to change the travis configuration once jvm builds are green again? would now be a good moment to do that? | 21:25 | |
psch | bartolin: yeah, i suppose we can | ||
bartolin: i mean, install-core-dist.pl runs through completion, so yeah | 21:26 | ||
dalek | kudo/nom: 58c2628 | lizmat++ | src/core/List.pm: Make List.elems about 4.5x faster - don't allocate when we don't need to - rewrite logic in nqp, and be smarter about it - now also capable of handling bare Lists without $!reified |
||
timotimo | wow, damn | 21:29 | |
psch | i think the osx jvm builds still have some other problem..? | 21:31 | |
i have no idea how i'd check, fwiw | |||
well, except via travis :P | 21:32 | ||
dalek | kudo/nom: 5a83924 | peschwa++ | .travis.yml: Unallow failures for linux/jvm. I think we have some other problem on OSX, but I can't test that. |
21:33 | |
bartolin | this was a recent commit (looks good to me): travis-ci.org/rakudo/rakudo/builds/142733653 | ||
psch | ah, alright | ||
gonna unallow those too then | |||
dalek | kudo/nom: 897c22c | peschwa++ | .travis.yml: bartolin++ for checking that jvm on OSX works too. |
21:34 | |
bartolin | \o/ (but for the records: I'm unable to test on OS X myself) | 21:35 | |
psch trusts in travis | 21:38 | ||
dalek | kudo/nom: 110704d | lizmat++ | src/Perl6/Pod.nqp: Revert "preparing for upcoming changes:" This reverts commit a44cdd48428a8e44e1d7550bea99305efa7e22d1. This broke the following spectest files: t/spec/S26-documentation/block-leading.t t/spec/S26-documentation/module-comment.t t/spec/S26-documentation/block-trailing.t t/spec/S26-documentation/multiline-leading.t t/spec/S26-documentation/multiline-trailing.t t/spec/S26-documentation/wacky.t t/spec/S26-documentation/why-both.t f583f22 | lizmat++ | src/core/List.pm: Make List.Bool about 4x faster - don't allocate when we don't need to - rewrite logic in nqp, and be smarter about it - now also capable of handling bare Lists without $!reified |
21:51 | |
lizmat | looks like t/spec/S19-command-line/repl.t needs a fix after recent REPL text changes | 22:04 | |
lizmat | and this concludes my hacking for today | ||
so good night, #perl6-dev! | |||
dalek | ast: 06b24bb | (Zoffix Znet)++ | S12-introspection/meta-class.t: ID metamethods on a package are absent by design Per discussion on: github.com/rakudo/rakudo/pull/818#...-231468771 |
22:09 | |
Zoffix | PR to revert :ver and :auth on packages: github.com/rakudo/rakudo/pull/819 | ||
sortiz | Zoffix, sorry to be PR#815's killjoy :( | 23:21 |