01:38
colomon joined
02:21
colomon joined
02:46
geekosaur joined
|
|||
[Tux] | test 21.861 | 09:01 | |
test-t 12.330 | |||
csv-parser 25.651 | |||
09:15
RabidGravy joined
|
|||
masak | slightly better than yesterday | 10:09 | |
lizmat | yeah, but without any significant changes, means it is within noise? | 10:56 | |
masak | probably | 11:19 | |
dalek | kudo/nom: 1d12563 | lizmat++ | src/core/operators.pm: Fix for RT #127777 By basically giving it the same treatment as with combinations() |
11:32 | |
lizmat | m: say "z" before "aa" # shouldn't this be True ?? | 11:51 | |
camelia | rakudo-moar 1d1256: OUTPUT«False» | ||
lizmat | masak: ^^^ opinion ? | 11:53 | |
timotimo | i think even though we have the magical behavior in the .. operator, i'd prefer the before/after operators to behave like lt and gt in general | 11:56 | |
masak | lizmat: no, I think you're confusing the order postfix:<++> and .succ produces, with lexical order | 11:59 | |
I'd much prefer `before` to produce lexical order, just like `lt` does | |||
lizmat | then we have a great thinko in Range.iterator for strings :-) | 12:00 | |
lizmat looks at fixing this (instigated by RT #127753 | 12:04 | ||
timotimo | ah, so that's what busted the order of ranges | 12:10 | |
lizmat | yeah... | ||
it's been like that for a while :-( | |||
timotimo | seems like a big oversight to not have that tested by roast :S | ||
lizmat | m: .print for ":" .. "?" # should this work at all ??? | 12:11 | |
camelia | rakudo-moar 1d1256: OUTPUT«:;<=>?» | ||
lizmat | afaik, we only define .succ for a certain set of magical chars | 12:12 | |
(and .pred) | |||
perhaps we should disallow and ranges in which the endpoints are not within the set of magical chars ? | 12:13 | ||
*any | |||
timotimo | no, i want ranges like that | 12:16 | |
dalek | p: 28b7689 | (Pawel Murias)++ | src/vm/js/nqp-runtime/deserialization.js: [js] Remove dead code. |
12:17 | |
p: afa5dbf | (Pawel Murias)++ | src/vm/js/nqp-runtime/deserialization.js: [js] Remove code that support an old serialization version format. |
|||
p: bb7b097 | (Pawel Murias)++ | src/vm/js/ (2 files): [js] Implement nqp::{shift,unshift}_{s,i}. Remove legacy raw array support. |
|||
p: 8f2c3db | (Pawel Murias)++ | t/nqp/59-nqpop.t: Test nqp::{unshift, shift}_{s,i}. |
|||
lizmat | timotimo: ok, so if both endpoints are in a magical range, we use the .succ magic | 12:18 | |
else, we can only have 1 char as endpoints, and then we do this based on ord() | 12:19 | ||
timotimo | i'd expect that | 12:20 | |
dalek | p: a58284d | (Pawel Murias)++ | src/vm/js/nqp-runtime/ (4 files): [js] Run fixjsstyle. |
||
timotimo | an oversight it may be to haven't tested this, but it's also surprising nobody stumbled upon the bug in such a long time | 12:23 | |
lizmat | yeah... :-( | 12:26 | |
the .succ / .pred logic is heavily embedded in Str | |||
it would have to be moved to e.g. Rakudo::Internals to make it accesible in Range | 12:27 | ||
timotimo imagines a tightly-packed yarnball | |||
lizmat | yeah... | ||
sortiz "long time" now three months! | |||
13:20
colomon joined
|
|||
lizmat goes cycling while contemplating a solution that is both correct and order of magnitudes faster :-) | 13:21 | ||
13:24
Skarsnik joined
13:47
skids joined
|
|||
TimToady | .. turns into ... semantics when evaluated as a list, so you're proably better off optimizing ... for known endpoints | 15:48 | |
in particular, magical ranges come into play *only* if the two strings are of unequal length (or if the right endpoint is *) | 16:03 | ||
so 「.print for ":" .. "?"」 has nothing whatsoever to do with Str.succ, only Int.succ | 16:05 | ||
and hence it has nothing to do with magical ranges either | 16:06 | ||
timotimo | ah, of course | 16:41 | |
19:30
FROGGS joined
19:40
geekosaur joined
|
|||
dalek | kudo/nom: e7a09e7 | lizmat++ | src/core/Bool.pm: Make sure Bool can also .enums like any other Enum The Bool refactor seems to have missed this particular method. Spotted while investigating RT #127789 |
20:15 | |
lizmat | m: say "9".succ # sorta expected Str "00" instead of Int 10 ?? | 20:19 | |
camelia | rakudo-moar 1d1256: OUTPUT«10» | ||
lizmat | m: dd "9".succ # hmmm... it's a Str after all ?? | 20:22 | |
camelia | rakudo-moar 1d1256: OUTPUT«"10"» | ||
lizmat | ah, I finally grok the reason for the carrydigit hash | 20:25 | |
21:01
colomon joined
|
|||
nine | lizmat: oh, indeed, I didn't even know .enums. Thanks for fixing :) | 21:25 | |
lizmat | yw :-) | 21:26 | |
masak | m: say "silly-image-of-cat-9.jpg".succ | 21:28 | |
camelia | rakudo-moar e7a09e: OUTPUT«silly-image-of-cat-10.jpg» | ||
masak | lizmat: that's probably why "9".succ is "10" | ||
lizmat | yeah, I figured it out :-) | 21:30 | |
masak | :) | ||
dalek | kudo/nom: ccc38a0 | lizmat++ | / (2 files): Refactor the templating logic for native arrays Before using it in other places. |
21:34 | |
22:04
geekosaur joined
|
|||
dalek | kudo/nom: 47d21b8 | lizmat++ | tools/build/makeMAGIC_INC_DEC.pl6: WIP on generating hashes for .succ/.pred |
22:25 | |
lizmat | good night, #p6dev! | 22:26 | |
timotimo | gnite lizmat! | ||
lizmat | .oO( it's going to be a short night ) |
22:27 |