|
Parrot 1.0 Released | parrot.org | 380 RTs left! Set by moderator on 28 March 2009. |
|||
| jonathan | But anyone who cares about the length of the UUID can just go grab the string and then get it's length. | 00:00 | |
| bacek | indeed. | 00:01 | |
| dalek | rrot: r37999 | bacek++ | branches/packfile_revamp/docs/pdds/pdd13_bytecode.pod: Remove uuid_length from public Packfile attributes. string and then get it's length." (c) jonathan |
00:04 | |
| jonathan | oh noes responsibility! | ||
|
00:09
AndyA joined
|
|||
| bacek laughing evilly | 00:09 | ||
| Coke_afk hears how many slots perl gets for GSOC, preliminarily. | 00:11 | ||
| jonathan | Coke: Hopefully > 0? | ||
|
00:13
kid51 joined
|
|||
| Coke | I odn't know if it's secret, but yes. =-) | 00:14 | |
| subject to change until monday. | 00:15 | ||
| Hey! Chromium got slots. that's cheating! | |||
| Infinoid | Android probably did too | 00:19 | |
| Coke | TPF got a preliminary allocation of 10 slots. subject to change. I don't see anything that says it's a secret. | ||
| no android. | |||
| Infinoid | aaw | ||
| is the parrot foundation applying separately, or are we begging for some slots from TPF? | 00:20 | ||
| (the fact that they both have the same acronym is a bit unwieldy) | 00:21 | ||
| Coke | damint. | ||
| "I was just asked an excellent question on IRC - is it OK to share | 00:22 | ||
| these numbers with our community? | |||
| The answer is that is up to you. If it were me deciding, I would not | |||
| " -LH | |||
| ah well. =-) | |||
| so, I stress again this is PRELIMINARY. | |||
| Infinoid: pafo applied under the TPF umbrella. | |||
| "nothing will be final until monday" | 00:24 | ||
|
00:31
tetragon joined
|
|||
| jonathan | bacek: Taking sleep now, catch me tomorrow if you have any more packfile questions :-) | 00:35 | |
| night all | |||
| bacek | good night jonathan | ||
| I'll probably will have more packfile answers :) | |||
| ETOOMANYWILLS | |||
|
00:48
s1n joined
01:20
Tene joined
01:59
eternaleye joined,
acajou joined
|
|||
| dalek | rrot: r38000 | cotto++ | trunk/src/pmc/integer.pmc: [PMC] change Integer's i_add to a VTABLE function where possible |
02:06 | |
|
02:07
jdv79 left
02:09
Infinoid joined
02:48
janus joined
02:52
petdance joined
03:26
joubert joined
03:51
amoc joined
|
|||
| dalek | rrot: r38001 | coke++ | trunk/t/codingstd/c_function_docs.t: [t/docs] un TODO passing test |
03:54 | |
| Coke ponders 'make depend' | 04:04 | ||
|
04:11
TiMBuS joined
|
|||
| GeJ | what would that do? | 04:12 | |
| Coke | automatimcally updated dependency information extractable from pmcs, c, ops, perl, pir, and pasm. | 04:15 | |
| so we don't have to actively manage it anymore. | |||
| dalek | rrot: r38002 | coke++ | trunk/src/oo.c: [cage] remove unreachble code |
04:54 | |
|
05:00
dalek joined
05:01
japhb joined
|
|||
| Infinoid | bacek: ping | 05:05 | |
| bacek | Infinoid: pong | ||
| erm. Good morning? | 05:06 | ||
| Infinoid | Any objections to me making Socket.get_bool() return openness rather than createdness? | ||
| it's 10pm here :) | |||
| bacek | Infinoid: no objections. | 05:07 | |
| Infinoid | I want to rip the guts out of it, reverse the logic and use it to implement Parrot_io_socket_is_closed() (TT #547) | ||
| so get_bool will just become a negating wrapper around that | |||
| bacek | +1 | ||
| purl | 1 | ||
| Coke | isn't that a functional change? | ||
| (and therefore requires a dep-cycle?) | 05:08 | ||
| bacek | Anyway, Socket.socket returns PMCNULL in case of error | ||
| Infinoid | Coke: No, all this stuff was added post-1.0.0. | ||
| bacek | Coke: it's between releases :) | ||
| Infinoid | Anyway, get_bool will still return exactly the same thing... the "createdness" thing I mentioned above was what the documentation said it did, which isn't actually what it did. | 05:09 | |
| Infinoid is updating the doc too | |||
| Coke | docs win. | ||
| (then you're fixing a bug. whee.) | |||
| bacek | How I can run single step from Configure? | ||
| Coke | bacek: with great pain. | ||
| bacek | Coke: thank you :) | 05:10 | |
| Coke | holy crud is 'make splint' noisy. | ||
| bacek | Looks like &gen_from_def can't handle non-int defines... | 05:11 | |
| dalek | rrot: r38003 | Infinoid++ | trunk/src (2 files): [io] Implement Parrot_io_socket_is_closed(). |
05:14 | |
| Infinoid | Damn, didn't quite get the ticket closed within half an hour of its creation. :) | 05:15 | |
| *sigh* Shoulda stuck to my rule, never commit anything after 10pm | 05:18 | ||
| Coke-ZZZ | Infinoid+++ | 05:22 | |
| dalek | rrot: r38004 | Infinoid++ | trunk (2 files): [io] Build fixes from previous botched patch. Infinoid-- |
05:24 | |
| bacek | *incoming* | 05:29 | |
| bacek hides under table | |||
| dalek | rrot: r38005 | bacek++ | branches/packfile_revamp/src/pmc/packfile.pmc: Start reimplementing Packfile PMCs to generate underlying PackFile instead of using it directly. |
05:30 | |
| rrot: r38006 | bacek++ | branches/packfile_revamp/t/pmc/packfile.t: Restructure t/pmc/packfile.t. |
|||
| rrot: r38007 | bacek++ | branches/packfile_revamp/t/pmc/packfile.t: Add tests for Packfile unpacking. |
|||
| rrot: r38008 | bacek++ | branches/packfile_revamp/src/pmc/packfile.pmc: Really unpack PackFile and set internal attributes. |
|||
| rrot: r38009 | bacek++ | branches/packfile_revamp/src/pmc/packfiledirectory.pmc: Refactor PackfileDirectory PMC to use Hash as main storage. |
|||
| rrot: r38010 | bacek++ | branches/packfile_revamp/t/pmc/packfiledirectory.t: Update packfiledirectory.t to current behaviour. |
|||
| bacek moves deeper under table | 05:33 | ||
| Infinoid | go git-svn go | ||
| (small, easily reviewable patches)++ | |||
| bacek | Infinoid: It's not actual history of commits :) | 05:34 | |
| dalek | rrot: r38011 | bacek++ | branches/packfile_revamp/t/pmc/packfiledirectory.t: Remove unused test |
||
| Infinoid | are you using stgit and constantly revisioning things, like I do? | ||
| rrot: r38012 | bacek++ | branches/packfile_revamp/docs/pdds/pdd13_bytecode.pod: Remove int-based accessors from PackfileDirectory. Add get_iter method. |
|||
| rrot: r38013 | bacek++ | branches/packfile_revamp/t/pmc/packfileconstanttable.t: Update packfileconstanttable.t to properly search for ConstantTable. |
|||
| rrot: r38014 | bacek++ | branches/packfile_revamp/src/pmc/packfileconstanttable.pmc: Refactor PackfileConstantTable to use RPA and RIA for store data. |
|||
| bacek | No, just git rebase --interactive | ||
| rrot: r38015 | bacek++ | branches/packfile_revamp/src/pmc/packfileconstanttable.pmc: Implement loading of PackFile_ConstTable into PackfileConstantTable PMC. |
|||
| rrot: r38016 | bacek++ | branches/packfile_revamp/src/pmc/packfile.pmc: Invoke PackfileDirectory.set_pointer to instantiate all other PMCs. |
|||
| rrot: r38017 | bacek++ | branches/packfile_revamp/src/pmc/packfileconstanttable.pmc: Remove unused code. |
|||
| rrot: r38018 | bacek++ | branches/packfile_revamp (2 files): Generate PASM constants for PackFile_Constant types. integers for PFC_* constant types. |
|||
| rrot: r38019 | bacek++ | branches/packfile_revamp/src/pmc/packfilerawsegment.pmc: Reimplement PackfileRawSegment to use own stored data. |
|||
| bacek | One or two commits more... | ||
| Infinoid | nice, I've never tried that. | ||
| Infinoid tends to split patches the hard way | |||
| bacek | Ah. It was last commit. | 05:36 | |
| Infinoid: I commit every finished bit. And than just squash commits into single logical one. | 05:37 | ||
| (Not always unfortunately...) | |||
| Infinoid | Me too, I just keep it all in stgit | 05:38 | |
| For some reason it fits my brain patterns better than raw git ever did. | |||
| bacek | I never tried stgit. But TMWTDI works perfectly in git :) | 05:40 | |
| Infinoid notes for the logs that he's now maintaining git://squawk.glines.org/parrot-all/ in addition to git://squawk.glines.org/parrot-trunk/ | 05:41 | ||
| Yes. But third party tools don't always work perfectly together... for instance git-svn confuses stgit and vice-versa | |||
| good thing I don't run both of them within the same checkout :) | 05:42 | ||
| bacek | :) | ||
| git-svn still hangs with "Resetting to the latest refs/remotes/packfile_revamp"... | 05:48 | ||
|
05:48
Theory joined
|
|||
| cotto | bacek | 05:51 | |
| bacek | cotto: | ||
| cotto | bacek++ #packfile hackery | ||
| bacek | :) | ||
| cotto | my fingers don't look fat, but the evidence says otherwise | ||
| bacek | I think it was cat which jumped on keyboard | 05:52 | |
| cotto | ENOCAT | ||
| bacek | don't tell anyone :) | 05:53 | |
| dalek | rrot: r38020 | bacek++ | branches/packfile_revamp/t/pmc/packfileconstanttable.t: Use packfile_constants.pasm instead of "magik constants" in packfileconstanttable.t. |
05:58 | |
| rrot: r38021 | bacek++ | branches/packfile_revamp/src/pmc/packfilesegment.pmc: Add PackfileSegment.set_pointer. |
|||
| kudo: 3280d41 | pmichaud++ | build/gen_whatever_pir.pl: Whatever arguments appear to have been flipped. |
06:06 | ||
| shorten | dalek's url is at xrl.us/beog4y | ||
| cotto | The use of "whatever" as a technical term never ceases to amuse me. | 06:08 | |
| dalek | rrot: r38022 | bacek++ | branches/packfile_revamp/src/pmc/packfilesegment.pmc: Remove throwing exception from PackfileSegment.set_pointer. bacek-- |
06:11 | |
| rrot: r38023 | bacek++ | branches/packfile_revamp/t/pmc (4 files): Move common test functions into packfile_common.pir |
|||
| bacek | ok, little brake and start packing :) | 06:12 | |
| dalek | kudo: 712982a | pmichaud++ | docs/spectest-progress.csv: spectest-progress.csv update: 356 files, 10284 passing, 0 failing |
06:17 | |
| shorten | dalek's url is at xrl.us/beog5g | ||
|
06:47
silug joined
08:08
barney joined
08:12
iblechbot joined
08:23
mikehh joined
09:09
rob joined
|
|||
| bacek | Infinoid: around? | 09:18 | |
|
09:25
uniejo joined
10:00
schmalbe joined
|
|||
| cotto | anyone here conscious who knows mmd? | 10:23 | |
| bacek | mind melting dispatch? | 10:28 | |
| cotto | sounds accurate | ||
| bah. sleep | 10:30 | ||
| night | |||
| bacek | g'night | 10:32 | |
|
11:02
Whiteknight joined
|
|||
| acajou | Is Rakudo supposed to pass a spectest cleanly right now? | 12:00 | |
| acajou gets 105 failed tests... | 12:01 | ||
| jonathan | acajou: What are you seeing failing? | 12:04 | |
| I thought it passed cleanly at the moment. | |||
| acajou | perl.pastebin.com/d4d917b6f | ||
| Tried several times yesterday and this morning. Never clean, but a lot of source updates too... | 12:05 | ||
| Using git fetch / perl Configure.pl --gen-parrot / make / make spectest | 12:06 | ||
| acajou afk | 12:07 | ||
| jonathan | Odd. :-| | 12:15 | |
| I guess try a make realclean first too... | |||
| But I'm not seeing any failings like that... | |||
|
12:17
rafl joined
|
|||
| acajou | OK, I'll try that today, thanks. | 12:26 | |
| acajou waves. | |||
|
12:26
acajou left
12:44
silug joined
|
|||
| Infinoid | bacek: I am now | 13:08 | |
| mikehh | I am getting failures in codetest at r38023 - t/codingstd/copyright.t - Duplicate copyright statement found in 12 files: | 13:13 | |
| I had a look at the files and I am not sure if we can remove the duplicates | 13:14 | ||
| Infinoid | That's a work in progress | ||
|
13:14
joubert joined
|
|||
| mikehh | The imcc stuff has Copyright (C) 2002 Melvin Smith <melvin.smith@mindspring.com> then Copyright (C) 2003-2009, Parrot Foundation | 13:15 | |
| others are found in prunt statements and POD | 13:16 | ||
| s/prunt/print/ | 13:17 | ||
| some are relayed to GNU copyright and things like that | 13:18 | ||
| s/relayed/related/ | |||
| for examp;e in Configure.pl at line 31 - # Copyright (C) 2003 Free Software Foundation, Inc. | 13:20 | ||
| Infinoid | Yeah. From TT #543, it looks like the test was untodoed accidentally... but it's still a work in progress | 13:21 | |
| mikehh | That I can see - cotto at r37939 - maybe the test should only test the first couple of lines of the file - not further? | 13:23 | |
| Infinoid | that sounds reasonable | 13:24 | |
|
13:46
gryphon joined
13:52
Tene joined
|
|||
| Coke-ZZZ | mikehh: no. | 14:13 | |
| Coke | right now there's a bug in the test that it acts differently in 5.8 vs. 5.10 | 14:14 | |
| Infinoid | Coke, are there any cases where the test should be looking for copyright tags somewhere other than, say, the first 5 lines of the file? | 14:32 | |
| Coke | the cases where the copyright is at the end of the file? =-) | ||
| Infinoid | Fair enough. :) | ||
| Coke | I'm honestly curious why it behaves diffeently. | ||
| I think a passing todo is better than a failing, though. | 14:33 | ||
| Infinoid | Yeah, the portability issue should be fixed too. I just thought mikehh's idea was an interesting one | ||
|
14:59
riffraff joined
15:05
Theory joined
15:06
tetragon joined
15:16
joubert joined
|
|||
| bacek | oh... 1:30AM here... | 15:36 | |
| bacek hates pure C even more... | |||
| Infinoid: good... erm... evening? | 15:37 | ||
| Coke | Infinoid: Parrot_str_append - should *a be ARGIN ? | ||
| Infinoid | bacek: good morning :) | 15:38 | |
| bacek | I'm not sure is it already morning or still night... | 15:39 | |
| dalek | rrot: r38024 | bacek++ | branches/packfile_revamp/src/pmc/packfilesegment.pmc: Strip PackfileSegment to bare minimum. |
||
| bacek | here we go | ||
| rrot: r38025 | bacek++ | branches/packfile_revamp/src/pmc/packfileconstanttable.pmc: Reindent code in packfileconstanttable.pmc |
|||
| rrot: r38026 | bacek++ | branches/packfile_revamp (9 files): Initial implementation of writable Packfile PMCs. Every Packfile*.pmc responsible for creating "native" PackFile segment in VTABLE get_pointer. Those pointers gathered by PackfileDirectory and added into PackFile_Directory for serialization and memory management. |
|||
| bacek | Incoming :) | ||
| bacek | Infinoid: r38026 | ||
| 10 hours of fight with packfiles... | 15:40 | ||
| Infinoid | Coke: doesn't it append to *a? | ||
| bacek: I'm in TZ=PST8PDT :) | |||
| bacek | Infinoid: ah. Yesterday morning was beautiful. I still remember :) | 15:41 | |
| Coke | Infinoid: the docs say "don't use the one you pass in, we return a value, use that. | 15:42 | |
| splint complains. | |||
| "Fresh storage result (type STRING *) not released before assignment:" | |||
| Infinoid | The docs are wrong, then. | 15:43 | |
| a->bufused += b->bufused; | |||
| a->strlen += b_len; | |||
| a->hashval = 0; | |||
| return a; | |||
| Coke | hurm. but we do potentially modify it, I suppose. just wondering how to make make splint happy | ||
| Infinoid | hmm. where was that splint warning, exactly? | 15:44 | |
| is that in the utf16 upgrade stuff? | 15:45 | ||
| nopaste | "coke" at 72.228.52.192 pasted "splint output" (139725 lines) at nopaste.snit.ch/16221 | 15:46 | |
| Coke | it's a little big. | ||
| Infinoid | oh, the warning is in the caller | 15:47 | |
| Splint doesn't understand that the old value will be GC'd, I guess | |||
|
15:53
Psyche^ joined
|
|||
| Coke | hurm. my nopaste still hasn't completed locally. =-) | 15:53 | |
| Infinoid | still "transferring data" here too | 15:54 | |
| Coke | you have access to feather? | ||
| ackshully... | |||
| Infinoid | yeah | ||
| my browser is reading thousands of " src/nci.c:11750:9: Fresh storage temp_pmc created" lines at the moment | |||
|
15:55
shucho joined
|
|||
| Coke | feather now has ~coke/splint.out.gz | 15:56 | |
| er, make that ~coke/public_html/splint.out.gz | |||
| (which is presumably available via the web) | |||
| Infinoid | bacek: (r38026) Nice patch, I think it's a really interesting approach, and you're going to shortcut whole bunch of effort this way. How well does it work? :) | 16:01 | |
| ^a | |||
|
16:01
joubert joined
|
|||
| bacek | Infinoid: t/pmc/packfile.t It can produce simple PBC. And even parse it after :) | 16:02 | |
| Infinoid | bacek++ | ||
| bacek | I actually spend whole day trying to do it in different way - properly reuse available PackFile_* functions... EPIC FAIL. | 16:03 | |
| spent | |||
|
16:03
jrockway joined
|
|||
| Infinoid | Yeah, that was the route I was going down. It was time-consuming. | 16:04 | |
| bacek | And it's almost impossible... Even serialized PackFile_Directory depends on PackFile. | 16:06 | |
| Infinoid | Coke: That page finally finished loading | 16:07 | |
| Whiteknight | cotto: ping | ||
| bacek | need sleep | 16:10 | |
| afk # zzzz.... | |||
| Infinoid | Coke: I think quite a lot of that could be cleared with a few annotations to teach splint about the gc | 16:18 | |
| Infinoid is reading www.splint.org/manual/html/sec5.html, section 5.2.5 looks promising | |||
| unfortunately I'm out of parrot tuits until probably monday night | 16:21 | ||
| particle- | sailing? | 16:23 | |
| purl | sailing is pretty careful about wind too. ;) | ||
| Infinoid | among other things, yes :) | 16:24 | |
|
16:41
flh joined
16:45
joubert joined
17:27
contingencyplan joined
|
|||
| dalek | kudo: 029aa51 | (Moritz Lenz)++ | (2 files): bring Hash.reverse up to date; test S32-hash/reverse.t |
17:35 | |
| shorten | dalek's url is at xrl.us/beoij2 | ||
| dalek | kudo: 7b2c47c | (Moritz Lenz)++ | (2 files): Implement Hash.push, and add S32-hash/push.t to spectest.data type() from is_deeply in Test.pm |
||
| shorten | dalek's url is at xrl.us/beoij4 | ||
| dalek | kudo: c44a05b | (Moritz Lenz)++ | t/spectest.data: we pass the new S09-typed-arrays/hashes.t |
||
| shorten | dalek's url is at xrl.us/beoij6 | ||
| dalek | kudo: 79aba97 | (Moritz Lenz)++ | README: [README] reworked paragraph on rakudo.org |
||
| shorten | dalek's url is at xrl.us/beoij8 | ||
|
17:45
Tene joined
|
|||
| dalek | rrot: r38027 | whiteknight++ | trunk (2 files): Finally resolve RT#46719. One of the tests was using ge instead of isge, which miraculously produced a passing test using the old initialized register values. Fixing the test enables us to avoid initialization |
17:49 | |
| cotto | Whiteknight, pong | 18:34 | |
|
18:44
tetragon_ joined
|
|||
| Whiteknight | hey cotto, I wanted to ask what the status of the UnionVal stuff was? | 18:49 | |
| nopaste | "pmichaud" at 72.181.176.220 pasted "fun with comparisons" (22 lines) at nopaste.snit.ch/16228 | 18:51 | |
| cotto | Whiteknight, most PMCs have been converted. There are a couple of questionable status that I haven't done (Pointer, Ref) that are deprecated but still in use. | 18:53 | |
| I don't think there are any blockers to finishing the task. | 18:54 | ||
| The remaining tasks are: | 18:55 | ||
| * fix the remaining PMCs which use the macros (Pointer, Ref, SharedRef, Namespace, maybe others) | 18:56 | ||
| * ack for any other instances of the macros and try to DTRT with them | |||
| * remove the macros from pobj.h | |||
| * done! | 18:57 | ||
| For some reason, PMC_pmc_val appears a bunch in sharedref.c and ref.c. I haven't looked into why, but it's probably not a difficult fix. | 18:59 | ||
| If you'd like to help, it should parallelize well. | 19:00 | ||
|
19:05
tetragon joined
|
|||
| Coke | cotto: ref is up for removal. don't fix it, kill it. | 19:06 | |
| cotto | with fire? | 19:08 | |
| Coke | nuke it from orbit. | ||
| nuke it from orbit is <reply>It's the only way to be sure. | |||
|
19:11
acajou joined
|
|||
| acajou | Hi all. So, since yesterday, I can't get a clean spectest... My latest attempt (with a source tree from this AM): perl.pastebin.com/d8b9e694 | 19:13 | |
| Is no one else getting errors? | |||
|
19:19
altious joined
19:34
japhb joined
|
|||
| dalek | rrot: r38028 | whiteknight++ | trunk/src (2 files): Fix Parrot_str_to_int to detect overflow and throw an EXCEPTION_ERR_OVERFLOW. |
19:36 | |
| Whiteknight | acajou: what platform are you on? | ||
| acajou | gcc version 4.2.4 (Ubuntu 4.2.4-1ubuntu3) | ||
| (Kubuntu 8.04.2) | 19:37 | ||
| DId git fetch / make realclean / make / make spectest | |||
| (and perl Configure.pl --gen-parrot b4 make) | |||
| I've tried about 6 times since yesterday, and it's never been clean. | 19:38 | ||
| jonathan | acajou: Any chance you can run one of the tests directly and nopaste the output here or (perhaps better) on #perl6? | 19:39 | |
| acajou | 10-4 | 19:44 | |
| purl | 6 | ||
| acajou | eg perl.pastebin.com/d341958bd | 19:47 | |
| jonathan | acajou: Ah, but if there is a .rakudo fiel you should run that, not the .t... | 19:48 | |
| Otherwise can't tell which tests are failing that are not expected to be. | 19:49 | ||
| dalek | rrot: r38029 | cotto++ | trunk/src/pmc (2 files): [PMC] replace some code that uses Ref |
||
| acajou | OK, sorry. | ||
| Coming right up. | 19:50 | ||
| perl.pastebin.com/d29e19c3b | 19:52 | ||
| jonathan | acajou: You certainly have no local patches? | 19:54 | |
| This is...odd. | 19:55 | ||
| acajou | None whatsoever. git clone and then only git fetch. I'm just the peanut gallery here, excited that Perl6 looks almost here... | 19:56 | |
| Maybe I should nuke my source three and re-clone... | |||
| acajou AFK. | 19:59 | ||
| jonathan | acajou: That would eliminate the possibility of contamination, for sure. | 20:03 | |
| particle- | unless there's an installed parrot in the way, still | 20:06 | |
| dalek | rrot: r38030 | cotto++ | trunk (9 files): [PMC] remove Ref and SharedRef, bump PBC_COMPAT |
20:09 | |
|
20:21
rob joined
|
|||
| jonathan | particle-: I hadn't considered that one, good point. | 20:36 | |
|
20:45
contingencyplan joined
20:48
darbelo joined
|
|||
| cotto | Infinoid, ping | 20:50 | |
| Infinoid, unping | 20:57 | ||
| jonathan, ping | |||
| mikehh | I keep my parrot and rakudo dirs separate - I generally start with parrot and do a amke realclean, svn up, perl Configure.pl+, make world, make html, make smoke and then- | 20:59 | |
| jonathan | cotto: pong | ||
| cotto | jonathan, t/pmc/sockaddr.t is confusing | 21:00 | |
| It looks like it should be testing the SockAddr PMC, but it creates a Socket. | 21:01 | ||
| mikehh | in rakudo: make realclean, git pull, perl Configure.pl --parrot-config=../parrot/parrot_config, maje, makr test, make spectest | ||
| cotto | Could you add a quick note explaining why it does that? | ||
| jonathan | cotto: No, because (a) I didn't write it and (b) I don't understand sockets even if I did write it. ;-) | ||
| cotto: bacek will probably have much more helpful answer | 21:02 | ||
| cotto: My best guess though, is the design changed. | |||
| cotto | ok | ||
| jonathan | When the ops went away, SockAddr was created by some method instead. | ||
| cotto | svn blamed you for most of it, but I didn't dig very deeply. | ||
| jonathan | Yes, I applied the patch that added it. | ||
| I more reviewed the C code and ensured the tests passed. And I think it looked sane at the time... | 21:03 | ||
| cotto | gotcha | ||
| jonathan | Anyway, bacek wrote the patch - it was before he got the commit bit though. :-) | ||
| cotto | I'll ask him about it then. | ||
| seen bacek | 21:04 | ||
| purl | bacek was last seen on #parrot 4 hours, 53 minutes and 10 seconds ago, saying: afk # zzzz.... | ||
| jonathan | Cool. Sorry to not be much more helpful. | ||
|
21:04
Whiteknight joined
21:07
msmatsko joined
21:11
iblechbot joined
|
|||
| dalek | rrot: r38031 | cotto++ | trunk (11 files): [PMC] more cleanup from the Ref/SharedRef removal |
21:15 | |
| rrot: r38032 | whiteknight++ | trunk/src/pmc/hash.pmc: remove a PMC_struct_val from src/pmc/hash.c |
21:25 | ||
| acajou nuked his repo and re-cloned, running spectest now. | 21:36 | ||
| Looks like it'll be clean(er), already apssed S02 without error. Not sure what had happened... | 21:38 | ||
| dalek | rrot: r38033 | cotto++ | trunk/src/pmc/namespace.pmc: [PMC] remove PMC_pmc_val from NameSpace PMC |
||
| jonathan | acajou: Maybe some weird makefile not cleaning something issue... | 21:40 | |
| acajou nods. | 21:43 | ||
| acajou would like to take a moment to express his fondness for the "-i" command line argument from Perl5, as S19 has its future as uncertain... | 21:45 | ||
| acajou wonders why Kopete doesn't wrap long lines... | 21:46 | ||
| dalek | rrot: r38034 | cotto++ | trunk/src/pmc/lexinfo.pmc: [PMC] removed unneeded PMC_pmc_val usage |
21:48 | |
| rrot: r38035 | cotto++ | trunk/src/dynpmc/dynlexpad.pmc: [dynpmc] switch DynLexPad to ATTRs |
21:55 | ||
| acajou | All tests successful | 21:56 | |
| purl | ship that sucker | ||
| particle- | we ship every month, purl. | ||
| purl | particle-: what? | ||
| jonathan | acajou: Ah, phew. :-) | 22:00 | |
| GeJ | Good morning everyone | 22:01 | |
| dalek | rrot: r38036 | cotto++ | trunk (5 files): [PMC] PMC_pmc_val is history |
22:02 | |
| acajou | :) | ||
|
22:02
Limbic_Region joined
|
|||
| dalek | rrot: r38037 | whiteknight++ | trunk (2 files): update src/pmc/pointer.pmc to use ATTRs instead of PMC_struct_val. Requires a change to the nci thunk generator. |
22:08 | |
|
22:11
darbelo left
|
|||
| cotto | Whiteknight++ #I was hoping to avoid doing that. | 22:13 | |
| Although I'd have used a VTABLE in the generated NCI code. | 22:14 | ||
| dalek | rrot: r38038 | whiteknight++ | trunk/src/pmc/pointer.pmc: add a custom destroy method to the Pointer PMC, so it can free the new memory that it allocates |
22:15 | |
|
22:17
clunker9_ joined
|
|||
| dalek | rrot: r38039 | cotto++ | trunk (2 files): [PMC] PMC_str_val is history |
22:18 | |
| Whiteknight | cotto: yeah, a vtable might be a little better | 22:21 | |
| probably should have just used get_pointer I guess. want to rework it? | 22:22 | ||
| cotto | Sure. It's an easy fix. I'll just have to remember to build without jitcapable. | ||
| Whiteknight | oh, I didn't do that. I guess I haven't tested it properly then | 22:23 | |
| cotto | We'll find out how it works soon enough. ;) | ||
| While that's going, do you want to tackle PMC_struct_val? | 22:24 | ||
| There aren't many instances of it left, so I don't think it'll take very long. | |||
| PMC_int_val should be easy too. You could probably take them both out with a fairly small commit. | 22:25 | ||
| Whiteknight | I was working on PMC_pmc_val in the Namespace PMC, but am running into problems | 22:26 | |
| cotto | I got that already. | ||
| Whiteknight | oh okay, that makes it easy then | ||
| I'll take PMC_struct_val? | |||
| cotto | go for it | ||
| and svn up frequently ;) | 22:27 | ||
| Whiteknight | Yeah, lesson learned | ||
| cotto | make svn up yours | 22:30 | |
| dalek | rrot: r38040 | cotto++ | trunk (2 files): [NCI] use VTABLE functions instead of twiddling with PMC internals directly |
22:41 | |
| bacek | good morning | 22:42 | |
| cotto: (sockaddr.t) Previously it tests "sockaddr" op. But nowdays "sockaddr" is just method on Socket | 22:43 | ||
| Whiteknight | good morning bacek | 22:50 | |
| bacek | good morning Whiteknight | ||
|
22:54
eternaleye joined
|
|||
| acajou waves. | 23:01 | ||
|
23:01
acajou left
|
|||
| cotto | Whiteknight, I'm working on PMC_int_val. | 23:06 | |
| Whiteknight | okay, I'm about to commit PMC_struct_val, just running a quick coretest to make sure I'm not acting retarded at the keyboard | 23:07 | |
| cotto | sweet | 23:08 | |
| Tests are nice like that. | |||
| Whiteknight | okay, committed. What's next? | 23:10 | |
| cotto | Just PMC_int_val, although it's used in some jit code. | 23:11 | |
| dalek | rrot: r38041 | whiteknight++ | trunk (6 files): kill PMC_struct_val. kill it dead |
||
|
23:12
kid51 joined
|
|||
| cotto | nm. It's used in a switch *after* a break. It's unreachable. | 23:12 | |
| Whiteknight | yay! dead code | ||
|
23:22
eternaleye joined
23:29
eternaleye joined
23:32
tetragon_ joined
23:46
tetragon joined
|
|||
| cotto | Whiteknight, it looks like 38037 also requires some jit code to be fixed. | 23:56 | |
|
23:58
joubert joined
|
|||