|
Parrot 0.8.1 "Tio Richie" Released! | www.parrot.org/ Set by moderator on 22 November 2008. |
|||
| pmichaud | # continue working in lex2 branch | 00:00 | |
| allison | oh, I see, yes it is the reverse | ||
|
00:00
johbar joined
|
|||
| allison | you'd create a lot more branches that way | 00:00 | |
| pmichaud | yes, but the individual merge steps only contain the things I've affected, not what everyone else has done | ||
| allison | I'd just do : | ||
| cd lex1 | |||
| svn merge trunk | 00:01 | ||
| pmichaud | the difference is that svn merge trunk contains all of the changes that people have made that are really irrelevant to my branch | ||
| also, in pre-svn-1.5 versions, I have to keep track of which revisions I've done previous merges in | |||
| allison | yeah, but when I do merge it back into trunk: | ||
| pmichaud | going the other way I don't have to keep track of that. | ||
| allison | cd trunk | ||
| dalek | r33017 | chromatic++ | mmdcache: | ||
| : [MMD] Cleaned up headers and types and conversion warnings in | |||
| : src/multidispatch.c. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33017 | |||
| allison | svn merge lex1 | ||
| pmichaud | that's the same in my case as well. | 00:02 | |
| allison | the diff it generates still only includes the changes I made in the branch | ||
| but, yeah, both would work | |||
| and your way might be a good way to get around a foobar'd merge | 00:03 | ||
| like, when someone deletes and immediately readds an entire directory | |||
| oooh, actually | |||
| pmichaud | yes, and the individual sync-with-trunk steps mirror what we do when we merge back to trunk for real. | ||
| allison | that may be how I can recover reini's install branch | ||
| pmichaud | s/mirror/are the same as/ | 00:04 | |
| dalek | r33018 | chromatic++ | mmdcache: | ||
| : [MMD] Made Parrot_mmd_sort_candidates() return the best candidate for a | |||
| : multidispatch, not the entire sorted candidate list. If we want the sorted | |||
| : list, we'll have to add in code to sort it later. The default case now can | |||
| : avoid the sort altogether. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33018 | |||
| pmichaud | i.e., it's the same as merging back to trunk, only we do it to a copies of trunk until we're ready to do it for real. | ||
| allison | pmichaud: yes | ||
| pmichaud: for multiple users working on a branch, it's a bit tricky to tell them "wait, stop work on that branch, now work this other branch" | 00:05 | ||
| dalek | r33019 | chromatic++ | mmdcache: | ||
| : [PMC] Removed get_pmc VTABLE override from MultiSub; previously it returned the | |||
| : sorted candidate dispatch list for the current arguments in the most recent | |||
| : Parrot context. No one used that, and it seemed weird, so now it returns the | |||
| : invocant PMC. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33019 | |||
| allison | pmichaud: but when you're the only developer, there's really no difference between the two strategies | ||
| pmichaud | yes, that might be the case if there are large numbers of users. I haven't run into that situation much. | 00:06 | |
| allison | pmichaud: thanks, good alternative | ||
| pmichaud | speaking as someone who reviews svn diffs that show up in email -- the alternative approach makes it easier to see/review what is changing :-) | ||
| but yes, they're just two ways of looking at the same process. | |||
| allison | pmichaud: yeah, I just ignore the commits that have a standard "bring X branch up-to-date with trunk" message, since they have no impact | 00:07 | |
|
00:09
AndyA joined
|
|||
| dalek | r33020 | allison++ | pdd30install_stage2: | 00:14 | |
| : Creating branch for ongoing 'make install' work, integrating trunk with branch. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33020 | |||
| allison | pmichaud: when you do 'cd new_branch; svn merge old_branch' it doesn't demand a revision number? | 00:16 | |
| pmichaud | it might, but you just use the starting revision number of old_branch | ||
| which you can get from svn log --stop-on-copy | 00:17 | ||
| you can probably also use dates -r '{2008-01-01}:head' although I haven't tried that yet. | |||
| I typically do | 00:19 | ||
| svn log --stop-on-copy # in old branch | |||
| then in new_branch I do svn merge -r #####:head | 00:20 | ||
| er, | |||
| then in new_branch I do svn merge -r #####:head old_branch | |||
| afk for a while. | |||
|
00:21
alvar joined
|
|||
| dalek | r33021 | chromatic++ | trunk: | 00:25 | |
| : [t] Made t/pmc/eval.t always use Parrot::Test::Util to create temporary files. | |||
| : This helps prevent spurious failures from temporary file collisions during | |||
| : parallel testing. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33021 | |||
| jonathan -> sleep | 00:27 | ||
| chromatic | Hm, do we have no say_p_X opcodes? | ||
|
00:45
chromatic joined,
leto_ joined
|
|||
| dalek | r33022 | chromatic++ | trunk: | 00:47 | |
| : [t] Made t/pmc/io.t always use Parrot::Test::Util to create temporary files. | |||
| : This helps prevent spurious failures from temporary file collisions during | |||
| : parallel testing. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33022 | |||
|
01:06
davidfetter joined
|
|||
| dalek | r33023 | chromatic++ | trunk: | 01:09 | |
| : [t] Made t/pmc/filehandle.t always use Parrot::Test::Util to create temporary | |||
| : files. This helps prevent spurious failures from temporary file collisions | 01:10 | ||
| : during parallel testing. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33023 | |||
| r33024 | pmichaud++ | ctxpmc: | 01:12 | ||
| : Create a branch for refactoring contexts into PMCs. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33024 | |||
| r33025 | allison++ | pdd22io_part2: | 01:40 | ||
| : [pdd22io] Interpreter initialization routines for the I/O subsystem move off to | |||
| : their own file. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33025 | |||
| pmichaud | ...if I manage to refactor contexts into PMCs, will we have to wait a release before merging to trunk? | 01:42 | |
| afk. | 01:47 | ||
| dalek | r33026 | pmichaud++ | ctxpmc: | 01:48 | |
| : Changed my mind about the branch for now. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33026 | |||
| Whiteknight | pmichaud, you still here? | 01:56 | |
| dalek | r33027 | allison++ | pdd22io_part2: | 01:58 | |
| : [pdd22io] Strip the functions needed for interpreter initialization/destruction | |||
| : down to a bare minimum. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33027 | |||
| r33028 | allison++ | pdd22io_part2: | 01:59 | ||
| : [pdd22io] Moving a few commens over to the core.c file. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33028 | |||
| r33029 | allison++ | pdd22io_part2: | 02:02 | ||
| : [pdd22io] Add headizer hooks for src/io/core.c. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33029 | |||
| r33030 | allison++ | pdd22io_part2: | 02:11 | ||
| : [pdd22io] Bringing the pdd22io_part2 branch up-to-date with trunk r33029. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33030 | |||
| r33031 | allison++ | pdd22io_part2: | 02:22 | ||
| : [pdd22io] Generated headers for io/core.c. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33031 | |||
| r33032 | allison++ | pdd22io_part2: | 02:23 | ||
| : [pdd22io] Add src/io/core.c to list of I/O C files. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33032 | |||
|
02:40
TiMBuS joined
|
|||
| chromatic | pmichaud, I wouldn't think so. | 02:47 | |
|
02:51
leto_ joined
|
|||
| pmichaud | missed whiteknight :-( | 03:26 | |
|
03:39
tak joined
|
|||
| Coke | I think contexts count as 'internal guts with which you shouldn't be messing" | 03:50 | |
| Infinoid | pmichaud: oh, wow, you're refactoing contexts into PMCs? nice. | 03:54 | |
| on a whim, I created a "Context" PMC that contained nothing but a pointer to a Parrot_Context, in an effort to make them take part in normal garbage collection. mark_context() marked the PMC as active, and Context.destroy() is the only thing in the system that calls Parrot_free_context() | 03:55 | ||
| It doesn't quite work, though. | |||
| by the time I got anything working at all, lex4 had already gone its own way, so rather than tracking your refcounting tweaks, I was planning to redo that hack using trunk this time. | 03:56 | ||
| actually moving all the Parrot_Context members into context.pmc ATTRs seemed like a lot more effort, I just wanted a proof of concept to see how badly it failed... | 03:57 | ||
|
04:02
elmex_ joined
04:08
chromatic joined
04:13
leto_ joined
|
|||
| Coke needs faster tcl. :| | 04:33 | ||
|
04:34
tak joined
|
|||
| allison | Infinoid: effort that's already done, see RT #60564 | 04:42 | |
| pmichaud: when I get a PGE error like:unrecognized statement at line 5, near "c\\x{c0}\\x{2}pc\\x{c0}\\x{2}\\x{90}c\\x{c0}" | 04:43 | ||
| pmichaud: any thoughts on what kind of file I should be looking for? a grammar file? a generated PIR file? | |||
| Coke | Parrot VM: PANIC: Out of mem! | 04:44 | |
|
04:45
jimmy joined
|
|||
| allison | Coke: fun | 04:45 | |
| Coke cries. | 04:46 | ||
| tries to skip to the panic, but removing the big chunk of code makes it not happen anywhere near the same plac. | 04:48 | ||
| It is very hard to find bits of partcl to work on that don't involve parrot exploding. :| | |||
| Infinoid | allison: thanks | 04:50 | |
| allison | Coke: what are you doing that uses so much memory? | 04:51 | |
| Coke | running one of tcl's spec tests. | ||
| allison | or, what's the backtrace before running out of memory? | ||
| Coke | how do I get the backtrace ? just run via gdb? | ||
| allison | Coke: does the test churn through a large number of objects, or is this some runaway process? | 04:52 | |
| Coke: yes, gdb will do it | |||
| Coke | running now. give me 10m for it to crash again. | ||
| 3384 14133 100007 library/tcltest/tcltest.tcl | 04:53 | ||
| 693 3903 23569 t_tcl/regexp.test | |||
| 4077 18036 123576 total | |||
| so, I'm running 4077 lines of tcl, give or take. | |||
| (and this particular test is using the regexp engine, so I'm using even more PGE) | 04:56 | ||
| backtrace ends here: 3384 14133 100007 library/tcltest/tcltest.tcl | 05:00 | ||
| 693 3903 23569 t_tcl/regexp.test | |||
| 4077 18036 123576 total | |||
| er, here: | |||
| #2099 0x00002ca8 in main (argc=2, argv=0xbfffef78) at src/main.c:61 | |||
| is there a way to save off the trace to a file? | |||
| allison | the first 10 lines should be a good start | 05:04 | |
| just to get an idea of what it's doing | |||
| Coke | how can I get the first 10 lines? =-) | ||
| can I tell gdb to pipe things through more? | |||
| the 2099 lines blows through my screen buffer. | 05:05 | ||
| allison | bt n | ||
| (where n is the number of lines) | |||
| (well, the number of call frames) | 05:06 | ||
| nopaste | "coke" at 72.228.52.192 pasted "bt" (3 lines) at nopaste.snit.ch/14690 | 05:07 | |
| "coke" at 72.228.52.192 pasted "here's the full bt" (2100 lines) at nopaste.snit.ch/14691 | 05:11 | ||
| Coke | ah. "set logging on" | 05:12 | |
| see anything helpful in the bt? | 05:13 | ||
| -> | 05:29 | ||
| allison | Coke: absorbed by an I/O bug... looking now | 05:48 | |
| Coke: looks like you've got one of those infinite exception recursions | 05:49 | ||
| 'fail_if_type_exists' immediately followed by 'Parrot_ex_throw_from_c_args' keeps repeating over, and over, and over again | 05:50 | ||
| it all starts with a 'Parrot_newclass_p_p' | 05:51 | ||
| Coke: hmmm.... have you added any new PMCs without doing a 'make realclean'? | 05:52 | ||
| pmichaud | unrecognized statement... is this in Perl6Grammar, perhaps? | 06:08 | |
| at any rate, those look like utf8 | |||
| that weren't read as utf8 | |||
| Infinoid: I'm thinking about moving contexts to pmcs... but right now I'm not much motivated to do much more on this for a while. Too much effort for too little progress this week. | 06:09 | ||
| chromatic | ... and thus Jonathan and I have nearly doubled the speed of that benchmark. | 06:14 | |
| dalek | r33033 | chromatic++ | mmdcache: | ||
| : [src] Demoted the two PMCs used to track argument and results signatures in | |||
| : Parrot_pcc_invoke_sub_from_sig_object() to temporary PMCs. This gives a 9.93% | |||
| : speedup in the primes2.pir benchmark example. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33033 | |||
|
06:17
Theory joined
|
|||
| allison | pmichaud: yes, it was in Perl6Grammar, and I figured it out. the 'slurp' method was reading beyond the end of the file and dumping in garbage | 06:23 | |
| pmichaud | ah, good. | ||
| (good that you figured it out.) | |||
| I'm just reading about Whiteknight's issues with '$/' -- I suspect he's running into the same lexical issues that we've been having in other parts of development. | 06:24 | ||
| is there a current branch I could troubleshoot on that would be helpful? (i.e., is the calling conventions branch "up to date" so that I could try reproducing what he's seeing?) | 06:25 | ||
| allison | pmichaud: yes, it the calling conventions branch should be up-to-date | 06:29 | |
| pmichaud | allison: okay, I've duplicated the error... checking. | 06:36 | |
| it _has_ to be a problem with lexicals in the branch. | 06:37 | ||
| allison | pmichaud: it sounds like it, but why on the branch and not in trunk? | ||
| pmichaud: the changes in this branch are very conservative | 06:38 | ||
| pmichaud | I think it's the same context sensitivity that's been giving me trouble all week. | ||
| but in this case, I can fairly well prove the code itself is correct. | |||
| just a sec. | |||
| allison | pmichaud: that makes sense | ||
| nopaste | "pmichaud" at 72.181.176.220 pasted "lexical problem in calling_conventions branch (for allison/Whiteknight)" (171 lines) at nopaste.snit.ch/14692 | 06:40 | |
| pmichaud | you can see from the top of the nopaste that I'm getting the same error that Whiteknight reported. | ||
| the problem is in the 'TOP' method, which is at the very end of the file (and nopaste) | |||
| line 134 is the one that reads | 06:41 | ||
| .lex "$/", param_15 | |||
| oops, sorry. | |||
| line 134 is the one that reads | |||
| purl | i already had it that way, pmichaud. | ||
| pmichaud | find_lex $P19, "$/" | ||
| however, it's very clear that a lexical called '$/' must exist -- it was declared four lines above (the .lex line I just inadvertently pasted). | 06:42 | ||
| allison | got it | ||
| pmichaud | so the problem isn't in the PIR code that's being generated. | ||
| given that the find_lex is in the same sub where '$/' is being declared, that tells me that the lex_pad must not be created correctly. | 06:43 | ||
| allison | so this is one of those cases where it expects the lex_pad to come from an outer sub? | ||
| pmichaud | no. | 06:44 | |
| at least, it shouldn't be. The '$/' lexical is declared as being the first parameter to the sub. | |||
| allison | if the context had been reclaimed prematurely, then the other code in the method wouldn't work either... so is it just a problem with the code that constructs the lex_pad for the context? | 06:45 | |
| pmichaud | there's no outer context here. | 06:47 | |
| the problem is typically that outer contexts get reclaimed prematurely. | 06:48 | ||
| allison | there is another possibility, if get_hll_global or find_lex perform any C-level method calls, and that is that pcc_invoke isn't restoring the context properly | ||
| (C-level method calls is what changed slightly in the branch) | |||
| pmichaud | I've not had a case where the current context gets reclaimed prematurely.... although there's very little at the moment in Parrot to prevent that from happening. | 06:49 | |
|
06:49
bacek joined
|
|||
| allison | pmichaud: okay, not with the context of the sub itself being reclaimed, only outer contexts, then not relevant here | 06:49 | |
| pmichaud | so yes, it could be pcc_invoke that is causing the context to be lost. | ||
|
06:50
Theory joined
|
|||
| pmichaud | I'll go ahead and put this information in the message for Whiteknight as well. | 06:50 | |
| allison | excellent, thanks | ||
| dalek | r33034 | chromatic++ | trunk: | 06:55 | |
| : [PMC] Added missing braces around an if condition in Role PMC to prevent a | |||
| : potential crash when passing a NULL PMC to a function that doesn't expect it. | |||
| : See CID #137, found by Coverity scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33034 | |||
| r33035 | allison++ | pdd22io_part2: | 06:59 | ||
| : [pdd22io] Incorporate old I/O functionality for UTF8 into new I/O subsystem. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33035 | |||
| chromatic | Don't ever pass stderr when you meant to pass interp. | 07:05 | |
| cotto | easy mistake | 07:06 | |
| they're hard to tell apart | |||
| chromatic | "Why don't I have an ->arena_base? Why is my program crashing?" | ||
| dalek | r33036 | chromatic++ | trunk: | 07:07 | |
| : [src] Fixed a resource leak from not freeing a string_to_cstring() result in | |||
| : find_exception_handler()'s no-handler-found case. See CID #149, reported by | |||
| : Coverity Scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33036 | |||
| r33037 | chromatic++ | trunk: | 07:16 | ||
| : [IMCC] Fixed a resource leak by freeing the result of a string_to_cstring() | |||
| : call in IMCC_subst_constants(). See CID #151, reported by Coverity Scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33037 | |||
| r33038 | chromatic++ | trunk: | 07:26 | ||
| : [pdb] Added pointer to free to PDB_free_file()'s argument list; this makes the | |||
| : memory leak reported in RT #46127 and CID #152 trivial to fix. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33038 | |||
|
08:18
jimmy joined
|
|||
| jimmy | l | 08:18 | |
| cotto | That must have been a very important 1. | 08:28 | |
| or l | |||
| Zaba | that was l | 09:00 | |
| Zaba just realised 1 and l are very similar in terminus | |||
|
09:03
iblechbot joined
09:21
Hadi joined
09:27
barney joined
09:28
Zaba joined
09:38
alvar joined
|
|||
| cotto | barney, is there something special I need to do for Pipp to make --target=pir work? | 09:42 | |
| parse, past and post work fine | 09:43 | ||
| The build process assumes that --target=pir --output=foo.pir works, although it doesn't seem to break anything. | 10:14 | ||
| sleep & | |||
| night | |||
| dalek | bernhard.schmalhofer@gmx.de | Pipp: | 10:54 | |
| link: www.perlfoundation.org/parrot/index.cgi?pipp | |||
| barney | cotto: sorry, I was listening to talk about coroutines at Frankfurt Perl Workshop | 10:55 | |
| I tried to support PIT output, but didn't finish that. | |||
| moritz | let me guess, Mark Lehmann? | ||
|
10:56
rob joined
|
|||
| barney | No, implementing gather take with opcode rewriting | 10:56 | |
| barney talking about Pipp at FPW | |||
| moritz | by whom? | ||
| dalek | r33039 | fperrad++ | trunk: | 11:10 | |
| : [Lua] | |||
| : - rewrite with keyed namespace | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33039 | |||
|
11:10
samlh joined
|
|||
| barney | moritz: Rolf Langsdorfer | 11:18 | |
| cotto: You can probably use the 'command_line' sub from HLLCompiler | |||
| barney finished lightening talk on Pipp | 11:19 | ||
| s/Langsdorfer/Langsdorf/ | 11:20 | ||
| jonathan | ahojte | 11:24 | |
|
11:28
Alias joined
|
|||
| moritz | never heard of him | 11:29 | |
| barney | from Darmstadt | 11:31 | |
| dalek | r33040 | bernhard++ | trunk: | 11:35 | |
| : [codingstd] remove trailing whitespace | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33040 | |||
| r33041 | bernhard++ | trunk: | 11:39 | ||
| : [codingstd] fix indention of C-macros | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33041 | |||
|
11:59
Hadi joined
12:00
davidfetter joined
|
|||
| dalek | r33042 | bernhard++ | trunk: | 12:14 | |
| : [codingstd] satisfy c_macro.t | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33042 | |||
|
12:40
masak joined
12:41
Zaba joined
12:49
jimmy joined
12:56
Hadi left
|
|||
| moritz | jonathan: I see just one mention of 'is default' in the specs, and that's in S29, and doesn't explain it at all | 13:05 | |
| jonathan: and that's probabaly why there are no spectests either | 13:06 | ||
| jonathan | moritz: It's mentioned in S12 | 13:07 | |
| moritz: And yes, there's no tests - it's on my task list unless I get beaten to it. | |||
| Just been distracted hacking on the MMD caching... | |||
| moritz | jonathan: ah, maybe my grep missed it because it's not mentioned as 'is default' there | ||
| I'll give it a shot tonight, unless I forget | 13:08 | ||
| jonathan | moritz: I think it just mentions it as the default trait. | ||
| There isn't a code example. | |||
| jonathan -> slovak and housework | 13:10 | ||
| dalek | r33043 | moritz++ | trunk: | 13:13 | |
| : [rakudo] add test for slurpy params to spectest | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33043 | |||
|
13:54
rdice joined
14:08
kid51 joined
14:25
ambs joined
14:26
ambs left,
Whiteknight joined
14:40
jimmy joined
|
|||
| dalek | r33044 | fperrad++ | trunk: | 14:40 | |
| : [Lua] | |||
| : - add mathx library | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33044 | |||
| jimmy | Will Perl6 implement bootstrapping? | 14:41 | |
| moritz | what do you mean? | ||
| bootstrapping is a technique, not a language feature | |||
| jimmy | hmm, I mean Perl6 compile itself. | 14:42 | |
| moritz | every implementation is free do it as they like | ||
| jimmy | Will it? | 14:43 | |
| purl | Will it is it being written to use djb's string functions and such? | ||
| moritz | depends on the "it" | ||
| jimmy | and Pugs had been stopped? | ||
| Infinoid | perl6 compiling perl6 is a requirement of eval() | ||
| whether the various implementations of perl6 use this feature heavily in their implementation is another question | 14:44 | ||
| moritz | Infinoid: no. It can still be a C backend that does the eval | ||
| Infinoid | true, but transparent from the user's perspective | ||
| jimmy | i.e, GCC compile itself. Java compile itself. | ||
| and one target of perl6 is to compile itself | 14:45 | ||
| moritz | jimmy: audreyt wanted to continue pugs development once ghc6.10.1 ewas released, which happend one or two weeks ago. That's the last thing I know about it | ||
| there will likely be a bootstrapping compiler at one point | |||
| the Perl 6 compiler named 'elf' is bootstrapping, for example | 14:46 | ||
| lathos | The.. what? | ||
| purl | hmmm... the is a few features i love in firefox, i will totally love you so. or an article | ||
| jimmy | form svn, I saw Pugs had been stopped for two years. I head the target of Pugs is to help perl6 to compile itself. | ||
| moritz | lathos: perl.net.au/wiki/Elf | ||
| lathos | Sounds interesting. | 14:47 | |
| jimmy | Parrot is more up to date, and is being developed, but has been focused on implementing language features, rather than on becoming generally usable. | 14:50 | |
| moritz | purl, msg jonathan I've added t/spec/S12-methods/default-trait.t, one TODO test for 'is default' on methods, the tests for subs all pass | 14:52 | |
| purl | Message for jonathan stored. | ||
| lathos | Oh, I dunno. The last few updates I've seen have all been in src/ and very little in languages/. Parrot's a good VM. | ||
|
14:54
Coleoid_ joined
|
|||
| jimmy | so perl6 does not implement self-hosting? | 14:55 | |
| moritz | not at the moment. | ||
| jimmy | maybe it is difficult and unnecessary,because of Parrot. | 14:57 | |
| Coleoid | jimmy: it's not as black-and-white as it would be with, for example, a C++ implementation. | 14:58 | |
|
14:58
jan joined
|
|||
| Coleoid | Rakudo (atop Parrot) has some of its features written in Perl 6 already... | 14:59 | |
| moritz | right; for example the grammar is already written in Perl 6, and parsed by the same program that executes Perl 6 regexes | ||
| masak | hi jimmy! | ||
| jimmy | hi masak | ||
| some people of china had been confused. for Parrot perl6 and pugs. | 15:00 | ||
| masak | jimmy: ä»å¤©ęåØå¦ä¹ äøęć | 15:01 | |
| jimmy | i want to give them a clear answer. | ||
| masak: čæę„å¾å¤§å§ļ¼åµåµć | 15:02 | ||
| masak: é³ę ēé®é¢čæä¹ ęÆåļ¼ | |||
| masak | jimmy: the tones? I think they will always be a problem to me. | 15:03 | |
| but maybe less and less of a problem. | |||
| I credit myself with having a good ear for tones. | 15:04 | ||
| jimmy | masak: yes. åŗę¬åå¾éč¦ļ¼ę们å°ę¶åä¹ä¼å¦å幓äøå¹“ēé³ę | ||
| masak | jimmy: let me try to untagle that sentence for a while :) | 15:05 | |
| basic success is important, that's true. | |||
| jimmy | i.e: ÄĆ”ĒĆ ,ÅóĒò,ÄĆ©ÄĆØ,ūúĒù | 15:06 | |
| masak | ah, you're talking about the written accents, not the spoken tones? | ||
| yes, I definitely like Pinyin better with the accents than without them. :) | |||
| jimmy | vowel practising is very very important. | 15:07 | |
| masak | I'll keep that in mind. | ||
| jimmy | s/practsing/practice | ||
| masak | I have a 'language partner', a student from China who helps me along each week. | 15:08 | |
| jimmy | ÄĆ”ĒĆ ,ÅóĒò,ÄĆ©ÄĆØ,ūúĒù, I studied it using a year. | ||
| masak | wow. | ||
| actually, I use Perl 6 to practice Chinese characters! | |||
| jimmy | it is basic skill, every student in china should practise it. | 15:09 | |
|
15:09
alvar joined
|
|||
| masak | aye. | 15:09 | |
| jimmy | some student's teacher who does not speak mandarin in china , his students does not. such as guangdong | 15:10 | |
| masak: good. | 15:11 | ||
| lathos | I can more-or-less read Chinese thanks to Japanese. | ||
| jimmy | Chinese is different from japanese, I can't read japanese. | 15:12 | |
| :( | |||
| lathos | Japanese people can read Chinese but Chinese people can't read Japanese. | 15:13 | |
| jimmy | masak: When you come to China. You will be my language partner. :) | ||
| lathos | Japan added two more alphabets just to be confusing. | ||
| jimmy | lathos: I know nothing about Japanese. | 15:14 | |
| masak | jimmy: gladly. | 15:15 | |
| lathos: they're not alphabets, though. | |||
| lathos | Well, syllabaries. Bah. | 15:16 | |
| Bloody pedants. | |||
| masak | :) | ||
| we're programmers, what do you expect? | |||
| jimmy | masak: can you speak other language? | ||
| masak | jimmy: yes. | ||
| jimmy: can you? | 15:17 | ||
| jimmy | as I know. rudian ren can speak many languages. | ||
| masak | actually, it varies from ruidian ren to ruidian ren :) | ||
| jimmy | I just english ,mandarin and native dialect. | 15:18 | |
| moritz can speak perl, Perl 6, Eiffel, a bit of each C++, C, Fortran, PIR, HTML, CSS, JS, Java... ;) | |||
| jimmy | I mean human's language. | ||
| not computer's language. | 15:19 | ||
| masak | jimmy: moritz is teasing you :) | ||
| I took some French in high school, and I know a bit of Russian from my grandmother. I'm very good at Esperanto, for some reason. I also dabbled in Jiddisch and can recognize some German expressions. | |||
| I want to learn Tibetan. | |||
| jimmy | it is not necessary | 15:20 | |
| masak | jimmy: somehow, I knew you'd say that. :) | 15:21 | |
| sorry, I have a lot of preconceptions about Chinese. | |||
| jimmy | mandarin is enough. | 15:22 | |
| masak : nothing. | |||
| purl | i heard nothing was free! or worth nothing but it's free, feeling good was easy, Lord, then Bobby sang the blues, nothing left was all she left for me or nothing crunches like a triscuit | ||
| masak | jimmy: if I though that way about languages, I would have stopped at English. | ||
| moritz | jimmy: so you would also people discourage from learning kantonese? | ||
| masak | purl: forget nothing | ||
| purl | masak: I forgot nothing | ||
| jimmy | there is many native dialects in china every city may be not the same. | 15:23 | |
| masak | jimmy: I apologise if this is a touchy issue. might not native Tibetans object to speaking Mandarin with a foreigner? | 15:24 | |
| lathos | If you want to get to know Tibetans, you need to learn Tibetan. | ||
| jimmy | if there is many foreigners in a city. They should speak mandarin. | 15:25 | |
| masak | lathos: that's my feeling as well. | ||
| lathos | Yeah, yeah, they should speak Mandarin because TIBET IS CHINA. But really, learn TIbetan. | ||
| masak | aye. | 15:26 | |
| jimmy | And if you want to travel in Tibet, then learn it. | ||
| and that is my option. | 15:27 | ||
| lathos | masak: Where are you based? | 15:28 | |
|
15:29
Andy joined
|
|||
| masak | lathos: Uppsala, Sweden. | 15:29 | |
| lathos: what about you? | |||
| lathos | Near Kyoto. | ||
| masak | lathos: yesterday I learned, from my local sushi provider, that 大 is pronounced "oki" in Japanese | 15:37 | |
| lathos | Long o, but yes. | ||
| masak | how does one write that? | ||
| lathos | Or "dai", in compounds. | 15:38 | |
| masak | ah. | ||
| that's closer to Mandarin pronunciation. | |||
| lathos | Åkii. | 15:39 | |
|
15:39
kj joined
15:42
cognominal joined
|
|||
| masak | lathos: thank you | 15:44 | |
|
16:05
Zaba joined
16:06
cognominal joined
|
|||
| dalek | r33045 | kjs++ | trunk: | 16:21 | |
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33045 | |||
| r33046 | kjs++ | trunk: | 16:31 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33046 | |||
| r33047 | kjs++ | trunk: | 17:00 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33047 | 17:01 | ||
| r33048 | kjs++ | trunk: | 17:02 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33048 | |||
| r33049 | kjs++ | trunk: | 17:07 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33049 | |||
|
17:10
Theory joined
|
|||
| dalek | r33050 | kjs++ | trunk: | 17:22 | |
| : [t] change PASM registers to PIR registers in PIR tests. Note that the PIR code is in a Perl string, so escape the $. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33050 | |||
| r33051 | kjs++ | trunk: | 17:31 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33051 | |||
|
17:31
Hadi joined
17:33
gryphon joined
|
|||
| dalek | r33052 | kjs++ | trunk: | 17:34 | |
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33052 | |||
| r33053 | kjs++ | trunk: | 17:37 | ||
| : [t] change PASM registers to PIR registers in PIR tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33053 | |||
|
17:47
Andy joined
18:07
spinclad joined
18:12
spinclad_ joined
|
|||
| dalek | r33054 | moritz++ | trunk: | 19:24 | |
| : [rakudo] add tests for 'is also' on classes | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33054 | |||
|
19:37
Hadi left
19:46
rob joined
|
|||
| dalek | r33055 | kjs++ | trunk: | 19:47 | |
| : [imcc] Disallow PASM registers. This is a huge patch, but that's to prevent failures; quite some changes were needed. I ran make test several times; all should be fine now, but will recheck again. The IMCC lexer needed quite some work, more than the initial patch by Coke++. Also, there may be a memory leak in there, as chromatic indicated earlier. Will fix that in separate patch. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33055 | |||
| moritz | kj: I've get a failure in t/pmc/threads.t | 19:52 | |
| error:imcc:'I5' is only a valid register name in PASM mode | |||
| # in file '/home/moritz/src/parrot/t/pmc/threads_2.pir' line 4 | |||
| kj | that's weird.. but I'll have a look at it right now | ||
| dalek | r33056 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Convert UTF-8 utility functions from old I/O layer. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33056 | |||
| kj | moritz: thanks for testing | 19:53 | |
|
19:55
johbar joined
|
|||
| kj | fixed, committing now. | 19:56 | |
| nopaste | "moritz" at 89.14.204.204 pasted "patch for threads.t, fixes all but one failure (for kj to review)" (122 lines) at nopaste.snit.ch/14694 | ||
| moritz | ah, you're faster ;) | ||
| kj | hehe, I've done for the past 2 hours :-) | 19:57 | |
| so I trained my eye :- | |||
| there's 1 test failing, but it's already failing on a clean checkout, without my huge patch. It's the dynpmc test | |||
| dalek | r33057 | kjs++ | trunk: | 19:58 | |
| : [t] and also [compilers/json]. Fixed some forgotton PASM registers into PIR regs. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33057 | |||
| kj | moritz: could you recheck again? | 19:59 | |
| moritz | kj: yes, I'm running 'make test' right now | 20:00 | |
| kj | ah, thanks! | ||
| phew, that was a tricky task :-| | |||
|
20:02
masak joined
|
|||
| nopaste | "moritz" at 89.14.204.204 pasted "test summary for kj" (17 lines) at nopaste.snit.ch/14695 | 20:02 | |
| dalek | r33058 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Convert IMCC over to new I/O subsystem. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33058 | |||
| kj | moritz: thanks, will look into it. | 20:03 | |
|
20:05
silug joined
|
|||
| dalek | r33059 | allison++ | pdd22io_part2: | 20:05 | |
| : [pdd22io] Update documentation for Unix I/O utility functions to match reality. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33059 | |||
| r33060 | kjs++ | trunk: | 20:14 | ||
| : [t] more PASM -->PIR reg conversion. reported by moritz++. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33060 | |||
| r33061 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Remove stray static prototypes for old functions. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33061 | |||
| r33062 | kjs++ | trunk: | |||
| : [t] and yet another PASM->PIR reg conversion. reported by moritz++. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33062 | |||
| jonathan | make nom | 20:15 | |
| Assertion Failed: frying_chips->location != oven | |||
| MAKE: make nom FAIL | |||
| masak | nom has too many dependencies. | ||
| moritz recommends 'fmake', the non-GNU Food Make | 20:16 | ||
| jonathan should read the packet while in the supermarket next time, not just look at the picture of nice looking chips (fries for any Americans ;)) on the packet... | |||
| moritz | kj: t/pmc/bigint.t line 98, error:imcc:'$N10' is not a valid register name in pasm mode | ||
| jonathan: so what did you buy? ;-) | 20:17 | ||
| dalek | r33063 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Add a 'get_bool' vtable function, and 'readall' and 'encoding' methods. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33063 | |||
| r33064 | kjs++ | trunk: | |||
| : [imcc] do str_dup() after the PASM reg check, as suggested by chromatic++. This prevents a potential memory leak. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33064 | |||
| r33065 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Add tests for 'readall' method on the FileHandle PMC. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33065 | |||
| kj | moritz: bigint.t should be fine now. | 20:18 | |
| moritz | kj: it's not, here | 20:19 | |
| kj | mmm that's weird. | 20:20 | |
| moritz | forgot to commit it? | ||
| or skipped on your platform? | |||
| kj | nope, prove t\\pmc\\bigint.t is fine here | ||
| eek! | |||
| no it isn't | |||
| masak | ook! | ||
| moritz | and t/dynpmc/foo.t also fails | ||
| kj | yes, but that one already failed on my system | 20:21 | |
| moritz | but the failures look very much like new errors here | ||
| # error:imcc:'P1' is only a valid register name in PASM mode | |||
| # in file '/home/moritz/src/parrot/t/dynpmc/foo_5.pir' line 14 | |||
| kj | yes those are new indeed. | ||
| i'll work on them | 20:22 | ||
| moritz | kj: I can take care of the dynpmc test, it passed here before your update | 20:23 | |
| kj | if you don't mind (A) | ||
| it seems my test is different... | |||
| I can't find a P1 anymore in my foo_5.pir | 20:24 | ||
| neither the bigint.t failure; can't find the error. | |||
| moritz | with r33066 the dynpmc test passes | 20:25 | |
| dalek | r33066 | moritz++ | trunk: | ||
| : [t] use PIR registers consitently in t/dynpmc/foo.t | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33066 | |||
| kj | aah. this explains: conflicht. | ||
| conflict* | |||
| so much for the "I'll fix this in an hour" :-S | 20:26 | ||
|
20:26
tewk joined
20:27
kid51 joined
|
|||
| kj | moritz: t\\dynpmc\\foo.t fails (test 3) ; class 'Foo' not found. | 20:27 | |
| moritz | kj: it's in runtime/parrot/include/fp_equality.pasm, it seems | ||
| kj | but that was already the case on my system | ||
| moritz: yes i had to edit that one as well | 20:28 | ||
| dalek | r33067 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Add a test for setting and retrieving the 'encoding' attribute of a | |||
| : FileHandle PMC. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33067 | |||
| kj | it has a couple of macros that use PASM register; they had to be converted to PIR regs as well, but they were also used in some PASM tests, so I copy/pasted the macros, and added a _pasm suffix to them | 20:29 | |
| moritz | well, if I remove those dollars in fp_equality.pasm bigint.t works fine | 20:32 | |
| kj | the fp_eq macro should be renamed to fp_eq_pasm | 20:33 | |
| so instead of expanding .fp_eq (or .fp_ne) | |||
| use .fp_eq_pasm | |||
| and .fp_ne_pasm, respectively | |||
| jonathan | moritz: I bought chips...that you have to fry. Putting them in the oven didn't work out so good... | ||
| moritz | right, because just removing the dollars make t/pmc/fixedfloatarray.t fail | ||
| kj | exactly; like I mentioned earlier, these macros are use din both PASM and PIR mode | 20:34 | |
| so had to duplicate them, and change one set into PIR=style | |||
| so in PASM mode, always use .fp_eq_pasm, and otherwise just .fp_eq | |||
| moritz | ah, i didn't understood what you said until I ran into the same issue :) | ||
| kj | oh sorry for being unclear then :-) | ||
| yea, well it's not the nicest solution, but I figured removing PASM regs in PIR mode is more important than those macros | 20:35 | ||
| I guess another way of fixing it may be to use .local num's | |||
| moritz doesn't care as long as 'make test' is clean ;) | 20:36 | ||
| kj | how does t/dynpmc/foo.t work on your system? | ||
|
20:36
kid51 joined
|
|||
| moritz | Result: PASS | 20:38 | |
| kj | ok, fails here (windows) | ||
| so does t/pmc/eval.t | |||
| "permission denied" on t/pmc/eval.t... | |||
|
20:43
kid51 joined
|
|||
| dalek | r33068 | allison++ | pdd22io_part2: | 21:13 | |
| : [pdd22io] Documentation and exception message fixes for I/O interpreter | |||
| : initialization/destruction functions. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33068 | |||
|
21:14
Andy joined
|
|||
| dalek | r33069 | allison++ | pdd22io_part2: | 21:16 | |
| : [pdd22io] Convert the interpreter to use new I/O subsystem. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33069 | |||
| r33070 | allison++ | pdd22io_part2: | 21:20 | ||
| : [pdd22io] Convert all calls to old I/O system in src/interpreter.c. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33070 | |||
| r33071 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system in src/thread.c. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33071 | |||
| moritz | the 'parrot-devel' link on www.parrot.org/download points to parrot-0.8.0 - aren't we up to 0.8.1 right now? | ||
| dalek | r33072 | allison++ | pdd22io_part2: | 21:21 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33072 | |||
|
21:22
Theory joined
|
|||
| dalek | r33073 | allison++ | pdd22io_part2: | 21:23 | |
| : [pdd22io] Change filehandle mode parsing to use Parrot string directly, add a | |||
| : comparison function for encoding. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33073 | |||
| r33074 | allison++ | pdd22io_part2: | 21:25 | ||
| : [pdd22io] Document 'encoding' method on FileHandle PMC. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33074 | |||
| r33075 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33075 | |||
| r33076 | allison++ | pdd22io_part2: | 21:27 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33076 | |||
| r33077 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33077 | |||
|
21:28
kj joined
|
|||
| dalek | r33078 | allison++ | pdd22io_part2: | 21:29 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33078 | |||
| r33079 | allison++ | pdd22io_part2: | 21:32 | ||
| : [pdd22io] Add headers for encoding check and UTF-8 read/write functions. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33079 | |||
| r33080 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Add compilation of UTF-8 utility functions. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33080 | |||
| r33081 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33081 | |||
| jonathan | My does she patch fast... ;-) | ||
| dalek | r33082 | allison++ | pdd22io_part2: | 21:34 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33082 | |||
| r33083 | allison++ | pdd22io_part2: | 21:36 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33083 | |||
| moritz | allison: should the pdd22io_part2 branch build right now? it hangs during ./miniparrot config_lib.pasm > runtime/parrot/include/config.fpmc | 21:39 | |
| dalek | r33084 | kjs++ | trunk: | 21:45 | |
| : [imcc] implement .line directive. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33084 | |||
| r33085 | allison++ | pdd22io_part2: | 21:55 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33085 | |||
| r33086 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33086 | |||
| r33087 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33087 | |||
| r33088 | allison++ | pdd22io_part2: | 21:56 | ||
| : [pdd22io] Convert all references to old I/O system (even in comments). | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33088 | |||
| r33089 | allison++ | pdd22io_part2: | 21:57 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33089 | |||
| GeJ | Good morning everyone | 21:58 | |
| dalek | r33090 | allison++ | pdd22io_part2: | 21:59 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33090 | |||
| r33091 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33091 | |||
| masak | GeJ: evenin' | ||
| GeJ | hej masak | 22:01 | |
| masak | hej, hej :) | 22:02 | |
| dalek | r33092 | allison++ | pdd22io_part2: | 22:03 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33092 | |||
| r33093 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33093 | |||
| r33094 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33094 | |||
| r33095 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all references to old I/O system (even in comments). | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33095 | |||
| r33096 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33096 | |||
| r33097 | allison++ | pdd22io_part2: | 22:06 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33097 | |||
| r33098 | allison++ | pdd22io_part2: | 22:09 | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33098 | |||
| r33099 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33099 | |||
| r33100 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33100 | |||
|
22:09
kjs_ joined
22:12
chromatic joined
|
|||
| dalek | r33101 | allison++ | pdd22io_part2: | 22:13 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33101 | |||
| r33102 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33102 | |||
| r33103 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33103 | |||
| r33104 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33104 | |||
| r33105 | allison++ | pdd22io_part2: | |||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33105 | |||
| chromatic | svk push! | 22:15 | |
| purl | svk push is a simple concept | ||
| allison | moritz: no, the pdd22io_part2 branch shouldn't build until I finish the current round of conversions (there's just no way to break this into pieces, once the interpreter switches, everything has to switch) | ||
| moritz | allison: ok | ||
| allison | chromatic: really, I planned to do this all in one commit, but somewhere along the way I added a segfault, so I'm code reviewing each file as I commit it, and then testing it on another checkout of the branch | 22:16 | |
| jonathan | segfault => epic karma win | ||
| moritz | lol | 22:17 | |
| karma allison | |||
| purl | allison has karma of 1120 | ||
| jonathan wonders where we are with the mmdcache branch... | |||
|
22:17
TiMBuS joined
|
|||
| moritz | jonathan: parrot's 'make test' worked all fine here, rakudo's 'make spectest' seemed to barf on every single file | 22:18 | |
| jonathan wonders what was committed | |||
| dalek | r33106 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33106 | |||
| chromatic | jonathan, I'm going to merge from trunk and see what happens. | 22:19 | |
| jonathan | Last time I checked, make test failed some and Rakudo's make spectest was clean. ;-) | ||
| chromatic: OK, awesome. | |||
|
22:19
Andy joined
|
|||
| jonathan waits for that before re-building | 22:19 | ||
| dalek | r33107 | allison++ | pdd22io_part2: | 22:20 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33107 | |||
|
22:22
rdice joined
|
|||
| chromatic | Had to resolve two conflicts; rebuilding now. | 22:24 | |
| Built fine; testing now. | 22:25 | ||
| dalek | r33108 | allison++ | pdd22io_part2: | 22:26 | |
| : [pdd22io] Convert all calls to old I/O system. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33108 | |||
| chromatic | Hm, t/pmc/bigint.t failed. | 22:27 | |
| moritz | chromatic: yes, that's related to kj's changes... | ||
| also fails in trunk | |||
| (the changes that forbid dollar-less registers in PIR) | |||
| chromatic | Okay, not our problem then. | 22:28 | |
| Hmm. | 22:30 | ||
| t/pmc/perl6multisub-arity....... All 13 subtests passed | |||
| t/pmc/perl6multisub-basic....... All 4 subtests passed | |||
| t/pmc/perl6multisub-tiebreak....ok | |||
| t/pmc/perl6multisub-type........ All 13 subtests passed | |||
| jonathan | If they all passed why doesn't it just say "ok"? | 22:32 | |
| chromatic | #5 0xb7b67743 in mem_sys_free (from=0x806fc28) at src/gc/memory.c:304 | ||
| #6 0xb7908bae in Parrot_Perl6MultiSub_destroy (interp=0x804f040, | |||
| pmc=0xb6e83634) at ./perl6multisub.pmc:666 | |||
| double-free | |||
| dalek | r33109 | allison++ | pdd22io_part2: | ||
| : [pdd22io] Convert HLLCompiler to use new read mode values, and to call | |||
| : 'readall' method, instead of 'slurp'. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33109 | |||
| jonathan | Just wanted to make sure... ;-) | ||
| Hmm...how can we end up calling Parrot_Perl6MultiSub_destroy twice? | 22:33 | ||
| For the same PMC? | |||
| chromatic | I don't think that's the problem. | ||
| I think it's trying to free the same pointer twice. | |||
| jonathan | Oh? | ||
| Have I done something incredibly dumb? | |||
| Entirely possible... | 22:34 | ||
| chromatic | That depends what's in the candidates attribute and who allocated each item. | ||
| jonathan | oh, what on earth... | ||
| mem_sys_free(cache); | 22:35 | ||
| I shoulda been calling Parrot_mmd_cache_destroy there for one... | |||
| SETATTR_Perl6MultiSub_cache(interp, SELF, cache) | |||
| shyould be | |||
| GETATTR_Perl6MultiSub_cache(interp, SELF, cache) | |||
| ...was I drunk? | |||
| chromatic | Let me commit the merge, and you can fix this. | ||
| jonathan | Sure. | 22:36 | |
| chromatic | We won't move back to trunk before this all passes. | ||
| jonathan | Aye. | ||
| Is make test clean apart from the known bigint? | 22:37 | ||
| chromatic | Yes. | ||
| jonathan | Great. | 22:38 | |
| dalek | r33110 | chromatic++ | mmdcache: | 22:40 | |
| : [mmdcache] Merging from trunk into mmdcache branch from r33003. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33110 | |||
| jonathan svn up's | 22:41 | ||
| Wish my C library got upset about double frees too... | |||
| chromatic | You don't get a SIGABRT? | 22:42 | |
| jonathan | Doesn't seem so. | 22:43 | |
| dalek | r33111 | allison++ | pdd22io_part2: | 22:44 | |
| : [pdd22io] Convert all calls to old I/O system in C tests. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33111 | |||
| jonathan | Otherwise I'd likely have spotted the issue in local testing. | ||
| chromatic | Fair enough. | 22:46 | |
| dalek | r33112 | allison++ | pdd22io_part2: | 22:50 | |
| : [pdd22io] Change HLLCompiler to set the filehandle's encoding using the | |||
| : 'encoding' method instead of by pushing a layer name. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33112 | |||
| chromatic | 17 items found in the latest Coverity scan from this morning! | 22:51 | |
| 12 need addressing. Very nice. | |||
| dalek | r33113 | allison++ | pdd22io_part2: | 22:57 | |
| : [pdd22io] Headerizer fix to function doc. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33113 | |||
| r33114 | allison++ | pdd22io_part2: | 22:58 | ||
| : [pdd22io] Headerizer reorder to function prototype. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33114 | |||
| jonathan | chromatic: Try 33115 | 23:02 | |
| Trying it here. | |||
| So far so good, but then it worked here before anyway... | |||
| dalek | r33115 | jonathan++ | mmdcache: | 23:03 | |
| : [rakudo] Fix epic fail at destroying MMD cache in Perl6MultiSub. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33115 | |||
| jonathan | We were actually trying to free an uninitialized pointer! | ||
| chromatic | That'll do it. | 23:04 | |
|
23:06
Alias joined
|
|||
| moritz | Unicode is impressive, my mail client can display Arabic spam correctly ;) | 23:13 | |
| jonathan | chromatic: Rakudo looks good for me here, waiting for Parrot's make test to complete, then I give mmdcache branch the thumbs up on Win32. | 23:17 | |
| moritz | jonathan: rakudo smoked fine here in the mmdcache branch too | 23:18 | |
| jonathan | moritz: Russian spam is what mostly keeps my ability to read the cyrillic alphabet alive between trips to countries that use it. ;-) | 23:19 | |
| chromatic | Excellent. | ||
| Do you have any other changes in the branch before we merge to trunk? | |||
| Your journal mentioned cache invalidation or something. | |||
| jonathan | I need to sort out cache invalidation. | ||
| Yeah | |||
| chromatic | Okay, let's do that and then merge. | 23:20 | |
| jonathan | It's a few line additoin. | ||
| I don't mind doing it post-merge. | |||
| Or I can do it now...I was sorta having a day off-ish. ;-) | |||
| chromatic | Less risk to do it on the branch. | ||
| jonathan | OK, doing it. | 23:21 | |
| purl | somebody said doing it was beyond difficult. | ||
| jonathan | Maybe for you, purl, maybe for you... | ||
| dalek | r33116 | chromatic++ | trunk: | 23:25 | |
| : [PMC] Clarified the exit condition of a loop in NameSpace PMC's | |||
| : get_pointer_keyed(). It's still a little bit confusing, but it's clearer now. | |||
| : See CID #118, from Coverity Scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33116 | |||
| moritz | jonathan: parrot's 'make test' is also clean in the mmdcache branch for me (except for bigint tests which also fail in trunk) | 23:27 | |
| purl | okay, moritz. | ||
| moritz | and now time for bed ;) | 23:28 | |
| jonathan | schlaft gut | 23:29 | |
| dalek | r33117 | chromatic++ | trunk: | 23:43 | |
| : [src] Added an assert against a NULL vtable->mro in Parrot_create_mro(). See | |||
| : CID #146 from Coverity Scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33117 | |||
| chromatic | That makes 10 open Coverity Scan issues. | 23:46 | |
|
23:49
bacek_ joined,
tetragon joined
|
|||
| dalek | r33118 | allison++ | pdd22io_part2: | 23:50 | |
| : [pdd22io] Move src/io/utf8.c higher in the compile list. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33118 | |||
| r33119 | allison++ | pdd22io_part2: | 23:51 | ||
| : [pdd22io] Check encoding and use UTF-8 utility functions if set to | |||
| : encoding attribute of filehandle is set to 'utf8'. | 23:52 | ||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33119 | |||
| r33120 | chromatic++ | trunk: | 23:56 | ||
| : [MMD] Moved a NULL check to before the first dereference, avoiding a potential | |||
| : crash. See CID #154 from Coverity Scan. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33120 | |||
| r33121 | allison++ | pdd22io_part2: | 23:57 | ||
| : [pdd22io] Add basic tests of reading/writing with UTF-8 encoding set on | |||
| : the filehandle. | |||
| diff: www.parrotvm.org/svn/parrot/revision?rev=33121 | |||