dalek | ast: 5844a89 | samcv++ | S15-nfg/ (5 files): S15-nfg update mass-roundtrip-* for Unicode 9.0 The script is now updated to work with Unicode 9.0 as well. |
01:00 | |
ast: f8070c4 | samcv++ | S15-nfg/mass-equality (2 files): test-gen.p6's mass-equality.t generation has been moved to a new script This was working fine for Unicode 9.0, so it was not messed with due to it being much more complex and involved rearranging the codepoints. |
|||
notviki | samcv: this might be up your alley. Is that even a bug? doesn't look like that closing quote is the matching quote for the opener: rt.perl.org/Ticket/Display.html?id=130406 | 01:04 | |
geekosaur was under the impression matching pairs weren't part of the spec... because they can depend on the locale | 01:07 | ||
samcv | let me look | 01:11 | |
m: Q„1“; | |||
camelia | rakudo-moar 1374fc: OUTPUT«===SORRY!=== Error while compiling <tmp>Couldn't find terminator ” (corresponding „ was at line 1)at <tmp>:1------> Q„1“;⏏<EOL> expecting any of: ”» | ||
samcv | well that seems fine right. unless | ||
yeah i don't think that | 01:12 | ||
notviki | Reading the code, it actually goes out to NQP land and uses our brackets string to find the matching losing delimiter | ||
samcv | yep | ||
yeah it's not in bidi paired brackets or bidi mirroring glyphs | 01:13 | ||
so it is not a proper matched bracket (/quote) | 01:14 | ||
notviki | m: Q„1”; | ||
camelia | rakudo-moar 1374fc: OUTPUT«WARNINGS for <tmp>:Useless use of constant string "1" in sink context (line 1)» | ||
samcv | m: "„".uniprop.say | ||
camelia | rakudo-moar 1374fc: OUTPUT«Ps» | ||
samcv | m: "„".uniname.say | 01:15 | |
camelia | rakudo-moar 1374fc: OUTPUT«DOUBLE LOW-9 QUOTATION MARK» | ||
samcv | there is no matching thing for this. | ||
almost certain | |||
notviki bets $1000 samcv doesn't use 80-char linewidth | |||
samcv | yeah notviki there is no matching character for that | ||
u: double quotation mark | |||
unicodable6 | samcv, U+00AB LEFT-POINTING DOUBLE ANGLE QUOTATION MARK [Pi] («) | ||
samcv, U+00BB RIGHT-POINTING DOUBLE ANGLE QUOTATION MARK [Pf] (») | |||
samcv, U+201C LEFT DOUBLE QUOTATION MARK [Pi] (“) | |||
samcv, gist.github.com/987342005a219bbaf5...b968f36cbc | |||
samcv | yep | 01:16 | |
i remember because of before when i checked every single Ps and Pi thingy | |||
m: "„".uniprop('Bidi_Mirrored').say | 01:17 | ||
camelia | rakudo-moar 1374fc: OUTPUT«False» | ||
notviki | i.imgur.com/kA9qztb.png :) | ||
samcv | well. it was like 100000000000000000 calumns wide before! | ||
notviki | :) | ||
samcv | all on one line :P | ||
i will fit to 80 columns though | |||
soonish. idk sometime today | 01:18 | ||
notviki, about that bug report. tell them that Q quoting is not the same as quotation marks | 01:19 | ||
and so take two opposite/matching delimiters. in most cases they overlap | 01:20 | ||
like “ ” but not for this | |||
notviki | m: dd Q“x” | ||
camelia | rakudo-moar 1374fc: OUTPUT«"x"» | ||
samcv | oh they messaged me saying they got the CLA | 01:21 | |
notviki | \o/ | ||
samcv | should i forward that email to somebody | ||
notviki | In my case they notified jnthn and he gave me the commit bit | ||
b2gills | m: say „1“; | ||
camelia | rakudo-moar 1374fc: OUTPUT«1» | ||
samcv | You can let the Rakudo admins know that we've received the CLA. | ||
well i am letting you guys know :P unknown whether they sent anything | |||
it was Allison which replied to my email | 01:22 | ||
notviki passes samcv the torch | |||
I'm no longer the newest core member \o/ | |||
samcv | \o/ | ||
dalek | kudo/nom: 1ee24cc | (Zoffix Znet)++ | src/core/List.pm: Fix spurious warnings in List.reduce When the mapper sub has associativity set, the %prec ends up having stuff (so the first conditional passes), but the `prec` key is still absent. So when we infix:<eq> on it, it triggers warnings about stringifying uninitialzed values. Fix by checking if it's inited first. Bugfind: irclog.perlgeek.de/perl6/2016-12-26#i_13799375 |
02:06 | |
ast: 261c2c3 | (Zoffix Znet)++ | packages/Test/Util.pm: Add doesn't-warn() utility test sub Checks whether a warning is emitted. This lets us test for warnings faster than by using is_run to fire up an entire perl6 for us. |
02:11 | ||
ast: 27ddcc8 | (Zoffix Znet)++ | S32-list/reduce.t: Remove trailing whitespace |
|||
ast: b2e46ab | (Zoffix Znet)++ | S32-list/reduce.t: Test .reduce doesn't produce spurious warnings 9418115 | samcv++ | S15-unicode-information/uniprop.t: Fudge some more tests for uniprop |
|||
notviki stares down dalek | |||
hm, I can teach huggable to take over for him when he fails like that | 02:18 | ||
t/spec/S15-unicode-information/uniprop.rakudo.moar (Wstat: 768 Tests: 152 Failed: 3) | 02:23 | ||
Failed tests: 121-123 | |||
Non-zero exit status: 3 | |||
samcv | do i need to fudge some more? | 02:24 | |
line numbers? | |||
actually nvm i will do it | 02:25 | ||
samcv needs to get multiple installations of rakudo installed | |||
notviki | Rakudobrew is just for that: github.com/tadzik/rakudobrew | 02:26 | |
samcv | notviki, how do you switch between them easiest notviki | 02:30 | |
ok. passes now :) | |||
notviki | By "multiple" you mean different commits? I think it's rakudobrew switch | 02:36 | |
notviki doesn't actually use multiple versions | |||
samcv | ack ok my changes to moarvm mess up one of the tests | 02:41 | |
oddly somehow it's this rt.perl.org/Ticket/Display.html?id=125190 | 03:04 | ||
hm | 03:05 | ||
ack so it wosn't any of my changes at all. | 03:16 | ||
hmm well i got it fixed. so that is all good | 03:21 | ||
er maybe not. this is really weird. will need to investigate. this is quite odd | 03:24 | ||
m: say ' ' ~~ /<:space>/ | 03:56 | ||
camelia | rakudo-moar 1ee24c: OUTPUT«「 」» | ||
samcv | m: say ' ' ~~ /<:Space>/ | ||
camelia | rakudo-moar 1ee24c: OUTPUT«「 」» | ||
samcv | wtf. so for some reason it's failing for :space but not :Space, which works fine | 03:57 | |
ugh what gives. uniprop-bool for 'space' returns true, so it seems to be fine. but it fails that test | 04:01 | ||
with the regex | |||
dalek | kudo/nom: b01dfcd | (Zoffix Znet)++ | src/Perl6/Actions.nqp: Warn when using Bool:D as type constraint in signatures Fixes RT#130182: rt.perl.org/Ticket/Display.html?id=130182 |
06:34 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130182 | ||
dalek | ast: 7095892 | (Zoffix Znet)++ | S06-signature/types.t: Test Bool:D as literals in signatures warn RT#130182: rt.perl.org/Ticket/Display.html?id=130182 Rakudo fix: github.com/rakudo/rakudo/commit/b01dfcda90 |
06:35 | |
synopsebot6 | Link: rt.perl.org/rt3//Public/Bug/Displa...?id=130182 | ||
gfldex | .u „ | 09:57 | |
yoleaux2 | U+201E DOUBLE LOW-9 QUOTATION MARK [Ps] („) | ||
gfldex | .u “ | 09:58 | |
yoleaux2 | U+201C LEFT DOUBLE QUOTATION MARK [Pi] (“) | ||
gfldex | .u “ | ||
yoleaux2 | U+201C LEFT DOUBLE QUOTATION MARK [Pi] (“) | ||
gfldex | notviki: this will come up again because of de.wikipedia.org/wiki/Anf%C3%BChru...e_Sprachen | 09:59 | |
loads of eastern european languages use them | 10:00 | ||
samcv | the main question becomes, do we support unicode matching brackets + mirroring glyphs that are open/closers AND all the quoting constructs we allow? | 10:03 | |
at the moment we only allow matching brackets/mirroring glyphs and also characters that are the same open and same close | |||
gfldex | m: say „we do, but not for Q“. | 10:04 | |
camelia | rakudo-moar b01dfc: OUTPUT«===SORRY!=== Error while compiling <tmp>Malformed postfix callat <tmp>:1------> say „we do, but not for Q“.⏏<EOL>» | ||
samcv | if we add that one, we should add every one of the quotation marks | ||
m: say ”test”; say Q”test” | |||
gfldex | m: say „we do, but not for Q.“ | ||
camelia | rakudo-moar b01dfc: OUTPUT«===SORRY!=== Error while compiling <tmp>Use of a closing delimiter for an opener is reservedat <tmp>:1------> say ”test”; say Q⏏”test”» | ||
rakudo-moar b01dfc: OUTPUT«we do, but not for Q.» | |||
samcv | yeah exactly | ||
gfldex | i'm cool with that, just need to doc it. | ||
samcv | i personally favor just going with brackets/mirroring glyphs, but being consistent is the most important | 10:05 | |
it's about time for my bed. night everybody :) | 10:12 | ||
dalek | kudo/nom: 2496963 | lizmat++ | src/core/Str.pm: Re-imagine Str.split(<a b c>) some more - 10% (small string, 1 hit each) to 40% (larger, 100 hits each) faster - don't use nqp::p6sort anymore, instead use R:I.MERGESORT-REIFIED-LIST-WITH - so we don't need to setup indices + we have mapping simplified - don't copy values when limiting, just limit the original result |
14:11 | |
lizmat | this should probably also be noticeable in test-t ^^^ | ||
only one more step towards the elimination of nqp::p6sort | |||
but that's for later today | 14:12 | ||
afk& | |||
notviki | m: dd | 16:37 | |
camelia | rakudo-moar 249696: OUTPUT«block <unit>» | ||
notviki | m: dd ‘…‚ | ||
camelia | rakudo-moar 249696: OUTPUT«===SORRY!=== Error while compiling <tmp>Unable to parse expression in curly single quotes; couldn't find final "’" at <tmp>:1------> dd ‘…‚⏏<EOL> expecting any of: argument list curly single quotes…» | ||
notviki | m: dd “…„ | 16:38 | |
camelia | rakudo-moar 249696: OUTPUT«===SORRY!=== Error while compiling <tmp>Unable to parse expression in curly double quotes; couldn't find final '”' at <tmp>:1------> dd “…„⏏<EOL> expecting any of: argument list curly double quotes…» | ||
notviki | If there's anything we can learn from supporting U+2212 is to say "no" once in a while. | 16:39 | |
[Tux] | Nice ♥ ! | 16:58 | |
This is Rakudo version 2016.12-102-g2496963ea built on MoarVM version 2016.12-19-ge2ffc358 | |||
csv-ip5xs 3.057 | |||
test 14.270 | |||
test-t 5.328 | |||
csv-parser 15.208 | |||
notviki | \o/ | 17:05 | |
buggable: speed | |||
buggable | notviki, ▅▅▅▆▅▅▅▄▅▅▆▅▅▅▆▄▄▄▄▄▅▄▄▅▄█▃▃▄▄▃▄▄▃▄▃▄▄▃▄▄▄▂▃▂▂▂▂▁▁ data for 2016-12-04–2016-12-26; variance: 5.328s–7.592s | ||
notviki | :D | ||
hmm... I got $node.node.CURSOR.worry("Useless use of $sym in sink context"); call in unwanted() and I added debug prints to see that I'm hitting that conditional, but no warning gets printed | 18:02 | ||
mst | notviki: your debug print is going to stderr or an autoflushing filehandle right? | 18:08 | |
(probably a stupid question but I've seen/done it so many times) | |||
notviki | Yeah to stderr. Why? | 18:10 | |
m: 1...2 | |||
camelia | rakudo-moar 249696: OUTPUT«Potential difficulties: Useless use of ... in sink context at <tmp>:1 ------> 1...⏏2» | ||
notviki | It's the same conditional that prints ^ that warning | ||
I'm trying to make | |||
m: 1 but 2 | |||
camelia | ( no output ) | ||
notviki | Warn and it doesn't... so I'm missing something else that needs to be done | 18:11 | |
lizmat | .tell samcv did you forget to push Missing test file: t/spec/S15-nfg/mass-chars.t ? | 18:53 | |
yoleaux2 | lizmat: I'll pass your message to samcv. | ||
dalek | kudo/nom: 7ac0455 | lizmat++ | src/core/Rakudo/Internals.pm: Introducing R:I.MERGESORT-REIFIED-LIST-AS - sort a reified list using a Schwartzian transform |
19:05 | |
kudo/nom: def5262 | lizmat++ | src/core/Any-iterable-methods.pm: Any.sort(&by) uses R:I.MERGESORT-REIFIED-LIST-AS - no need to set up indexes (MERGESORT-REIFIED-LIST-AS does that) - otherwise same algorithm as MERGESORT-REIFIED-LIST - about 40% faster for a 100-element list |
|||
kudo/nom: 3a1fb3a | lizmat++ | src/vm/moar/Perl6/Ops.nqp: RIP nqp::p6sort Thanks for all the hard work! |
19:06 | ||
notviki | lizmat: it was purposefully deleted: github.com/perl6/roast/commit/3baa...d1f30dda15 | 19:40 | |
And I .tell'ed jnthn to review and OK that change. | 19:41 | ||
And if it is, we'll just stick a dummy file in place, I guess.... to make make spectest quiet about missing files :\ | 19:42 | ||
(we can't just remove it from t/spectest.data, because it's in 6.c-errata) | 19:43 | ||
samcv | uhm is this a bug rt.perl.org/Ticket/Display.html?id=117683 | 20:11 | |
yoleaux2 | 18:20Z <jnthn> samcv: About test-gen.p6 it was iirc written before some fairly major changes (including the Great List Refactor), and it's entirely possible the generator script is busted as a result. It was run against UCD 8.0 originally, so certainly produced passing tests against that. So I suggest "use UCD 8.0 and fix it to prdouce the same output first" would be one way to go. | ||
18:21Z <jnthn> samcv: Will try and catch up with your MoarVM PR and other Unicode bits that may want my input soon too; getting some much-needed rest these last days. :-) | |||
18:53Z <lizmat> samcv: did you forget to push Missing test file: t/spec/S15-nfg/mass-chars.t ? | |||
samcv | lizmat, i removed it | ||
.tell lizmat it has been removed see my explanation here: github.com/perl6/roast/commit/3baa...d1f30dda15 | 20:12 | ||
yoleaux2 | samcv: I'll pass your message to lizmat. | ||
samcv | and all the fudged tests we were failing from masschars.t, the reason was just the script generating them making bad assumptions. so using the GraphemeBreakTest, Unicode actually provides HOW graphemes are broken up. instead of us trying to do it ourselves, and so the fudged tests in GraphemeBreakTest are actually correct (aka SHOULD be being passed but don't) instead of tests which shouldn't pass, but the script generated them and they were bad | 20:16 | |
notviki, i can always write something new for masschars.t if you want, if it becomes needed, though. or maybe write something to test how we further break up the characters (instead of just checking .chars). going more deeply into the GraphemeBreakTest and checking each grapheme boundary instead of our total grapheme count | 20:21 | ||
dogbert2 | samcv: shouldn't t/spec/S15-nfg/mass-chars.t be removed from 'spectest.data' as well? | 20:22 | |
samcv | no, becuase it's still in roast | ||
for 6.c-errata | 20:23 | ||
dogbert2 | so the 'Missing test file: t/spec/S15-nfg/mass-chars.t' when running 'make spectest' is ok then? | ||
samcv | yes :) | ||
read my commit here github.com/perl6/roast/commit/3baa...d1f30dda15 | 20:24 | ||
and because the graphemebreaktest actually tells US how they break, instead of us creating the number of characters it 'should be' using our own rules on the fly on test creation | 20:25 | ||
which is not a very forward compatible way to do it | |||
well and kind of bad regardless. since we need to take a lot more into account than one unicode property, plus the rules could change in the future | 20:26 | ||
even if we did check all the rules _now_ on test creation, they could change and then we'd have to rewrite the generator to create the write 'answers' much better to rely on unicode's answers instead | |||
dalek | ast: de2fe04 | (Zoffix Znet)++ | S15-nfg/mass-chars.t: Silence unwated missing file warnings |
20:28 | |
samcv | notviki++ | ||
dalek | kudo/nom: 1e54371 | lizmat++ | src/core/List.pm: Add List.sort(&by) candidate On reified lists, it should not have to do a copy of the list being sorted. Gives about a 5% gain on reified List/Array. |
||
dogbert2 | Zoff, eh I meant notviki++ to the rescue :) | 20:31 | |
dalek | kudo/nom: a7bf1ea | lizmat++ | src/core/List.pm: Oops, R:I.MRLA is the one that doesn't need cloning R:I.MRLW does need a clone to work with, although no spectest failed without the clone, oddly enough. |
20:37 | |
samcv | somehow I missed seeing this ticket. thanks notviki github.com/perl6/atom-language-perl6/issues/29 | 20:41 | |
fixing now | |||
i've got a good workflow going for the highlighter, wrote a unescape.p6 that takes stdin and will unescape and print out the cson. copy the text: xclip -o | unescape.p6 | xclip | 20:43 | ||
and then it's on my clipboard unescaped :) | |||
then have one to re-escape it once i'm done getting it working | |||
notviki notices TimToady got a fancy hostmask for Christmas | 20:48 | ||
notviki leaves to nurse a burn out | 22:05 | ||
lizmat wonders whether she should mention www.reddit.com/r/perl/comments/5jk...h=b90d1568 in the P6W | 22:19 | ||
yoleaux2 | 20:12Z <samcv> lizmat: it has been removed see my explanation here: github.com/perl6/roast/commit/3baa...d1f30dda15 | ||
samcv | wow that's harsh | ||
that reddit | |||
lizmat decided against it | 22:21 | ||
timotimo | there was some extra text to it or something? | 22:26 | |
lizmat | timotimo: yes, there was, but that was removed by the admins | 22:43 | |
but the title is still visible, and says enough, really | |||
and another Perl 6 Weekly hits the Net: p6weekly.wordpress.com/2016/12/26/...s-the-end/ | 22:48 | ||
notviki | lizmat++ good weekly. | 22:56 | |
lizmat: FWIW MasterDuke's 10% improvement is *on top* of the 40% improvement to perl6 -ne '' he did: github.com/rakudo/rakudo/pull/960 | |||
lizmat | notviki: added, thanks! | 22:58 | |
notviki | lizmat++ | 22:59 | |
samcv | arghhhhhhh | ||
0xF.uniprop-bool('space') #> true; 0xF.uniprop-bool('White_Space') #> False | 23:00 | ||
goddamit space property. always flipping which is broken |