| Time |
S |
Nick |
Message |
| 00:03 |
|
|
bacek joined #parrot |
| 00:38 |
|
|
Zak joined #parrot |
| 00:39 |
|
|
mrsaturns12 joined #parrot |
| 00:41 |
|
|
perlguiman joined #parrot |
| 00:44 |
|
|
tendas joined #parrot |
| 00:45 |
|
|
tendas left #parrot |
| 00:47 |
|
dalek |
decnum-dynpmcs: r123 | darbelo++ | trunk/aux/decTest/src/pct/actions.pm: |
| 00:47 |
|
dalek |
decnum-dynpmcs: [decTest] Tweak the past so that each test get's it's own sub. Output is now |
| 00:47 |
|
dalek |
decnum-dynpmcs: equivalent to what the perl script was generating. Some ruddimentary benchmarks |
| 00:47 |
|
dalek |
decnum-dynpmcs: show this to be marginally less slow than before. I have no clues as to why, |
| 00:47 |
|
dalek |
decnum-dynpmcs: however. |
| 00:47 |
|
purl |
Stop complicating things. |
| 00:47 |
|
dalek |
decnum-dynpmcs: review: http://code.google.com/p/decnu[…]urce/detail?r=123 |
| 00:58 |
|
|
perlguiman left #parrot |
| 00:58 |
|
darbelo |
Great. Now the bot is giving me coding advice. |
| 00:59 |
|
|
bacek joined #parrot |
| 00:59 |
|
darbelo |
OTOH it seems like it was good advice. purl++, I guess. |
| 01:00 |
|
kid51 |
purl, nice bot |
| 01:00 |
|
purl |
kid51: i'm not following you... |
| 01:00 |
|
kid51 |
purl++ |
| 01:18 |
|
|
hiroyuk__ joined #parrot |
| 02:10 |
|
* kid51 |
must sleep |
| 02:10 |
|
purl |
$kid51->sleep(8 * 3600); |
| 02:11 |
|
chromatic |
http://use.perl.org/~missingth[…]int/journal/39359 |
| 02:25 |
|
GeJ |
Just read it. When $life permits (hopefully in a couple of weeks), I'd like to do the same and resume my Parrot activity. |
| 02:26 |
|
davidfetter |
$life->alloc(8 * interval '1 hour') |
| 02:27 |
|
davidfetter |
dang! |
| 02:42 |
|
|
janus joined #parrot |
| 02:46 |
|
|
dukeleto joined #parrot |
| 03:20 |
|
|
donaldh joined #parrot |
| 03:26 |
|
dalek |
parrot: r40296 | chromatic++ | trunk/config (2 files): |
| 03:26 |
|
dalek |
parrot: [config] Fixed splint temporary directory detection by adding a TEMPDIR |
| 03:26 |
|
dalek |
parrot: variable to the generated Makefile (fixes at least part of TT #859 ). |
| 03:26 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40296/ |
| 03:26 |
|
dalek |
TT #859 closed by chromatic++: 'make splint' needs graceful failure message |
| 03:41 |
|
|
Andy joined #parrot |
| 03:56 |
|
dalek |
close: r83 | Austin++ | trunk/src/parser/statements.pg: |
| 03:56 |
|
dalek |
close: Fixed syntax error in statements.pg |
| 03:56 |
|
dalek |
close: review: http://code.google.com/p/close/source/detail?r=83 |
| 04:13 |
|
|
abesapien joined #parrot |
| 04:26 |
|
|
tetragon joined #parrot |
| 04:29 |
|
|
dukeleto joined #parrot |
| 04:45 |
|
|
japhb joined #parrot |
| 06:01 |
|
dalek |
cardinal: 166f0ee | (Danius Michaelides)++ | (2 files): |
| 06:01 |
|
dalek |
cardinal: Implement reverse_each(). |
| 06:01 |
|
dalek |
cardinal: Signed-off-by: Ted Reed <ted.reed gmail.com> |
| 06:01 |
|
dalek |
cardinal: review: http://github.com/cardinal/car[…]72271df16f27398a1 |
| 06:01 |
|
dalek |
cardinal: d830bad | (Danius Michaelides)++ | (3 files): |
| 06:02 |
|
dalek |
cardinal: Implement Array#fetch() with test. |
| 06:02 |
|
dalek |
cardinal: Signed-off-by: Ted Reed <ted.reedgmail.com> |
| 06:02 |
|
dalek |
cardinal: review: http://github.com/cardinal/car[…]eb2cb9044e807aa45 |
| 06:03 |
|
|
uniejo joined #parrot |
| 06:44 |
|
|
desertm4x joined #parrot |
| 07:04 |
|
dalek |
parrot: r40297 | dukeleto++ | trunk/t/tools/parrot_debugger.t: |
| 07:04 |
|
dalek |
parrot: Add some tests for the basic parrot_debugger commands |
| 07:04 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40297/ |
| 07:21 |
|
|
donaldh joined #parrot |
| 07:31 |
|
|
iblechbot joined #parrot |
| 07:45 |
|
dalek |
parrot: r40298 | chromatic++ | trunk/src/pmc/continuation.pmc: |
| 07:45 |
|
dalek |
parrot: [PMC] Avoided dereferencing a potentially NULL pointer before checking for |
| 07:45 |
|
dalek |
parrot: NULLness in Continuation's caller() method (Coverity CID #409). |
| 07:45 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40298/ |
| 07:52 |
|
chromatic |
:526 |
| 07:52 |
|
dalek |
parrot: r40299 | chromatic++ | trunk/src/gc/alloc_register.c: |
| 07:52 |
|
dalek |
parrot: [GC] Moved a pointer dereference after the check for its NULLness (Coverity CID |
| 07:52 |
|
dalek |
parrot: #407). |
| 07:52 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40299/ |
| 07:53 |
|
|
Zak joined #parrot |
| 07:59 |
|
dalek |
parrot: r40300 | chromatic++ | trunk/src/pmc/parrotinterpreter.pmc: |
| 07:59 |
|
dalek |
parrot: [PMC] Moved a pointer dereference after the check for its NULLness in |
| 07:59 |
|
dalek |
parrot: ParrotInterpreter's get_pmc_keyed() vtable entry (Coverity CID #411). |
| 07:59 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40300/ |
| 08:14 |
|
dalek |
parrot: r40301 | chromatic++ | trunk/src/pmc/exception.pmc: |
| 08:14 |
|
dalek |
parrot: [PMC] Rearranged code in Exception's destroy() vtable entry to check pointer |
| 08:14 |
|
dalek |
parrot: NULLness before dereferencing it (Coverity CID #408). |
| 08:14 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40301/ |
| 08:17 |
|
dalek |
parrot: r40302 | chromatic++ | trunk/compilers/imcc/pbc.c: |
| 08:17 |
|
dalek |
parrot: [IMCC] Removed a NULL pointer check after a successful dereference of that |
| 08:17 |
|
dalek |
parrot: pointer. If there's no code in the interpreter at the point of adding |
| 08:17 |
|
dalek |
parrot: annotations, something is far, far wronger than e_pbc_emit() can handle |
| 08:17 |
|
dalek |
parrot: (Coverity CID #410). |
| 08:17 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40302/ |
| 08:35 |
|
|
mokurai left #parrot |
| 08:37 |
|
dalek |
TT #879 created by jessevdam++: isa does not take (correctly) namespace, pmcarray,stringarray |
| 08:38 |
|
dalek |
parrot: r40303 | chromatic++ | trunk/src/oo.c: |
| 08:38 |
|
dalek |
parrot: [OO] Freed object attributes when created directly in Parrot_oo_clone_object(); |
| 08:38 |
|
dalek |
parrot: this was a memory leak (Coverity CID #357). |
| 08:38 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40303/ |
| 09:08 |
|
dalek |
TT #880 created by jessevdam++: class associated to namespace ["parrot";"Class"] gets overridden |
| 09:23 |
|
|
einstein joined #parrot |
| 09:27 |
|
mikehh |
post config tests FAILing - see TT#878 |
| 10:00 |
|
mikehh |
codetest FAIL, post config FAIL (see TT #878 ), all others PASS (pre config, smolder, nqp_test fulltest) at r40303 - Ubuntu 9.04 amd64 |
| 10:05 |
|
mj41 |
Hi. ftp://ftp.parrot.org/pub/parrot/releases/devel/1.4.0/ ... failed to change directory |
| 10:06 |
|
moritz |
mj41: 1.4.0 is not a devel/ release |
| 10:07 |
|
mj41 |
oh, .. but link is on http://www.parrot.org/news/2009/Parrot-1.4.0 |
| 10:09 |
|
Infinoid |
good catch |
| 10:09 |
|
moritz |
mj41: thanks for pointing that out, I just fixed the link |
| 10:11 |
|
|
bacek joined #parrot |
| 10:13 |
|
|
patspam joined #parrot |
| 10:38 |
|
|
wayland76 joined #parrot |
| 10:42 |
|
|
HG` joined #parrot |
| 10:45 |
|
|
ttbot joined #parrot |
| 10:46 |
|
|
jimmy joined #parrot |
| 10:52 |
|
bacek |
o hai |
| 10:53 |
|
nopaste |
"mikehh" at 90.209.128.139 pasted "codetest failures at r40303" (61 lines) at http://nopaste.snit.ch/17414 |
| 10:53 |
|
nopaste |
"mike" at 90.209.128.139 pasted "PATCH for codetest failures at r40303" (52 lines) at http://nopaste.snit.ch/17415 |
| 10:54 |
|
mikehh |
orry that should have been mikehh in the patch - codetest PASSes after the PATCH |
| 10:54 |
|
mikehh |
I kmywekeard is skipping occaisionally |
| 10:55 |
|
mikehh |
that was I think my wireless keyboard is skipping keypresses on occaision |
| 10:56 |
|
wayland76 |
Looks like it. You need to get an Avant Stellar keyboard |
| 10:56 |
|
wayland76 |
Or maybe an Optimus Maximus :) |
| 10:58 |
|
mikehh |
I got a Micro$oft Wireless Keyboard and Mouse - which actually fits my fingers properly - less typing mistakes in general - BUT ..... |
| 10:58 |
|
dalek |
parrot: r40304 | bacek++ | trunk (2 files): |
| 10:58 |
|
dalek |
parrot: [cage] Fix codetest failures. Patch courtesy mikehh++ |
| 10:58 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40304/ |
| 10:59 |
|
mikehh |
This is my forth keyboard after one of my grandsons spilt *Coke* on my old favourite |
| 11:00 |
|
mikehh |
4th and that was Coca-Cola |
| 11:00 |
|
wayland76 |
I think an Avant Stellar could be easily cleaned |
| 11:01 |
|
wayland76 |
(Btw, if you haven't seen the pricetag on either keyboard I mentioned, you haven't gotten the tease yet :) ) |
| 11:01 |
|
wayland76 |
(Even though I do actually have an Avant Stellar :) ) |
| 11:01 |
|
mikehh |
:-} |
| 11:03 |
|
wayland76 |
For those who can't be bothered looking, the Avant Stellar is relatively cheap at around $189 |
| 11:03 |
|
wayland76 |
:) |
| 11:03 |
|
wayland76 |
Whereas the Optimus Maximus came out at, IIRC, $1800, and is still over $1000 |
| 11:04 |
|
wayland76 |
But don't knock it until you've seen it -- it's *extremely* cool. If it were say $300, I'd probably have one already :) |
| 11:07 |
|
wayland76 |
Maybe even at $400 :) |
| 11:17 |
|
mikehh |
isn't that the one that is completely customizable with 100+ LED key layouts or something |
| 11:19 |
|
mikehh |
I think it was a russian design or somethhing like that |
| 11:20 |
|
wayland76 |
mikehh: That's the one :) |
| 11:20 |
|
wayland76 |
The legendary keyboard -- I don't think one's ever been seen in the wild, just conspiracy theorist photos :) |
| 11:21 |
|
wayland76 |
(afk, sorry &) |
| 11:21 |
|
|
donaldh joined #parrot |
| 11:44 |
|
mj41 |
TapTinder and ttbot are back online ... http://tt.ro.vutbr.cz/buildsta[…]r-Parrot/rp-trunk |
| 11:45 |
|
|
flh joined #parrot |
| 11:53 |
|
|
ruoso joined #parrot |
| 11:56 |
|
|
skv joined #parrot |
| 12:28 |
|
|
whiteknight joined #parrot |
| 12:29 |
|
|
tetragon joined #parrot |
| 12:31 |
|
whiteknight |
good morning #parrot |
| 12:32 |
|
whiteknight |
kid51: ping |
| 12:35 |
|
dalek |
TT #881 created by cono++: Due to locale settings, HEAD revision of svn was 0. |
| 12:37 |
|
mikehh |
post-config (buildtools_tests) FAIL (see TT #878 ) - All Others PASS (pre-config, smolder, nqp_test, fulltest) at r40304 - Ubuntu 9.04 amd64 |
| 12:48 |
|
|
jimmy joined #parrot |
| 12:55 |
|
|
whoppix joined #parrot |
| 13:02 |
|
|
Whiteknight joined #parrot |
| 13:02 |
|
dalek |
parrot: r40305 | whiteknight++ | trunk/src/ops/ops.num: |
| 13:02 |
|
dalek |
parrot: [ops] renumber the ops again because last time i apparently didn't do it correctly. Thanks kid51++ and mikehh++ for the report |
| 13:02 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40305/ |
| 13:06 |
|
dalek |
TT #878 closed by whiteknight++: Recent commits break buildtools_tests |
| 13:08 |
|
|
desertm4x joined #parrot |
| 13:14 |
|
jimmy |
Whiteknight: hello? |
| 13:14 |
|
purl |
Raise your hand in the back if you can't hear me. |
| 13:22 |
|
|
masak joined #parrot |
| 13:24 |
|
jimmy |
Hello masak |
| 13:24 |
|
masak |
你好, jimmy |
| 13:26 |
|
jimmy |
one question |
| 13:26 |
|
purl |
i guess one question is where to put these things and how to divide it up |
| 13:26 |
|
masak |
purl: no, one question is <reply> |
| 13:26 |
|
purl |
okay, masak. |
| 13:27 |
|
jimmy |
where is VTABLE_get_pointer defined? |
| 13:27 |
|
Coke |
depends on the PMC. |
| 13:27 |
|
jimmy |
It's uesd in core.ops |
| 13:28 |
|
Coke |
if it's a core PMC, check in src/pmc/<foo>.pmc - |
| 13:28 |
|
Coke |
every PMC has its own vtable. |
| 13:28 |
|
jimmy |
and core.ops ? |
| 13:28 |
|
jonathan |
jimmy: Also note that VTABLE_get_pointer is a macro. |
| 13:29 |
|
jonathan |
VTABLE_get_pointer(interp, pmc) becomes something like pmc->vtable->get_pointer(interp, pmc) or similar. |
| 13:29 |
|
Coke |
core.ops contains opcodes. those opcodes call C. (like the macro you mention). that macro in turn is taking a PMC, looking up its vtable, and then looking up a function pointer. |
| 13:29 |
|
jimmy |
void * ptr = VTABLE_get_pointer(interp, $2); ? |
| 13:29 |
|
Coke |
those functions are defined in src/pmc/*.pmc |
| 13:29 |
|
jimmy |
it's used in experimental.ops |
| 13:30 |
|
Coke |
jimmy: ... yes? |
| 13:30 |
|
Coke |
the opcodes are defined in src/ops/*.ops |
| 13:30 |
|
jimmy |
it's not pmc |
| 13:30 |
|
Coke |
opcodes work on pmcs. |
| 13:30 |
|
Coke |
when you do $P1 = getclass $P2, those are PMCs - the opcodes may be invoking vtable entries on those PMCs. |
| 13:31 |
|
Coke |
so, if you know which PMC type you're working on, you can look back into the PMC definitions to see the C implementation of that vtable. |
| 13:32 |
|
jimmy |
inline op get_labe used it. and I want to know where is it defined. from https://trac.parrot.org/parrot/changeset/40295 |
| 13:32 |
|
Coke |
jimmy: and I'm telling you. =-) |
| 13:32 |
|
Coke |
every PMC is different. |
| 13:32 |
|
Coke |
each PMC type has VTABLE - a list of function pointers. |
| 13:32 |
|
Coke |
you're invoking the get_pointer vtable - so you have to figure out which PMC you're working with, and look at the source for that PMC. |
| 13:33 |
|
jimmy |
thanks, what type does it return? |
| 13:33 |
|
jimmy |
ptr？ |
| 13:33 |
|
NotFound |
The macros are defined in include/parrot/vtable.h |
| 13:34 |
|
Coke |
those are generated from src/vtable.tbl - looks like get_pointer returns a void *. |
| 13:36 |
|
jimmy |
thanks, coke, jonathan and NotFound |
| 13:36 |
|
NotFound |
jessevdam? |
| 13:37 |
|
jimmy |
nopaste? |
| 13:37 |
|
purl |
nopaste is at http://nopaste.snit.ch/ (ask TonyC for new channels) or http://poundperl.pastebin.com/ or http://paste.scsys.co.uk/ or App::Nopaste or tools/dev/nopaste.pl or at http://www.extpaste.com/ or http://paste.scsys.co.uk (for #catalyst, #dbix-class, #moose and others) or http://gist.github.com/ or paste or gtfo |
| 13:38 |
|
nopaste |
"jimmy" at 119.122.35.120 pasted "Is this patch right?" (14 lines) at http://nopaste.snit.ch/17416 |
| 13:40 |
|
Coke |
jimmy: I am not a C programmer, but that looks right. |
| 13:40 |
|
NotFound |
jimmy: What's the purpose of that change? Avoid a variable? |
| 13:41 |
|
Coke |
yah, it seems to just take advantage of our existing macros. |
| 13:41 |
|
jimmy |
PTR2INTVAL is a dynamic marco |
| 13:41 |
|
PerlJam |
Does PTR2INTVAL do something different than the cast? |
| 13:41 |
|
jimmy |
not a static marco |
| 13:42 |
|
Coke |
I don't understand the difference between static and dynamic here. |
| 13:42 |
|
NotFound |
The macro usage may be right, dropping the local var might trigger warnings. |
| 13:42 |
|
Coke |
(it's all done at C compile time.) |
| 13:42 |
|
* PerlJam |
fails in resisting the uge to say "polo!" |
| 13:42 |
|
Coke |
NotFound: what warnings? |
| 13:42 |
|
purl |
somebody said warnings was just a magical bitmask, iirc |
| 13:43 |
|
NotFound |
Coke: the warnings that are avoided by using a local variable ;) |
| 13:43 |
|
|
HG` joined #parrot |
| 13:43 |
|
jimmy |
as I refer to all the codes, I think this patch is the right use. |
| 13:43 |
|
NotFound |
gcc warns about direct conversions from values returned from functions, not sure about in what circumnstances. |
| 13:44 |
|
|
pjcj joined #parrot |
| 13:45 |
|
jimmy |
NotFound: Wait a moment |
| 13:46 |
|
dalek |
TT #882 created by flh++: SUPER() not handled correctly for DynPMCs |
| 13:51 |
|
|
mikehh_ joined #parrot |
| 13:53 |
|
|
MoC joined #parrot |
| 13:54 |
|
nopaste |
"jimmy" at 119.122.35.120 pasted "PTR2INTVAL should use the right macro, not (INTVAL), isn't right?" (13 lines) at http://nopaste.snit.ch/17417 |
| 13:56 |
|
jimmy |
sorry, wrong description |
| 14:00 |
|
NotFound |
jimmy: yes, using the macro is better. That way we can (maybe someday) replace it with one that check limits in debug builds. |
| 14:01 |
|
NotFound |
Finding and replacing C casts is a lot more work. |
| 14:02 |
|
NotFound |
Is there some test for that experimental ops? |
| 14:03 |
|
|
jsut joined #parrot |
| 14:03 |
|
Coke |
I don't think trap was tested. no clue on whiteknight's new ones. |
| 14:03 |
|
jimmy |
for get_label, it is the same sa get_addr |
| 14:03 |
|
jimmy |
s/sa/as/ |
| 14:04 |
|
jimmy |
because these two ops are the same code |
| 14:04 |
|
NotFound |
jimmy: I suppose that that is the intention, but is tested? |
| 14:05 |
|
NotFound |
There are very few usages of get_addr in t/ BTW |
| 14:06 |
|
jimmy |
yes, should ask Whitenight for more |
| 14:09 |
|
Coke |
perhaps that can be a condition of them moving out of experimental. =-) |
| 14:10 |
|
dalek |
parrot: r40306 | NotFound++ | trunk/src/ops/experimental.ops: |
| 14:10 |
|
dalek |
parrot: [cage] use conversion macro in get_label, jimmy++ |
| 14:10 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40306/ |
| 14:10 |
|
NotFound |
It may be a good idea to create a ticket for any experimental op, and mention that ticket in the code. |
| 14:11 |
|
jimmy |
I think there were more places to use PTR2INTVAL, not (INTVAL), should create a ticket to do that job? |
| 14:11 |
|
Coke |
jimmy: seems reasonable, sure. |
| 14:11 |
|
Coke |
I am bemused that PTR2INTVAL is defined in terms of INTVAL2PTR. |
| 14:12 |
|
jimmy |
yes |
| 14:12 |
|
NotFound |
But make clear on the ticket that must be done and tested case-by-case, not by search-and-replace. |
| 14:12 |
|
Coke |
NotFound: any reason you did that in two lines? |
| 14:12 |
|
Coke |
just to avoid the unnamed potential warnings? |
| 14:13 |
|
jimmy |
Coke: seems to be an artifice |
| 14:13 |
|
NotFound |
Coke: yes, and following the principle: don't fix if is not broken. |
| 14:14 |
|
Coke |
NotFound: but we're already changing that code, so that doesn't seem to apply. =-) |
| 14:14 |
|
Coke |
the single line version seems clearer to me. |
| 14:14 |
|
NotFound |
Coke: touché ;) |
| 14:14 |
|
Coke |
I'm just trying to understand the rationale for the modified change. |
| 14:15 |
|
NotFound |
Coke: sometimes we have fixes because of that warning, I try to avoid the posibility of walking in circles. |
| 14:15 |
|
Coke |
what warning? |
| 14:15 |
|
NotFound |
Don't remeber the literal wording, casting from function result or something like that. |
| 14:15 |
|
* Coke |
sees if he can generate it. |
| 14:16 |
|
mikehh |
All Tests PASS (pre/post-config, smolder, nqp_test, fulltest) at r40305 - Ubuntu 9.04 amd64 |
| 14:16 |
|
Coke |
make src/ops/core_ops.o |
| 14:16 |
|
Coke |
make: *** No rule to make target `include/parrot/stacks.h', needed by `src/ops/core_ops.o'. Stop. |
| 14:17 |
|
* Coke |
reconfigs. |
| 14:18 |
|
NotFound |
If someone know the exact conditions that trigger that warning and how to avoid it, I also like to avoid that unsueful local vars. |
| 14:18 |
|
NotFound |
In the meantime, having the build almost warning free is a good thing. |
| 14:19 |
|
jimmy |
Can anyone create a ticket to change more places to use PRT2INTVAL correctly, but not (INTVAL) ? |
| 14:20 |
|
Coke |
NotFound: src/ops/experimental.ops:75: warning: cast from function call of type 'void *' to non-matching type 'long int' |
| 14:20 |
|
Coke |
that's the warning. |
| 14:21 |
|
NotFound |
Yes, that is. |
| 14:21 |
|
Coke |
(to be clear, your version is NOT generating that warning.) |
| 14:21 |
|
Coke |
Danke. |
| 14:23 |
|
jimmy |
using the const is right for me. thanks NotFound |
| 14:24 |
|
NotFound |
The const might be helpful for some optimizer, and doesn't hurt anyway. |
| 14:24 |
|
jimmy |
yes. |
| 14:25 |
|
jimmy |
But who can create the ticket for more changes? |
| 14:31 |
|
Coke |
anyone with a trac id. |
| 14:31 |
|
Coke |
that is, you can create the ticket - that'd be fine. |
| 14:32 |
|
jimmy |
thank Whitenight for creating that ticket. |
| 14:40 |
|
dalek |
TT #883 created by jimmy++: Use the right PTR2INTVAL() marco, not (INTVAL) for type conversion |
| 14:41 |
|
dalek |
close: r84 | Austin++ | trunk/ (5 files): |
| 14:41 |
|
dalek |
close: Started refactoring namespaces |
| 14:41 |
|
dalek |
close: review: http://code.google.com/p/close/source/detail?r=84 |
| 14:47 |
|
dalek |
parrot: r40307 | NotFound++ | trunk/lib/Parrot/Revision.pm: |
| 14:47 |
|
dalek |
parrot: [config] avoid troubles with locales and svn messages, TT #881 , cono++ |
| 14:47 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40307/ |
| 14:56 |
|
|
Andy joined #parrot |
| 15:01 |
|
|
flh joined #parrot |
| 15:02 |
|
|
quek joined #parrot |
| 15:09 |
|
dalek |
TT #881 closed by NotFound++: Due to locale settings, HEAD revision of svn was 0. |
| 15:12 |
|
|
theory joined #parrot |
| 15:17 |
|
dalek |
TT #767 closed by NotFound++: imcc allows multiple .locals of the same name |
| 15:21 |
|
|
donaldh joined #parrot |
| 15:41 |
|
|
cono joined #parrot |
| 16:01 |
|
|
dukeleto joined #parrot |
| 16:08 |
|
|
mokurai joined #parrot |
| 16:26 |
|
|
Psyche^ joined #parrot |
| 16:36 |
|
|
Whiteknight joined #parrot |
| 16:37 |
|
cotto |
Andy++ for talking to the Coverity guys. |
| 16:37 |
|
Andy |
Thanks. |
| 16:37 |
|
Andy |
Guy, singular. As far as I know, he's the only one who works there. :-) |
| 16:39 |
|
cotto |
s/s// |
| 16:41 |
|
cotto |
I hadn't logged in in forever, but apparently my cookie was still valid. |
| 16:42 |
|
|
Zak joined #parrot |
| 16:44 |
|
|
jan joined #parrot |
| 16:57 |
|
dalek |
parrot: r40308 | NotFound++ | trunk (2 files): |
| 16:57 |
|
dalek |
parrot: [cage] fix badly named variable in generated code, TT #217 |
| 16:57 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40308/ |
| 16:58 |
|
dalek |
TT #217 closed by NotFound++: fix badly named variable in generated code |
| 17:01 |
|
|
darbelo joined #parrot |
| 17:06 |
|
|
theory joined #parrot |
| 17:08 |
|
Coke |
cotto, andy - iwbni if we had a CoverityTesting page on the wiki. |
| 17:08 |
|
Andy |
iwbni? |
| 17:08 |
|
purl |
iwbni is "it would be nice if" |
| 17:09 |
|
Andy |
Coke: ok |
| 17:09 |
|
Coke |
NotFound++ - I saw your commit, and thought that it looked familiar. =-) |
| 17:09 |
|
Coke |
Andy: just to reduce the bus number. Thanks! |
| 17:09 |
|
Andy |
Coke: Go ahead and make one. :-) |
| 17:09 |
|
Andy |
which wiki? |
| 17:09 |
|
purl |
which wiki is poe.perl.org built on? |
| 17:10 |
|
Andy |
And don't thank me for something I haven't done, or agreed to do. :-) |
| 17:10 |
|
Andy |
I'll think you're my wife. |
| 17:11 |
|
* cotto |
thinks of several ways to make Andy feel more awkward |
| 17:11 |
|
Coke |
trac.parrot.org, please. |
| 17:11 |
|
Andy |
you could be a boothbabe and hand me a flyer for a linux fund credit card. |
| 17:12 |
|
Andy |
Coke: I'm not sure I have an account there. |
| 17:12 |
|
Coke |
Andy: you can have one, it's ok! |
| 17:12 |
|
Andy |
ah, I do |
| 17:12 |
|
Coke |
(i would create it for you, but I don't think I can.) |
| 17:12 |
|
Andy |
I've never used Trac wiki |
| 17:12 |
|
Andy |
playing |
| 17:13 |
|
Coke |
if you just throw in some text, someone else can format it. |
| 17:13 |
|
Andy |
First, I have to find the Edit button. :-) |
| 17:14 |
|
dalek |
tracwiki: v87 | petdance++ | WikiStart |
| 17:14 |
|
dalek |
tracwiki: https://trac.parrot.org/parrot[…]on=87&action=diff |
| 17:18 |
|
cotto |
Why is it that dependencies between c files and headers aren't generated? It seems like something that'd be reasonably easy to do. |
| 17:18 |
|
Andy |
Why do you want them? |
| 17:18 |
|
Andy |
To minimize your rebuild if you change a .h file? |
| 17:18 |
|
Coke |
that'd be nice. having 'make realclean' not be necessary would be nice. |
| 17:19 |
|
dalek |
parrot: r40309 | cotto++ | trunk/config/gen/makefiles/root.in: |
| 17:19 |
|
dalek |
parrot: [Makefile] add a couple dependencies between c files and pmc headers |
| 17:19 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40309/ |
| 17:19 |
|
cotto |
to make make -jn break less often |
| 17:19 |
|
Coke |
we have problems with .pmc files that include .h files, that's usually a problem. |
| 17:19 |
|
Andy |
I always figured it was seen as an unnecessary optmization in favor of safety. |
| 17:20 |
|
NotFound |
Given the number times we must tell someone to make realclean, it doesn't look unnecessary to me. |
| 17:20 |
|
Andy |
I'm not arguing. |
| 17:21 |
|
cotto |
make reconfig is nice in those cases |
| 17:21 |
|
Andy |
https://trac.parrot.org/parrot/wiki/Coverity |
| 17:21 |
|
dalek |
tracwiki: v1 | petdance++ | Coverity |
| 17:21 |
|
dalek |
tracwiki: https://trac.parrot.org/parrot[…]ion=1&action=diff |
| 17:21 |
|
dalek |
tracwiki: v88 | coke++ | WikiStart |
| 17:21 |
|
dalek |
tracwiki: make priorities more visible. |
| 17:21 |
|
dalek |
tracwiki: https://trac.parrot.org/parrot[…]on=88&action=diff |
| 17:21 |
|
mj41 |
bacek: I just setup daily cron job for code coverage see e.g. http://tapir2.ro.vutbr.cz/cove[…]er-results/40307/ |
| 17:22 |
|
cotto |
mj41++ |
| 17:22 |
|
cotto |
mj41++ |
| 17:22 |
|
cotto |
mj41++ |
| 17:22 |
|
Coke |
mj41: oooh. |
| 17:22 |
|
Coke |
I am bemused that the Boolean PMC is only half covered. |
| 17:23 |
|
Coke |
that would be a nice tool for finding unused vtable entries. |
| 17:23 |
|
Coke |
(like boolean's "instantiate") |
| 17:23 |
|
cotto |
I don't think that PMC needs instantiate |
| 17:23 |
|
mj41 |
script is here http://tapir2.ro.vutbr.cz/cove[…]/makecover.sh.txt ... I copy/paste some parts ... don't know if it is ok |
| 17:24 |
|
Coke |
mj41: throw it on the wiki. =-) |
| 17:24 |
|
cotto |
btw mj41, "tapir2"? |
| 17:29 |
|
NotFound |
new_closure is never used according Coverity, and ack has the same opinion. |
| 17:30 |
|
cotto |
NotFound, I was just noticing the same thing. |
| 17:30 |
|
NotFound |
Some objection to delete it? |
| 17:30 |
|
cotto |
Go for it! |
| 17:31 |
|
moritz |
wait! |
| 17:31 |
|
moritz |
grep partcl's source first |
| 17:31 |
|
cotto |
It's not marked PARROT_EXPORT |
| 17:31 |
|
moritz |
it's usually affected by all parrot changes ;-) |
| 17:31 |
|
cotto |
that's still a good idea, though. |
| 17:31 |
|
NotFound |
And is alredy deprecated by: C API coding standards cleanup [eligible in 1.1] |
| 17:32 |
|
Coke |
moritz: especially the ones designed to make rakudo work better. :P |
| 17:32 |
|
cotto |
no need to give Coke a(nother) nervous twitch |
| 17:33 |
|
cotto |
partcl doesn't use it |
| 17:33 |
|
cotto |
lua doesn't either |
| 17:34 |
|
cotto |
NotFound, go for it. |
| 17:35 |
|
NotFound |
Killing!!! |
| 17:35 |
|
purl |
hmmm... killing is wrong or messy. |
| 17:35 |
|
Coke |
whta about parrot_new_closure ? |
| 17:35 |
|
NotFound |
Is green. |
| 17:36 |
|
cotto |
Coke, it's used and has test coverage |
| 17:36 |
|
cotto |
There are a couple other nice non-trivial spots in sub.c that don't have any coverage either. |
| 17:36 |
|
Coke |
just wondering why we have two subs that are similarly named that do different things. |
| 17:37 |
|
Coke |
realizing there's no good answer to that. |
| 17:37 |
|
cotto |
Sometimes there is no "why", but you already knew that. |
| 17:37 |
|
NotFound |
Coke: probably someone started deprecating that function by creating a new one. |
| 17:40 |
|
|
payload joined #parrot |
| 17:40 |
|
dalek |
parrot: r40310 | NotFound++ | trunk (2 files): |
| 17:40 |
|
dalek |
parrot: [cage] delete unused and deprectad function new_closure, Coverity++ |
| 17:40 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40310/ |
| 17:41 |
|
cotto |
NotFound, when you close a defect from Coverity, you should reference the CID. |
| 17:43 |
|
dalek |
tracwiki: v2 | mj41++ | CodeCoverageStatistics |
| 17:43 |
|
dalek |
tracwiki: Daily updated test coverage |
| 17:43 |
|
dalek |
tracwiki: https://trac.parrot.org/parrot[…]ion=2&action=diff |
| 17:45 |
|
NotFound |
What's the CID? |
| 17:45 |
|
purl |
the CID is all MUA or Coverity ID |
| 17:45 |
|
cotto |
purl got it right |
| 17:45 |
|
purl |
cotto: what? |
| 17:46 |
|
cotto |
There's a nice big chunk of code in Parrot_capture_lex that looks like it's probably broken. |
| 17:46 |
|
NotFound |
But purl has no commit rights ;) |
| 17:47 |
|
cotto |
It's a good thing too. |
| 17:58 |
|
cotto |
I'm having trouble convincing myself that t/pmc/multisub.t actually tests the MultiSub PMC. |
| 17:59 |
|
|
einstein joined #parrot |
| 17:59 |
|
cotto |
s#t/pmc/multisub.t#anything# also applies |
| 18:00 |
|
cotto |
Hmm. It does seem to be used, just not comprehensively tested. |
| 18:01 |
|
cotto |
#ps in 30 and no reports yet |
| 18:01 |
|
Util |
cotto: Typing!... |
| 18:01 |
|
purl |
it has been said that typing is overrated.. "I should get aronud to converting" |
| 18:02 |
|
darbelo |
cotto: There are now. |
| 18:03 |
|
jonathan |
cotto: Thanks for the reminder! |
| 18:04 |
|
moritz |
darbelo: could you also comment on your progress relative to the timeline/plan? |
| 18:11 |
|
jonathan |
Pasted report, going to grab dinner, shouldn't be away long... |
| 18:13 |
|
|
allison joined #parrot |
| 18:15 |
|
|
s1n left #parrot |
| 18:15 |
|
cotto |
darbelo, how funny that you're running into the dynpmc SUPER issue now. Someone else just filed a bug on it earlier today. |
| 18:15 |
|
cotto |
I'll make it a priority. |
| 18:16 |
|
|
chromatic joined #parrot |
| 18:20 |
|
cotto |
darbelo, ping |
| 18:23 |
|
dalek |
tracwiki: v2 | petdance++ | Coverity |
| 18:23 |
|
dalek |
tracwiki: add links and email address |
| 18:23 |
|
dalek |
tracwiki: https://trac.parrot.org/parrot[…]ion=2&action=diff |
| 18:27 |
|
darbelo |
cotto: pong |
| 18:27 |
|
cotto |
darbelo, does https://trac.parrot.org/parrot[…]uper_dynpmc.patch fix the dynpmc SUPER problem for you? |
| 18:30 |
|
dalek |
website: darbelo++ | This week on decnum-dynpmcs |
| 18:30 |
|
dalek |
website: http://www.parrot.org/content/[…]-decnum-dynpmcs-0 |
| 18:30 |
|
|
payload joined #parrot |
| 18:34 |
|
|
eternaleye joined #parrot |
| 18:34 |
|
pmichaud |
#ps in -4 ? |
| 18:34 |
|
allison |
no chromatic? |
| 18:35 |
|
NotFound |
Monochrome ps |
| 18:37 |
|
japhb |
cotto: How are you going to avoid having in-memory profiling structures, while at the same time avoiding having a loop create gobs of disk I/O? |
| 18:38 |
|
cotto |
d'oh. I do have permissions to mark milestones completed. |
| 18:40 |
|
cotto |
japhb, I don't know if there's a good answer to that question. |
| 18:41 |
|
chromatic |
To some extent, the disk I/O doesn't matter for the profile because we don't measure it. |
| 18:42 |
|
japhb |
cotto: ISTR that Devel::NYTProf got considerably faster at writing the profile info when it fed the raw data through zlib, so that might be worth considering ... but I still think that I/O for streaming the profile info will completely swamp overhead of keeping counters in mem. |
| 18:43 |
|
allison |
purl: PIR book is http://www.amazon.com/dp/0977920127 |
| 18:43 |
|
purl |
OK, allison. |
| 18:43 |
|
japhb |
chromatic: But if it makes the app uselessly slow or fills a partition, that sucks. |
| 18:44 |
|
chromatic |
At which point we fix it. |
| 18:44 |
|
chromatic |
If it doesn't harm the accuracy or correctness of the profile, I'm not worrying about it yet. |
| 18:45 |
|
chromatic |
zlib is an interesting idea though; that's worth considering. |
| 18:46 |
|
darbelo |
cotto: Yup, the patch fixed it. flh++ |
| 18:47 |
|
cotto |
Sweet. I'll have to add a dynpmc to test that. |
| 18:47 |
|
PerlJam |
allison++ |
| 18:47 |
|
pmichaud |
allison/chromatic: Okay, I want to write the PCT book (coauthors welcome) |
| 18:47 |
|
allison |
pmichaud: that would be fantastic |
| 18:47 |
|
moritz |
l1? |
| 18:47 |
|
purl |
well, l1 is a hypothetical language that would be used to implement PMCs and PIR-visible ops so that they could all be easily jitted. or http://irclog.perlgeek.de/parr[…]9-04-21#i_1083550 or http://rt.perl.org/rt3/Ticket/[…]=39313#txn-471982 or magical unicorns and flying puppies. |
| 18:47 |
|
allison |
I was thinking the next book should be PCT, maybe starting with the squaak tutorials |
| 18:48 |
|
allison |
(and expanding from there) |
| 18:48 |
|
pmichaud |
I will look at squaak, but I may want to do a different tutorial base |
| 18:48 |
|
pmichaud |
although squaak is a good start as well. |
| 18:48 |
|
chromatic |
I have on my Big Board of Books to Publish "Parrot 2.0". |
| 18:48 |
|
cotto |
Isn't PCT going to be refactored in the forseeable future? |
| 18:48 |
|
pmichaud |
anyway, I agree the next book should be PCT. |
| 18:48 |
|
pmichaud |
cotto: PGE is being refactored, but that's more internal than external. |
| 18:48 |
|
pmichaud |
PCT might be rewritten in NQP, but the API should remain the same. |
| 18:49 |
|
cotto |
So it won't be user-visible. Ok. |
| 18:49 |
|
pmichaud |
What's a reasonable timeframe for putting together a PCT book? |
| 18:49 |
|
moritz |
Util: actually http://wknight8111.blogspot.co[…]arch/label/Parrot is a good place to start searching |
| 18:50 |
|
moritz |
Whiteknight++ wrote pretty much about it |
| 18:52 |
|
Util |
moritz: thanks |
| 18:52 |
|
pmichaud |
hmmpf! If I order the book from amazon today, it won't arrive until Friday, even with 1-day shipping. |
| 18:53 |
|
pmichaud |
amazon-- |
| 18:53 |
|
cotto |
Friday is only a day away, apparently. |
| 18:53 |
|
cotto |
It'll be a very short week. |
| 18:53 |
|
pmichaud |
Amazon must be using calendars as prescribed by Judge Gross </obscure, see who gets the reference> |
| 18:54 |
|
* davidfetter |
googles judge gross |
| 18:54 |
|
davidfetter |
hrm. groklaw comes up |
| 18:54 |
|
pmichaud |
that would be the reference, yes. |
| 18:54 |
|
PerlJam |
yeah, anyone who reads groklaw would get it. |
| 18:55 |
|
PerlJam |
so ... groklaw is obscure? :) |
| 18:56 |
|
pmichaud |
might be obscure on this channel :) |
| 18:56 |
|
cotto |
chromatic, do you think 1.6 is reasonable target for profiling? |
| 18:56 |
|
chromatic |
I'd love to see 1.5. |
| 18:56 |
|
chromatic |
1.6 is very realistic. |
| 18:56 |
|
cotto |
+1 to both |
| 18:56 |
|
japhb |
PerlJam: groklaw itself is not obscure. Actually having time to *read* it might be, around here ... |
| 18:58 |
|
pmichaud |
anyway, Judge Gross basically ignored a 15-day legal requirement by declaring 30 days to be equivalent to 15 :-) |
| 18:59 |
|
davidfetter |
does nobody besides me think sandboxing needs to be done as part of the design? |
| 19:00 |
|
pmichaud |
davidfetter: I don't know -- who is beside you right now? ;-) |
| 19:00 |
|
davidfetter |
heh |
| 19:01 |
|
* davidfetter |
notes that windows' access control problems stem, mostly, from having the access control bolted onto an architecture that never considered access control |
| 19:02 |
|
|
quek left #parrot |
| 19:06 |
|
Whiteknight |
the books have arrived at the house! |
| 19:06 |
|
Whiteknight |
(my wife ordered 3 copies to start) |
| 19:07 |
|
chromatic |
Ask Allison and we'll get you as many copies as you want at cost. |
| 19:08 |
|
Coke |
though I presume if you get them not-at-cost the proceeds go to the pafo. or something. |
| 19:09 |
|
chromatic |
If he keeps his money, he has more money available to fund his work. |
| 19:09 |
|
* PerlJam |
wonders what it would take to get university profs to start using parrot in compiler design classes. |
| 19:10 |
|
cotto |
PerlJam, It'd be ideal at this point. They'd need new textbooks every year. ;) |
| 19:10 |
|
Coke |
is there text in that book that was not in docs/book/pir ? |
| 19:10 |
|
PerlJam |
heh |
| 19:11 |
|
chromatic |
There shouldn't be, Coke. |
| 19:12 |
|
Coke |
chromatic: linkx to powells and B&N are 404s, btw. |
| 19:12 |
|
Coke |
"links" |
| 19:13 |
|
chromatic |
Will check, thanks. |
| 19:13 |
|
Coke |
(perhaps not literally, but they don't go anywhere with information about the book.) |
| 19:16 |
|
chromatic |
Fixed now, Coke. |
| 19:17 |
|
particle |
825/1164 |
| 19:17 |
|
purl |
0.708762886597938 |
| 19:30 |
|
cotto |
particle, why does pluggable runloops make you happy? |
| 19:30 |
|
* particle |
wants pluggable everything |
| 19:30 |
|
* Coke |
would rather have a functional everything. |
| 19:31 |
|
particle |
cotto: my vision is for configure-time selection of runloops to build/link in parrot, and run-time selection of which to use |
| 19:31 |
|
|
bacek joined #parrot |
| 19:31 |
|
particle |
but, at this point, run-time selection is enough |
| 19:31 |
|
cotto |
./parrot --runloop=prof |
| 19:32 |
|
particle |
perl Configure.pl --runloops=slow,prof,gcdebug |
| 19:32 |
|
chromatic |
I can make them dynloadable. |
| 19:32 |
|
particle |
does that help embedders? |
| 19:33 |
|
particle |
or do we expect static builds for small defices? |
| 19:33 |
|
particle |
*devices |
| 19:33 |
|
cotto |
chromatic, lmk if that work can be parallelized efficiently. |
| 19:33 |
|
Coke |
I think embedders are not going to want pluggable loops - I expect they'd want the single, best runloop. |
| 19:33 |
|
Coke |
lmk? |
| 19:33 |
|
cotto |
lmk is let me know |
| 19:33 |
|
Coke |
much like I want as a generic user. I don't want to have to pick a runloop. |
| 19:33 |
|
Whiteknight |
Coke: single best runloop changes for each platform |
| 19:33 |
|
cotto |
we can have a sensible default |
| 19:33 |
|
Whiteknight |
well, from among the current selection |
| 19:33 |
|
Coke |
Whiteknight: yes. we have config/hints, perfect. =-) |
| 19:33 |
|
particle |
the default should also be picked at configure time |
| 19:34 |
|
particle |
unless we start using .parrotrc files :) |
| 19:34 |
|
chromatic |
cotto, right now it's blocking on me defining the struct... just need a block of time and mental energy to do it. |
| 19:35 |
|
Whiteknight |
We can have several runcores built-in, including good defaults, but we want to be able to plug in additional ones |
| 19:35 |
|
|
hercynium joined #parrot |
| 19:35 |
|
Coke |
we who? |
| 19:35 |
|
* purl |
assimilates Coke into the borg |
| 19:35 |
|
Coke |
I don't. =-) |
| 19:35 |
|
dalek |
parrot: r40311 | NotFound++ | trunk (3 files): |
| 19:35 |
|
dalek |
parrot: [cage] rename example "lorito" to "cotorra" |
| 19:35 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40311/ |
| 19:36 |
|
Coke |
. o O (Spurius of Borg) |
| 19:36 |
|
Whiteknight |
okay, "I" want to be able to plug in additional ones |
| 19:39 |
|
NotFound |
The debugger runcore is a good candidate for dynloading |
| 19:40 |
|
particle |
coke: pluggable things encourage contribution |
| 19:40 |
|
particle |
pluggable garbage collectors are very important now |
| 19:40 |
|
particle |
because we need better garbage collectors |
| 19:41 |
|
|
theory joined #parrot |
| 19:42 |
|
cotto |
pluggability also enforces encapsulation |
| 19:42 |
|
cotto |
we need more of that |
| 19:43 |
|
chromatic |
Whiteknight, does the pluggable GC allow us to change the backing storage of mem_sys_allocate and mem_sys_free? |
| 19:43 |
|
Whiteknight |
chromatic: no, not really |
| 19:45 |
|
Whiteknight |
and doesn't really allow us to change the string allocator either |
| 19:46 |
|
chromatic |
That's too bad. |
| 19:47 |
|
Whiteknight |
yeah, it all still needs lots of work to make it decent |
| 19:51 |
|
chromatic |
I bet we could get a 10% performance improvement even in IMCC by using fixed size pools instead of relying on system malloc. |
| 19:54 |
|
NotFound |
chromatic: provided we have some fast way to know the pool at free time. |
| 19:56 |
|
chromatic |
A trick I learned from Greg KH: keep a pointer to the pool right *before* the start of allocated memory. |
| 19:58 |
|
|
donaldh joined #parrot |
| 20:00 |
|
NotFound |
A good start can be to insert that trick right now, and use it to check all mem_sys_free calls. |
| 20:29 |
|
Whiteknight |
chromatic: I've been saying that thing exactly. If we could repurpose the sized pools in the GC to hold simple fixed-sized objects, that would be ideal |
| 20:30 |
|
Whiteknight |
right now the only thing that uses those pools is the List_Chunk structure (Stack_Chunk did too, but I deleted that) |
| 20:39 |
|
|
bacek joined #parrot |
| 20:42 |
|
dalek |
TT #882 closed by cotto++: SUPER() not handled correctly for DynPMCs |
| 20:43 |
|
dalek |
parrot: r40312 | NotFound++ | trunk/examples/embed/Makefile: |
| 20:43 |
|
dalek |
parrot: [examples] update and fix Makefile for cotorra (formerly lorito) to buid with an installed parrot |
| 20:43 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40312/ |
| 20:43 |
|
dalek |
parrot: r40313 | cotto++ | trunk (7 files): |
| 20:43 |
|
dalek |
parrot: [pmc2c] make VTABLE function inheritance work between dynpmcs, plus a test (see TT #882 ) |
| 20:43 |
|
dalek |
parrot: flh++ for reporting and the MethodEmitter.pm part of this patch |
| 20:43 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40313/ |
| 20:46 |
|
dalek |
rakudo: 7700f40 | pmichaud++ | docs/spectest-progress.csv: |
| 20:46 |
|
dalek |
rakudo: spectest-progress.csv update: 423 files, 12033 passing, 14 failing |
| 20:46 |
|
dalek |
rakudo: Failure summary: |
| 20:46 |
|
dalek |
rakudo: S12-enums/basic.rakudo aborted 2 test(s) |
| 20:46 |
|
dalek |
rakudo: S12-introspection/walk.t aborted 6 test(s) |
| 20:46 |
|
dalek |
rakudo: S32-list/grep.rakudo aborted 6 test(s) |
| 20:46 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]c0a6b4140da6be1bb |
| 20:48 |
|
Coke |
msg cotto - if SUPER is now working, can we kill DYNSUPER ? |
| 20:48 |
|
purl |
Message for cotto stored. |
| 20:52 |
|
cotto |
Coke, what's the purpose of DYNSUPER vs SUPER? |
| 20:53 |
|
dalek |
rakudo: 426942a | pmichaud++ | src/builtins/eval.pir: |
| 20:53 |
|
dalek |
rakudo: Correct problem with statement-style package declarators (RT #67976). |
| 20:53 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]c984ec439f09404de |
| 20:53 |
|
Coke |
cotto: presumably, to do for dynamic PMCs what SUPER did for core PMCs. |
| 20:53 |
|
Coke |
but that's just a guess. there's no docs, and it generates invalid C code. |
| 20:53 |
|
Coke |
s/no/few?/ |
| 20:54 |
|
Coke |
Ah. there is some documentation; but I think it's outdated. |
| 20:54 |
|
cotto |
it looks like it's a runtime vs compiletime distinction |
| 20:55 |
|
Coke |
except that we don't do the compiletime anymore, do we? |
| 20:55 |
|
dalek |
parrot: r40314 | NotFound++ | trunk/compilers/imcc/cfg.c: |
| 20:55 |
|
dalek |
parrot: [cage] delete an unused var |
| 20:55 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40314/ |
| 20:55 |
|
Coke |
wasn't that all moved to runtime? |
| 20:55 |
|
cotto |
Yes. |
| 20:55 |
|
cotto |
I think the DYN- variants can be deprecated. |
| 20:55 |
|
Coke |
I think they can be ripped out. |
| 20:55 |
|
cotto |
Wouldn't that be an API change? |
| 20:56 |
|
Coke |
if it worked, sure. |
| 20:56 |
|
Coke |
at least DYNSUPER is broken - didn't you just address that ticket? |
| 20:56 |
|
Coke |
(and fixed it by making SUPER work?) |
| 20:57 |
|
NotFound |
Last week we had some consensus about no need to deprecate thigs that were not working for some time. |
| 20:58 |
|
Coke |
(also, DYNSUPER seems to be the last DYN* macro for pmc2c. |
| 20:58 |
|
cotto |
NotFound, thanks. That makes life a little simpler. |
| 20:59 |
|
cotto |
I don't see any point in keeping it. I'll rip it out. |
| 20:59 |
|
Coke |
cotto+= |
| 20:59 |
|
Coke |
cotto++ |
| 21:05 |
|
dalek |
rakudo: 5b477c0 | pmichaud++ | docs/release_guide.pod: |
| 21:05 |
|
dalek |
rakudo: Note the release manager for each of the previous releases, |
| 21:05 |
|
dalek |
rakudo: and put PerlJam++ (duff) on the hook for the October 2009 release. |
| 21:06 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]4debdbd88a791ba88 |
| 21:06 |
|
Coke |
PerlJam: Ha ha. |
| 21:06 |
|
* PerlJam |
volunteers Coke for the August release ;) |
| 21:07 |
|
Coke |
can follow directions like the next guy. |
| 21:11 |
|
|
iblechbot joined #parrot |
| 21:12 |
|
|
urkle joined #parrot |
| 21:13 |
|
dalek |
parrot: r40315 | NotFound++ | trunk (2 files): |
| 21:13 |
|
dalek |
parrot: [cage] remove obsolete function str_dup |
| 21:13 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40315/ |
| 21:14 |
|
Coke |
eek, damian's writing code again. someone get the taser. |
| 21:15 |
|
pmichaud |
damian? code? /me misses the reference. |
| 21:15 |
|
cotto |
ooh. where? |
| 21:16 |
|
* Coke |
quotes "Further to that idea, I'm actually planning to attempt to port |
| 21:16 |
|
Coke |
Contextual::Return to Perl 6 in the next few weeks." |
| 21:16 |
|
Coke |
(perl6 mailing lists) |
| 21:16 |
|
pmichaud |
aha! |
| 21:16 |
|
pmichaud |
very interesting. |
| 21:16 |
|
purl |
very interesting. is that a quote? |
| 21:16 |
|
dalek |
parrot: r40316 | cotto++ | trunk (2 files): |
| 21:16 |
|
dalek |
parrot: [pmc2c] rip out DYNSUPER, which isn't needed now that it and SUPER act the same |
| 21:16 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40316/ |
| 21:18 |
|
cono |
++ <- is that for purl? |
| 21:18 |
|
Coke |
karma cotto |
| 21:18 |
|
purl |
cotto has karma of 792 |
| 21:18 |
|
Coke |
karma cono |
| 21:18 |
|
purl |
cono has karma of 2 |
| 21:18 |
|
cono |
:D |
| 21:18 |
|
Coke |
karma coke? |
| 21:18 |
|
purl |
coke has karma of 3133 |
| 21:18 |
|
* Coke |
bets most of that is for the beverage. |
| 21:18 |
|
moritz |
karma c |
| 21:18 |
|
purl |
c has karma of 7873 |
| 21:19 |
|
particle |
karma crack |
| 21:19 |
|
purl |
crack has karma of 34 |
| 21:19 |
|
Coke |
karma karma? |
| 21:19 |
|
purl |
karma has karma of 58 |
| 21:19 |
|
Coke |
ok, I'm bored now. |
| 21:19 |
|
NotFound |
I've put a lot of karma into C :D |
| 21:19 |
|
* cono |
need work better for karma :D |
| 21:24 |
|
dalek |
decnum-dynpmcs: r124 | darbelo++ | trunk/src/pmc/decint.pmc: |
| 21:24 |
|
dalek |
decnum-dynpmcs: [DecInt] Use SUPER() for the set_*_ative VTABLEs. |
| 21:24 |
|
dalek |
decnum-dynpmcs: [DecInt] Make i_divide forward to i_floor_divide. |
| 21:24 |
|
dalek |
decnum-dynpmcs: review: http://code.google.com/p/decnu[…]urce/detail?r=124 |
| 21:24 |
|
dalek |
rakudo: 2665eef | pmichaud++ | docs/release_guide.pod: |
| 21:24 |
|
dalek |
rakudo: Add particle++ for the September release. |
| 21:24 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]4fb08c987cbf73d6d |
| 21:25 |
|
particle |
karma chameleon |
| 21:25 |
|
purl |
It comes and goes. |
| 21:25 |
|
jonathan |
karma ran over dogma |
| 21:25 |
|
purl |
ran over dogma has neutral karma |
| 21:36 |
|
pmichaud |
afk, errands |
| 21:37 |
|
|
dukeleto joined #parrot |
| 22:02 |
|
|
Bruce joined #parrot |
| 22:12 |
|
|
elmex_ joined #parrot |
| 22:20 |
|
dalek |
decnum-dynpmcs: r125 | darbelo++ | trunk/ (2 files): |
| 22:20 |
|
dalek |
decnum-dynpmcs: [decTest] Adjust paths so that we can run dectest from the root dir. |
| 22:20 |
|
dalek |
decnum-dynpmcs: [decTest] Add make targets to "make" & "make clean" decTest. |
| 22:20 |
|
dalek |
decnum-dynpmcs: Configure.pl must still be run separately. |
| 22:20 |
|
dalek |
decnum-dynpmcs: review: http://code.google.com/p/decnu[…]urce/detail?r=125 |
| 22:22 |
|
treed |
What does $<dot>='.' mean? |
| 22:22 |
|
* treed |
hasn't seen that $thing in a rule before. |
| 22:23 |
|
jonathan |
$<foo>=... means "capture the next atom into the named capture foo" |
| 22:23 |
|
moritz |
it means make the match of '.' (which is a dot) available in $/<dot> |
| 22:23 |
|
jonathan |
So in that case, yes, what moritz said. |
| 22:24 |
|
treed |
Huh. |
| 22:24 |
|
treed |
But the '.' actually has to match in that place? |
| 22:24 |
|
treed |
Or does that make a one-off rule that has to be used like <dot>? |
| 22:29 |
|
dalek |
rakudo: e2d6f13 | jnthn++ | src/ (3 files): |
| 22:29 |
|
dalek |
rakudo: Implement declaration of methods on the metaclass (e.g. method ^foo($obj, ... ) { ... }). Also we toss the use of Parrot's MMD in calling !meta_compose; it's less code to not use it anyway. |
| 22:29 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]40634384c950c88dd |
| 22:29 |
|
dalek |
rakudo: 6999e58 | moritz++ | docs/release_guide.pod: |
| 22:29 |
|
dalek |
rakudo: [docs] masak++ is release manager for November |
| 22:29 |
|
dalek |
rakudo: review: http://github.com/rakudo/rakud[…]ac66064e1b026d230 |
| 22:30 |
|
treed |
<.identifier> |
| 22:30 |
|
treed |
How about that . there? |
| 22:30 |
|
moritz |
it means that it's not captured |
| 22:31 |
|
moritz |
so tehre's no $/<identifier> capture anywhere |
| 22:34 |
|
treed |
Ah. |
| 22:34 |
|
treed |
But it still needs to be there for the match. |
| 22:35 |
|
moritz |
right. |
| 22:37 |
|
treed |
God, I just don't understand why the Cardinal grammar was done this way. |
| 22:37 |
|
treed |
There are rules/tokens that aren't used anywhere. |
| 22:37 |
|
* moritz |
suggests asking Tene ;-) |
| 22:37 |
|
treed |
But look like they should be. |
| 22:37 |
|
treed |
Tene would probably just say "Yeah, it's broken. I told you that." |
| 22:37 |
|
moritz |
;-) |
| 22:37 |
|
treed |
What I don't get is how it works right now. |
| 22:37 |
|
|
rg1 joined #parrot |
| 22:38 |
|
Tene |
treed: it was adapted from a translation of the real ruby grammar. |
| 22:39 |
|
|
kid51 joined #parrot |
| 22:39 |
|
Tene |
There are unused rules because I tried to reduce the number of rules used to make the parsing more reasonable. |
| 22:39 |
|
Tene |
OR |
| 22:39 |
|
Tene |
because I never got around to making that rule actually work |
| 22:39 |
|
Tene |
so it's not included anywhere because it crashed the parse. |
| 22:40 |
|
* treed |
nods. |
| 22:40 |
|
Tene |
also, the latter. ;) |
| 22:42 |
|
* treed |
finally starts figuring out how method calls are parsed. |
| 22:42 |
|
Tene |
I'm sorry. |
| 22:43 |
|
treed |
It's mostly just where I find something and think that it's what I'm looking for. |
| 22:43 |
|
treed |
Then find out "Oh, no, that's something else." |
| 22:44 |
|
treed |
I'm also not entirely sure why the methodcall rule uses "operation" as the subrule instead of just identifier |
| 22:44 |
|
treed |
AFAIK, ` isn't a valid method name. |
| 22:44 |
|
Tene |
dunno |
| 22:44 |
|
Tene |
change it and see if anything dies. |
| 22:44 |
|
purl |
Tene: that doesn't look right |
| 22:45 |
|
treed |
Heh. |
| 22:45 |
|
treed |
Huh. |
| 22:45 |
|
treed |
method operation uses $<identifier> |
| 22:45 |
|
treed |
even though in the rule, it's <.identifier> |
| 22:46 |
|
dukeleto |
howdy |
| 22:49 |
|
|
tetragon joined #parrot |
| 22:50 |
|
cotto |
hi dukeleto |
| 22:51 |
|
dukeleto |
cotto: i am working on writing tests and understanding the parrot_debugger, do you have any words of wisdom? |
| 22:52 |
|
|
hercynium joined #parrot |
| 22:56 |
|
cotto |
"" |
| 22:56 |
|
dukeleto |
cotto: 10-4 |
| 22:56 |
|
purl |
6 |
| 22:56 |
|
cotto |
unless those are two separate things |
| 22:57 |
|
|
HostileParadox_ joined #parrot |
| 22:57 |
|
dukeleto |
cotto: separate but related |
| 23:00 |
|
cotto |
I know tests at least marginally well, but I've hardly touch the debugger code. |
| 23:00 |
|
dukeleto |
i just added a bunch of tests last night, but now I am running into things that I don't think work correctly, so I am diving into the code |
| 23:03 |
|
cotto |
I'd be glad to provide an extra set of eyes. |
| 23:06 |
|
dukeleto |
https://trac.parrot.org/parrot/ticket/592 is kind of vague, what kind of docs are wanted? I would personally like to see some examples in the docs, because some things are not clear to me right now |
| 23:10 |
|
dalek |
parrot: r40317 | dukeleto++ | trunk/src (2 files): |
| 23:10 |
|
dalek |
parrot: Small improvements to the parrot_debugger documentation |
| 23:10 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40317/ |
| 23:12 |
|
cotto |
"kind of vague" is putting it mildly |
| 23:13 |
|
cotto |
I'm not sure how to flesh out the requirements, though. |
| 23:16 |
|
|
Whiteknight joined #parrot |
| 23:16 |
|
dukeleto |
cotto: what would be useful for you? |
| 23:17 |
|
dukeleto |
do you use the debugger much in your daily dev activities with parrot? |
| 23:18 |
|
cotto |
actually, hardly at all |
| 23:19 |
|
cotto |
but a lot of what I do is below PIR. |
| 23:19 |
|
cotto |
You might look at perl's debugger for ideas. |
| 23:21 |
|
dalek |
TT #884 created by dukeleto++: parrot_debugger generates a Bus error if you set a break point before the ... |
| 23:22 |
|
dukeleto |
cotto: i can't tell if you are trying to be really funny or really mean, but I have read through the perl5 debugger's source, and I don't know that we want to model parrot's in that image :) |
| 23:22 |
|
|
theory joined #parrot |
| 23:24 |
|
cotto |
dukeleto, neither. I was just suggesting that you look at the perl debuggers docs to see what's useful. |
| 23:24 |
|
chromatic |
Only the features. |
| 23:24 |
|
dukeleto |
chromatic: touché |
| 23:25 |
|
dukeleto |
i can't seem to get printing of registers working either, does anybody have a working example of that? |
| 23:26 |
|
dukeleto |
cotto: the perl5 debugger does have copious amounts of comments-as-docs, but much of it is contradictory or is commenting about not understanding code |
| 23:26 |
|
chromatic |
I thought that already worked. |
| 23:26 |
|
|
patspam joined #parrot |
| 23:27 |
|
* cotto |
is not very useful today |
| 23:31 |
|
dalek |
TT #885 created by dukeleto++: parrot_debugger printing registers does not work |
| 23:45 |
|
dalek |
parrot: r40318 | chromatic++ | trunk/src/embed.c: |
| 23:45 |
|
dalek |
parrot: [src] Added an fclose() to the IO error branch in Parrot_pbc_read() -- no point |
| 23:45 |
|
dalek |
parrot: in leaking a filehandle if there's an error condition. (Coverity CID #360). |
| 23:45 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40318/ |
| 23:59 |
|
dalek |
parrot: r40319 | chromatic++ | trunk/config/gen/platform/generic/exec.c: |
| 23:59 |
|
dalek |
parrot: [config] Plugged a short-lived, very theoretical memory leak when exec()ing a |
| 23:59 |
|
dalek |
parrot: child process fails. It's unlikely, but it's a good practice to pair up |
| 23:59 |
|
dalek |
parrot: allocating functions and their corresponding frees anyway. (Coverity CID #358) |
| 23:59 |
|
dalek |
parrot: review: https://trac.parrot.org/parrot/changeset/40319/ |
