TimToady | unless constant accepts that in the name slot, which I tend to doubt | 00:00 | |
MasterDuke_ | thanks, that helps. was looking at rt.perl.org/Public/Bug/Display.html?id=113954 and trying to figure which parameters needed to be checked for spaces | 00:03 | |
samcv | TimToady, yeah i assume all his are Artistic-2.0, i think he might have said that in channel actually | ||
01:49
ilbot3 joined
04:14
geekosaur joined
06:34
domidumont joined
06:39
domidumont joined
|
|||
samcv | ok guys. so now i need to think about the indexic op. and how to return the information about how long the haystack match it found was. | 07:35 | |
not sure what our choices are. but we need to return the index of the located needle's beginning in haystack. | 07:36 | ||
we also need to convey how many haystack chars expanded or just return back the index of the last cp of the match | |||
but more important is do i have it return an array object? or what? how do we handle returning multiple values i know uhm | 07:37 | ||
i know with radix we return an array | 07:38 | ||
where out[0] is the numerical value of the radix number, out[1] is the number of digits converted | |||
and out[2] is the offset after consumisg digits | |||
not sure why it returns both number of cp's converted and the offset afterward, is that for convenience? | 07:40 | ||
or is there something i'm not thinking of and why we do that. MasterDuke_ do you know | |||
github.com/perl6/nqp/blob/master/d...down#radix or can out[2] be always determined from out[1] + the offset it was handed off | |||
but i could do it exactly the same way as radix. and then it would be consistent as well | 07:41 | ||
lizmat | samcv: I think that would be a good start :-) | 07:44 | |
07:57
BinGOs joined
08:02
timotimo joined
|
|||
samcv | lizmat, :) | 08:21 | |
testing now on the eqatic op... cause i know if i mess it up i can actually launch up nqp and rakudo still :) and not have it die horribly with crazy errors | 08:22 | ||
my @list := nqp::eqatic('ļ¬ļ¬', 'stst', 0); say(@list[0] ~ " " ~ @list[1]) | |||
get back "1 2" woo | |||
1st is boolean on if it matched or not, second is the number of cp's the haystack expanded in the check | |||
nice. | |||
i dream of a day where this works :) | |||
m: say 'ļ¬a' ~~ m:i/st/ | 08:23 | ||
camelia | ļ½¢ļ¬aļ½£ | ||
samcv | getting closer every month | ||
lizmat | samcv: so it's returning an array of opaque Ints ? rather than a list_i ? | 08:25 | |
samcv | uh. it's doing the exact same that uhm | ||
radix does | |||
lizmat | ah, this is nqp, not Perl 6 | ||
samcv | well it'd still work in perl6 just like radix does | ||
but it's box int | 08:26 | ||
result = MVM_repr_alloc_init(tc, MVM_hll_current(tc)->slurpy_array_type); | |||
does that depend on the HLL in effect nqp vs perl6:? | |||
just copy pasting from radix heh | |||
lizmat | samcv: please ignore me, should have looked up nqp::radix more closely | 08:27 | |
afk& | |||
samcv | np | ||
timotimo | samcv: in theory we could have an nqp:: op with multiple w registers | 09:27 | |
samcv | hm | ||
timotimo | i just don't know how the rest of moar and nqp and rakudo will handle that :D | ||
samcv | k | ||
well help me with something | |||
lemme push this nqp branch | |||
timotimo | uh oh | ||
samcv | you know more about this.. | 09:28 | |
i am flying basically blind i have no clue how this works. or uh | |||
github.com/samcv/nqp/commit/3090cd...9e3824267b | |||
so i made a $o0 register. or something like that. holds objects. since now nqp::eqatic returns an object | 09:29 | ||
and i want to shift the first element of this array into the integer regestir that the eqatic op *used* to put the result directly into it | |||
how do | |||
timotimo | wow, my face when i restart the computer and chrome looks totally different | 09:30 | |
samcv | ok so it pushes to @ins, this op() thingy | ||
and the left thing is always the op | |||
the 2nd arg is the register the **result** is put into? yes or no | |||
i'm just guessing here. as i am very confused | 09:31 | ||
timotimo | from a register standpoint you are essentially "reading a register" here | ||
even though you are mutating the object that is pointed at by the register | |||
samcv | uhm so the eqaticop takes what's in $s0 | ||
and puts result into $i0 | |||
yes or no | 09:32 | ||
and %!reg<tgt> holds idk. and %!reg<pos. holds the haystack start position sent to eqatic? | |||
err or tgt is the string | |||
timotimo | um, i don't know? what does your moarvm patch look like? | ||
samcv | eqatic('string', 'string2', %!reg<pos) | ||
it returns an array | |||
and shift works on it | 09:33 | ||
same type of array radix returns | |||
with two elements | |||
timotimo | oh, now i think i see how your stuff looks | ||
samcv | 1st element is the normal return value of eqatic | ||
timotimo | well, if you're changing the type of register for the first one in eqatic, you'll need a deprecation cycle or something | 09:34 | |
samcv | so i need to get that first item in the array into where the return value used to go | ||
it's only used in that one | |||
place in nqp | |||
timotimo | OK | ||
samcv | it was only added in like 1 month ago | ||
before it was inaccessible completely inside nqp or rakudo | |||
timotimo | i think you have shift reversed | 09:35 | |
samcv | yeah idk what i'm doing | ||
timotimo | shift_o w(obj) r(obj) | ||
i'll have to afk in a little bit | |||
samcv | ok | ||
ok so look at the register to see how it lines up with that | 09:36 | ||
timotimo | i don't see why we wouldn't use a list_i for this task, though | ||
samcv | i don't either | ||
timotimo | if you're already unpacking it in the compiler anyway | ||
samcv | i've never worked with MVMObjects that are lists in Moar before | ||
timotimo | i.e. if we don't expect any user to ever write nqp::eqatic in their code | ||
no problem, you just use the functions in reprconv.h | |||
samcv | also what's bad about a list_i. you can use list_i normally right. well as list_i's i've used them before | 09:37 | |
timotimo | yeag | 09:38 | |
you konw how to create one inside moarvm? | |||
samcv | nope | ||
timotimo | you'll want the BOOTIntArray | ||
grep for other uses of that and you'll be able to steal code i bet | 09:39 | ||
BBL | |||
don't forget you'll have to change the type of the register in the oplist and regenerate all the files from that | 09:51 | ||
09:55
AlexDaniel joined
|
|||
samcv | i did that already yeah | 09:57 | |
timotimo | OK | 09:58 | |
jnthn | samcv: Yeah, I'm happy with AL2; I really need to get around to going through all of my modules and doing housekeeping | 11:38 | |
(Making sure they all have license, META.info => META6.json in a few, etc.) | |||
13:14
Ven joined
15:32
ggoebel joined
16:08
domidumont joined
|
|||
samcv | kk thx jnthn | 18:01 | |
18:47
Ven joined
|
|||
samcv | jnthn, not sure what's going on with this license github.com/jnthn/json-path | 18:49 | |
it says MIT (X11) but the license text is clearly the MIT one. since the X11 license contains a clause about not advertising products and mentioning Xorg foundation in the promotion of it or something | 18:50 | ||
so not relevant, and text is the standard MIT | |||
jnthn | I've no idea :) | 19:13 | |
I co-authored that module with masak long ago and he added the LICENSE file, and we did it as a port of the Perl 5 module | |||
And seem to have inehrited the license from that | 19:14 | ||
Thanks for all the PRs :) | |||
samcv++ | |||
samcv | :-) | 19:15 | |
well the text of the license is the MIT license | |||
it says MIT license. but in one place says MIT (X11) | 19:16 | ||
and X11 is the MIT license but an extra clause about advertising related to Xorg foundation. which clearly is ridiculous applied to this project | |||
so i'm guessing it was unintentional | |||
geekosaur | I would consider that a historical thinko | ||
the X11 license used to be the best known example of the MIT license | |||
samcv | *an* MIT license? cuase they added a cluase right? | 19:17 | |
argh can't type | |||
jnthn | Feels like historical accident, yes | 19:19 | |
geekosaur | it's complicated >.> much of X11 was still under MIT license for a couple decades | ||
jnthn | If you're PRing, I guess could remove the X11 mention given that isn't what the actual license is | ||
samcv | yeah | 19:21 | |
i checked the full text of it against MIT to make sure no differences just to be extra sure | |||
basically like writing `BSD 2 clause (3 clause) license` | 19:23 | ||
which is confusing | |||
19:56
geekosaur joined
20:17
zakharyas joined
|