[Tux] |
|
10:10 | |||||||||||||||||||||||||||||||||||||
lizmat waves from the Bailador workshop at the LPW | 11:53 | ||||||||||||||||||||||||||||||||||||||
timotimo | o/ | 11:58 | |||||||||||||||||||||||||||||||||||||
bartolin | \o | 12:05 | |||||||||||||||||||||||||||||||||||||
tadzik | o/ | 12:14 | |||||||||||||||||||||||||||||||||||||
tbrowder | \o/ | ||||||||||||||||||||||||||||||||||||||
tadzik | so many 6books! | ||||||||||||||||||||||||||||||||||||||
lizmat | lunch& | 13:00 | |||||||||||||||||||||||||||||||||||||
tbrowder | .tell AlexDaniel (not yelling) rakudo PR #1240 will reduce open bug count | 14:55 | |||||||||||||||||||||||||||||||||||||
yoleaux | tbrowder: I'll pass your message to AlexDaniel. | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | . | ||||||||||||||||||||||||||||||||||||||
yoleaux | 14:55Z <tbrowder> AlexDaniel: (not yelling) rakudo PR #1240 will reduce open bug count | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | will take a look in 5 minutes | ||||||||||||||||||||||||||||||||||||||
wow that's a lot of code! | 15:03 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo/master: 14 commits pushed by (Tom Browder)++, (Aleks-Daniel Jakimenko-Aleksejev)++ review: github.com/rakudo/rakudo/compare/f...4151423552 |
15:04 | |||||||||||||||||||||||||||||||||||||
rakudo: 4003b93f63 | (Aleks-Daniel Jakimenko-Aleksejev)++ | src/Perl6/Pod.nqp Tiny corrections (whitespace and comments) Untabified the whole buffer (because mixed styles were used), marked some TODO comments as TODO comments. |
15:11 | ||||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: you're using emacs, right? | 15:12 | |||||||||||||||||||||||||||||||||||||
tbrowder: have you tried better-defaults package? Or maybe you should at least set indent-tabs-mode to nil | 15:13 | ||||||||||||||||||||||||||||||||||||||
tbrowder: also, if possible, mark every todo comment as # TODO, otherwise it's a little bit hard to distinguish useful commentary from things that should be done | |||||||||||||||||||||||||||||||||||||||
tbrowder | AlexDaniel: yes, rogue Emacs needs fixin’...i’ll detabify and make corrections and let you know. will be later today | 16:03 | |||||||||||||||||||||||||||||||||||||
AlexDaniel | tbrowder: I already “corrected” it :) | ||||||||||||||||||||||||||||||||||||||
tbrowder | thnx | 16:17 | |||||||||||||||||||||||||||||||||||||
lizmat | m: role A { multi method a(Int $a) { ... }; multi method(Str $b) { ... } }; class B does A { multi method a(Str $b) { } } # correct | 17:32 | |||||||||||||||||||||||||||||||||||||
camelia | 5===SORRY!5=== Error while compiling <tmp> Multi method 'a' with signature :(B $: Int $a, *%_) must be implemented by B because it is required by a role at <tmp>:1 |
||||||||||||||||||||||||||||||||||||||
lizmat | m: role A { multi method a("foo") { ... }; multi method("bar") { ... } }; class B does A { multi method a("foo") { } } # so why is this not complaining? | 17:33 | |||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
lizmat | should this be considered a bug ? | ||||||||||||||||||||||||||||||||||||||
jnthn | I think that falls under "signatures with where clauses can't be compared" | 17:39 | |||||||||||||||||||||||||||||||||||||
timotimo | was it supposed to be "multi method()" rather than "multi method a()"? | 17:40 | |||||||||||||||||||||||||||||||||||||
jnthn | And "foo" is just sugar for Str $ where "foo" | ||||||||||||||||||||||||||||||||||||||
I figured it was meant to be a :) | |||||||||||||||||||||||||||||||||||||||
But I'm not sure if it'll make a difference | |||||||||||||||||||||||||||||||||||||||
lizmat | m: role A { multi method a("foo") { ... }; multi method a("bar") { ... } }; class B does A { multi method a("foo") { } } # so why is this not complaining? | 17:41 | |||||||||||||||||||||||||||||||||||||
camelia | 5===SORRY!5=== Error while compiling <tmp> Multi method 'a' with signature :(B $: Str $ where { ... }, *%_) must be implemented by B because it is required by a role at <tmp>:1 |
||||||||||||||||||||||||||||||||||||||
lizmat | ah, it *is* complaining | ||||||||||||||||||||||||||||||||||||||
m: role A { multi method a("foo") { ... }; multi method a("bar") { ... } }; class B does A { multi method a("foo") { }; multi method a("bar") { } } | |||||||||||||||||||||||||||||||||||||||
camelia | ( no output ) | ||||||||||||||||||||||||||||||||||||||
lizmat | whee :-) | ||||||||||||||||||||||||||||||||||||||
sorry for the noise :-) | |||||||||||||||||||||||||||||||||||||||
Geth | roast: FCO++ created pull request #358: Add test to Supply.zip |
17:48 | |||||||||||||||||||||||||||||||||||||
synopsebot | SPEC#358 [open]: github.com/perl6/roast/pull/358 Add test to Supply.zip | ||||||||||||||||||||||||||||||||||||||
jnthn | Ah, hmm | ||||||||||||||||||||||||||||||||||||||
Maybe literals are a special case in signature comparison? | 17:49 | ||||||||||||||||||||||||||||||||||||||
lizmat | looks like it :-) | 17:50 | |||||||||||||||||||||||||||||||||||||
jnthn | Guess that's fine, may as well do what we reasonably can :) | 17:51 | |||||||||||||||||||||||||||||||||||||
lizmat | it's probably a smart match under the hood ? :-) | 17:53 | |||||||||||||||||||||||||||||||||||||
Geth | roast: 43ca6ea5d6 | (Fernando Correa de Oliveira)++ | S17-supply/zip.t Add test to Supply.zip |
17:56 | |||||||||||||||||||||||||||||||||||||
roast: 225fd55385 | (Fernando Correa de Oliveira)++ | S17-supply/zip.t Change timeout for zip test |
|||||||||||||||||||||||||||||||||||||||
roast: 00e4de362c | (Jonathan Worthington)++ (committed using GitHub Web editor) | S17-supply/zip.t Merge pull request #358 from FCO/test-supply-zip Add test to Supply.zip |
|||||||||||||||||||||||||||||||||||||||
jnthn | lizmat: Probably. I think you wrote that bit ;) | ||||||||||||||||||||||||||||||||||||||
lizmat | :-) | ||||||||||||||||||||||||||||||||||||||
LPW shutting down& | 18:24 | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: FCO++ created pull request #1270: Make zip-latest's response supply be done when any supply become done |
18:32 | |||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1270 [open]: github.com/rakudo/rakudo/pull/1270 Make zip-latest's response supply be done when any supply become done | ||||||||||||||||||||||||||||||||||||||
Geth | rakudo: FCO++ created pull request #1271: Fix Supply.throttle() |
20:06 | |||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1271 [open]: github.com/rakudo/rakudo/pull/1271 Fix Supply.throttle() | ||||||||||||||||||||||||||||||||||||||
Geth | roast: FCO++ created pull request #359: Test if Supply.throttle($, $second) become done when its source becom… |
||||||||||||||||||||||||||||||||||||||
synopsebot | SPEC#359 [open]: github.com/perl6/roast/pull/359 Test if Supply.throttle($, $second) become done when its source becom… | ||||||||||||||||||||||||||||||||||||||
SmokeMachine | is it ^^ right? could I merge it? | 21:07 | |||||||||||||||||||||||||||||||||||||
Geth | rakudo: usev6++ created pull request #1272: Pass typenames to two compile time exceptions |
21:52 | |||||||||||||||||||||||||||||||||||||
synopsebot | RAKUDO#1272 [open]: github.com/rakudo/rakudo/pull/1272 Pass typenames to two compile time exceptions |