% iblechbot has joined #parrot or maybe i'm just retarded, can't even get the windows binary to work properly... % Ademan has left Ademan!~dan@h-67-101-102-38.snfccasy.dynamic.covad.net * Tene working on placeholder vars. r27374 | fperrad++ | trunk: : [Lua] : - implement bytecode loader in a PMC diff: http://www.parrotvm.org/svn/parrot/revision?rev=27374 Hm. Looks like I've got a very basic version of placeholder vars working, but it doesn't do lexicographic ordering And I think there were a few other details of placeholder vars that I really should pay attention to. No additional spectests fail, and it looks like four more spectests compile. Tene++ purl: rt? rumour has it rt is just RT (http://bestpractical.com/rt) or (:rt3) or (: rt bugs) or Obra's trouble ticketing system or the first IBM RISC workstation (http://www.contrib.andrew.cmu.edu/~shadow/ibmrt.html) or the bombsquad or the Right Thing or very very capable and open-source or an application framework that bundles a ticketing system or obra's baby or SOOOO slow :-S or email mailto:perlbug-owner@perl.org for access Let's see if I can find tests of placeholder vars in the pugs repo... % Ademan has joined #parrot There are no explicit tests of placeholder vars in the pugs test suite. another TODO task for Auzon++ (unless somebody else does it in the mean time) I can't quite find any documentation on where I should be adding tests for this. where are they specced? S06 maybe t/spec/S06-signatures/placeholders.t ? So I should make a new test and commit it to pugs. aye or I can do it if you want me to Sure, that would be great. Oh, there's a $:named form. Apparently not used in the pugs test suite at all. That should be easy. Tene: the $:named is very new Tene: I added S06-signature/positional-placeholders.t % jan has left jan!~chatzilla@89-253-66-101.customers.ownit.se smartlinks are still missing :( am I right in assuming that a sub without a signature can have placeholders? No. Wait, I misread that. A sub with a signature *cannot* have placeholders. that would be sub foo () { ... } "Also, it is illegal to use placeholder variables in a block that already has a signature, because the autogenerated signature would conflict with that." but sub foo { ... } doesn'T have a signature, right? just: sub foo { ... $^a } Right. ok because the spec only mentions bare blocks Oh, rakudo doesn't have : allowed as a twigil. How new is that? 1 or 2 months Okay. does rakudo do any LTM? Not yet. Hm. Separate file for named and positional placeholders? Or just rename test to placeholders.t? separate files intentially So then a third file to test that they both work in the same sub? if that's supported, yes Okay. but the rakudo effort showed that test are useless as a start if they are too complicated so I like to keep the more complicated ones in separate files that are just completely skipped as a start Okay. r27375 | paultcochrane++ | trunk: : [lua] Keep c_parens coding standard test happy diff: http://www.parrotvm.org/svn/parrot/revision?rev=27375 Tene: may I charge you with the task of fudging the placeholder tests once your patch is accepted? moritz: they need to be fudged? Tene: I don't know ;-) Tene: since I have no idea what your patch can do, and what don't I'm in no position to assess that I kept them simple, so I hope it's not necessary No, not necessary. good My policy so far is to just commit as long as I can verify that it doesn't break anything. r27376 | tene++ | trunk: : Basic support for placeholder vars in rakudo. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27376 % iblechbot has left iblechbot!~iblechbot@ppp-62-216-200-127.dynamic.mnet-online.de rakudo gets this wrong: multi sub t ($a) { say 'one positional' } multi sub t (:$a) { say 'one named' } t(1); purl: rakudobug? rakudobug is mailto:rakudobug@perl.org Jonathan will be delighted ;) % mj41_ has joined #parrot Well I've been delighted with his work lately, so it's a fair trade. ;-) % mj41 has left mj41!chatzilla@pc-jurosz.ro.vutbr.cz % mj41_ is now known as mj41 The problem with working at night is that I can't get immediate feedback on things like the :-whitespace thing. Oh, maybe that's discussed in the spec what :-whitespace thing? many of the tests have the following: $obj.meth:{here is a block}; STD.pm and rakudo's grammar both require whitespace after the : there. So it's a parse failure. hurm S06:613 Invocants may also be passed using the indirect object syntax, with a colon after them. The colon is just a special form of the comma, and has the same precedence: That's not the same. That's like: I don't think a comma needs a space after it meth $obj: {here is a block} you're right look for /legal forms above into a list operator by/ dunno what line that is in S06? S12 Sorry. I see space after the : in all of those examples. in S06:2425, is that the same construct? there's an exxample &substr.assuming:str($text):len(Inf); ah, but it's not a block in that case The rule in STD.pm is in methodop: | ':' That also doesn't have a , between :Str and :len in the example right above that, which I think isn't allowed. * moritz isn't competent enough to answer the question :( % Zaba has left Zaba!~zaba@ip102.148.adsl.wplus.ru Thanks for the help, moritz. :) % Zaba has joined #parrot % allison has left allison!~chatzilla@dsl-241-25-20.telkomadsl.co.za Okay, time for bed. % purl has left purl!purl@sentient.life % purl has joined #parrot % wknight8111 has joined #parrot % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net % mire has joined #parrot % iblechbot has joined #parrot % wknight8111 has joined #parrot % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net % cosimo has left cosimo!~cosimo@pat-tdc.opera.com % ask has left ask!~ask@pat-tdc.opera.com % cosimo has joined #parrot % claes has joined #parrot % claes has left #parrot % ask has joined #parrot % wknight8111 has joined #parrot % jq- has joined #parrot % jq- has left jq-!~jquelin@merlin.mongueurs.net % tetragon has left tetragon!~seneca@216.126.67.44 % jan has joined #parrot % gryphon has joined #parrot % masak has joined #parrot % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net how up-to-date is languages/perl6/docs/STATUS? it was last updated on 2008-04-17 % Psyche^ has joined #parrot has anything happened since in rakudo that was not entered? grammars were implemented, they aren't mentioned tehre I would change it, but (1) I'm not sure I'm the right person, (2) I'm not sure I have a commit bit you don't have a parrot commit bit ;) but you can skim over the last commit logs... no. if there is such a thing, I don't have it build a patch STATUS is probably out of date and send it rakudobug@perl.org moritz: that I can do lots has happened in rakudo since STATUS was last written pmichaud: I see a point in keeping it up to date sure, I agree good :) nobody will reject helpful documentation patches ;) * masak gets on with it % Patterner has left Patterner!~Psyche@e177238175.adsl.alicedsl.de % Psyche^ is now known as Patterner I updated STATUS a little while ago, but stuff has happened since then, it's certainly out of date. :-) I'll take a risk and start reading diffs from 2008-04-16 could just read the logs svn log -r '{2008-04-16}:head' % mire has left mire!~Frodo@252-172-222-85.adsl.verat.net pmichaud: sorry, meant logs that's what I'm doing doing is better than talking thanks, purl my pleasure masak and copying verbatim from the feature additions I find hard to go wrong doing that, I figure Yeah, though I tend to implement a given feature over several smaller patches, to get more karma^W^W^Wmake it easier to see what each change goes. s/goes/does/ (smaller patches)++ (more karma for jonathan)++ er, I guess the last is just jonathan++ :-) Jonathan: I see that. but still the new features seem to be on about the appropriate chunk level for STATUS karma jonathan jonathan has karma of 521 as long as it's not "junk level" ;-) Jonathan: logs say WHENCE is (partially) implemented masak: Partially, yeah. are others, like WHAT and HOW also implemented? Enough for the common use cases. STATUS doesn't say WHAT is implemented. There's implimentedish, but there's more to do there. that's what I'm asking ;) Does WHAT return the Right Thing? for many things we have "partial implementations" as in, object not a string? so it's hard to put everything into a "yes/no" box. I forget. WHAT returns a protoobject. (in the implementation) Ah, OK, good. I think that HOW also returns the class metaobject.... but I'm not sure of that. Then WHAT is done. HOW currently returns the Parrot class. right I've not made my mind up how right that is yet. :-) we should put a note in STATUS saying what "implemented" means I'll write WHAT, HOW, WHENCE then I do know we have to attach ^method's to the metaclass. there, mail sent pmichaud: have a couple of minutes to look at a ticket for me? in a couple of minutes, yes Okay, thanks. http://rt.perl.org/rt3/Public/Bug/Display.html?id=53804 I was about to look at that one -- I suspect the test is wrong but need to re-read the synopses to check purl: parrot logs? wish i knew, tene I saw the conversation in the scrollback :-) pmichaud++ # uses eyes for reading Tene: http://irclog.perlgeek.de/parrot/today % peepsalot has joined #parrot If that's the indirect object call thing - I'm not sure which is right, but even if you tweak it to have a space, I think it still gives a parse error. Jonathan: it's not inderect object call. Indirect method call syntax is spelled like: meth $obj: $arg1, $arg2; Ah, OK. karma more karma for jonathan? more karma for jonathan has karma of 1 (more karma for jonathan)++ this is for something like $foo.bar: args (karma karma karam karma karma baked beans karma karma and karma)++ Ah, OK, that syntax. STD.pm seems to require a space after the colon As mentioned in the ticket. however, S02 has for @x.grep:{ 0 <= $_ < all(@y) } {...} Hm. which may just be a typo in S02, also istr that at one time :{...} was a special syntax for passing subs. But I think that's since disappeared. s/subs/closures % davidfetter has left davidfetter!~chatzilla@start.fetter.org what's the rationale for forbidding :{} w/o a space in between, again? I'm not sure there is a rationale ;) I don't know that the required space is specifically for { } it may have been to prevent things like $foo.bar:a(1) masak: as written, STD.pm requires a space for *any* args. not just args which happen to be closures. ah anyway, I don't have an answer at the moment. My suggestion would be to add a space in the tests I'd suggest to wait for TimToady, because that affects quite many tests I also question whether .min and .max remain the same since we now have infix: and infix: so it's [min] @list? Yup. Reduction operator. well, that's one way to do it. just because we have [min] @list doesn't mean we can't have @list.min :-) Not sure if that means no min method for lists though... It *is* Perl. TMTOWTDI. ;-) but I think the minmax stuff in S29 predated the reduction operator *preceded does anyone else besides me think that rt#53808 belongs in the 'perl5' queue? % rdice has joined #parrot the elsif guy? I think he probably meant perl5, but he's going to get the same answer from either crowd, neh? No point in yelling at him about the wrong queue on top of that. oh, I wasn't going to remark on it -- I was just going to move it off of the perl6 queue I suppose I could silently mark it resolved, also *cough* spam queue *cough* as long as it's not in the parrot queue, I don't care. =-) * pmichaud moves it to the parrot queue, just to irk Coke. (just kidding :-) (irking coke)++ I think RT#39329 can be closed -- anyone disagree? I am so irked about my weight loss program, you can't hurt me. =-) pmichaud: is the issue leo mntioned resolved too? Coke: I'll double-check 3 tickets to go. looks like a few were missed in integer.pmc wonder if we need to be more thorough about testing such things. the problem with RT#39329 is that it's kinda of like trying to prove a negative there's not really a definitive way to know that everything has been converted, short of reviewing all of the .pmc code welp, for now, just comment that you found more cases in integer, and leave it for someone else to finish. I was going to go ahead and fix the cases I found :-) turns out that RT#53108 was the same problem as RT#39329 pmichaud++ % IllvilJa has left IllvilJa!~jilves@emea-netcache1.oracle.co.uk actually, there's some oddity in integer.pmc so I'll just comment on it and let someone else look at it. pmichaud-- ! ;) % ask has left ask!~ask@pat-tdc.opera.com RT #39132 should be pretty easy to close. (now) % iblechbot has left iblechbot!~iblechbot@ppp-62-216-197-145.dynamic.mnet-online.de (followup on :{...}) :{...} is an adverbial closure so the tests have it correct. it gets parsed by (at least, I think colonpair handles it) that fits nicely. <-- this is an adverbial closure. I would note that you can write grep:{...} {...} because :{...} is adverbial, but grep: {...}, {...} requires the comma since it's just an argument er, actually, the first form probably doesn't work either...hmm... % Coke has left Coke!~coke@cpe-72-228-52-192.nycap.res.rr.com 'cause it won't parse the second {...} as args since it only can parse the adverb after deciding grep was out of args, i think @args.grep:{...} is fine though, since the list is on the left TimToady: So does @args.grep:{...} parse under STD.pm? {...}.grep:{...} # seems confusing % uniejo has left uniejo!~uniejo@langebro.adapt.dk I just removed every occurence of \w:{ in the test suite under spec/ moritz: ...? * purl quietly listens while the crickets chirp \w:{ ? pmichaud: like occurences of @list.grep:{...} oh pmichaud: s/(\w):{/\1: {/g I think that may be wrong, now, since TimToady reminded me about adverbial closures I looked over all occurences and they were all map:{block} and grep:{block} and the like so @list.grep:{...} is the same as @list.grep :{...} ? map is defined as our List of Capture multi method map ( @values: Code *&expression ) why should the expression be adverbial? the *& is what allows it but is it forced to be adverbial? no so, in @list.grep:{...} the :{...} is parsed by the subrule in then the whitespaces aren't wrong, at least ;-) otherwise grep: {...} wouldn't work well I was thinking adverbs in expect_term, but I see it's ambiguous... er, expect_term, right (I miswrote) but there is the difficulty of distinguishing from infix:{...} I'll have to think about that some, I guess.. anyway, rakudo parsefails because it doesn't have * on terms anywhere yet well, technically, the :{} is not on a term, but on the prior operator, which happens to be .grep mmm hmmm. it tends to skip the intermediate term in cases like 0..10:by(3) but we've got the same difficulty there, if someone writes 0..$max:by(3), as to whether the identifier eats the adverbial or whether we have to require a space there right. I think when we originally discussed it I was thinking that infix:() etc is specially recognized but I think we generalized to ident:() at some point without recognizing the difficulty so now we have "use Perl:<...>" for instance, and that's not because Perl is special cased like infix would be so maybe we will have to change grep:{...} to grep :{...} or grep: {...} hrmm... can rakudo grammars have attributes yet? particle: as in ... ? particle: gcc-4.2.0-3.aix5.3.ppc.rpm grammar Perl; has StrPos $.ws_from; paco++ particle: no, that doesn't quite work yet. % [1]JordanG has joined #parrot * particle just did a diff -u between STD.pm and grammar.pg i'm studying the output I'm not exactly sure how that would work, anyway -- at least not as PGE currently models grammars at some point we're going to have to support attributes, roles, classes, constants... protos of course % JordanG has left JordanG!~JordanG@witnet.wit.edu % [1]JordanG is now known as JordanG seems like huge changes to pge are required if I focus on the totality of everything we'll have to support "at some point", then I think we don't make much progress. no, PGE is just a fancy compiler. It doesn't have to understand all of that other stuff. okay, pmichaud. argggh stupid bot Stupid is as stupid does. no, PGE no, PGE? PGE? hmmm... PGE is just a fancy compiler. It doesn't have to understand all of that other stuff. er. right. no, PGE is the Parser Grammar Engine okay, pmichaud. anyway, as I said, PGE really don't have to worry about attributes, constants, classes, etc. the core of PGE simply compiles regex syntax it's Rakudo that will handle the other items ok. point. but I guess it will have to support some more modifiers eventually ;) the issue at the moment is that PGE conflates grammars with match objects in order to get inheritance working it does need to support hashes ...hashes? maybe ident:{...} can always parse as ident :{...}, since the run-time analog of ident: is actually ident:[$a, $b], not the curly form... I think conflating grammars with match objects is a necessity--that's what STD5 does right now. it also happily conflates match objects with cursors :) TimToady: oh. Okay, I'm glad to see I was on the right track there, then and all of the fates and peeks and such are now communicated through the unified "match state" cursor however, it does make me wonder about something like has StrPos $.ws_from; I thought PGE was still sub based no, PGE has been method-based for a very long time grammars have to be able to send additional info along through the match state argggh (I need to stop writing "no, ") though STD5 cheats and pulls that down into Cursor5 because I don't have cloning in p5 :) method-based pge was the only way I could get inheritance to work reliably but it's been that way since, oh, 2005 or 2006. cool, I've been mistaken for a goodly long time then! :) I blame my surgery. :) I blame a lack of documentation. :) % Eevee has left Eevee!~eevee@c-67-160-3-54.hsd1.wa.comcast.net TimToady: btw, we received the books back okay, many thanks. It is greatly appreciated. well, if only I were omniscient, the problem wouldn't arise... the problem with demigods is that they're only demiomniscient... * TimToady wonders what a hemidemisemigod would look like... 0.125`god I suspect pmichaud, and in actions.pm, I can't get a straight way to get at the length of a match which makes adressing #52276 difficult "length of a match"? you mean "number of children nodes"? $/.chars ? we don't use this word "length" any more :) arf, I keep using length I meant numbers of matches cognominal: +@($/) +$/ or $/.elems or ... (not +$/) right that would integerize the string (I keep making that mistake also :-) actually, it numifies the result object which might not be a string. :-) I keep making that mistake too. @() should work in a numeric context which reminds me, is there a convenient method for getting the text associated with a match object (even if a result object has been set)? istr we discussed .text at some point but it was never formalized/codified/officially adopted as long as we have .from and .to we can probably have something that returns the appropriate substr, and .text seems as good as anything I've been using .item in STD5, but that's because I have my head in the sand re result objects right, but .item returns the result object, yes? yes okay, just checking. i'm still here, pmichaud. does .list still return the positional captures, and .hash the named ones? yes excellent. I'm wanting to make that change to PGE. say +@($/); # gets me Null PMC access in type() since those are just the contextualizers @ and % in theory we have .slice too but I'll not insist on it (yet) TimToady: Thank you. :-) cognominal: in NQP or Rakudo? if in actions.pm, say requires the () oops I always forget it yes, I'm thinking of adding rudimentary listop parsing to nqp, since people always forget the parens. but that does complicate things a bit. it is just a question of warming up to a language. when I program in PIR, I put semicolon everywhere I don't quite understand why elements in a match is needed for [], though. I suspect the array composer isn't coded right if that's the case. what's this "Cannot put to non-PIO PMC" stuff about? Infinoid: seems to be a problem with the makefile generation. because it is the comma operator with 0 or 1 arguments so there is no comma visible well, with 0 arguments there's no comma operator. nor is one needed even with 1 argument there's no comma operator that's the problem in rakudo, [ ] should translate to list even if is singleton or empty (er, list() ) currently the comma returns a list whatever the context scalar or not so in the mean time, I just want the 0 and 1 element array composer to work even if it is not the good way in the long run just to pass basic tests it's basically the same code that allows foo() foo(1) and foo(1, 2, 3) to work properly % sjansen has joined #parrot * pmichaud wants to add :pasttype('list') to PCT, too. +( $ ) got me something like no method get_number in Hash +@( $ ) perhaps? say( +@($/)); gets me Null PMC access in get_number moritz, I think I tried it too % allison has joined #parrot % IllvilJa has joined #parrot dang. studying that diff has reminded me of yet another task i haven't gotten to pge should allow +@( $ ), probably particle: is there a ticket for it yet? i don't think so go ahead and add one, I'll take it (so that it stays on my radar) pmichaud: i can take a stab at it now, presuming i can relearn pge well enough where's the parsing done? inside Perl6Regex.pir? okay. I'd say create the ticket, then assign it to one of us. yes, in Perl6Regex.pir i'm looking at PGE::Exp::Alias (namespace) but it would actually parse in parse_subname so probably don't need the PGE::Exp::Alias at all awesome, that's just what i needed er, parse_subrule line 763 note that this code will change a bit because of the ==> change if $S0 == '=' goto subrule_alias ... if an '=' is found following the subrule identifier, then the lhs becomes the cname entry (look farther down in the code for 'cname') and the rhs becomes the new subname really, though, the cname computation probably needs to be hoisted up a bit higher in fact, I would set the capture/negated flags before parsing out a subname % rdice_ has joined #parrot and look specifically for aliases before the ' ' or ':' heh. seems like you know exactly what you want to do here. should i even bother, or can you bang this out now? yes I'll bang it out today ok. i'll write up a ticket right after lunch. :-) I also need to figure out how to get to work and that's right in this code as well. ? ? will you look at the : changes too? non-capturing became <.foo>, and became zero-length match % rdice has left rdice!~richard_d@76-10-176-159.dsl.teksavvy.com ah, that's still undone. I already added <.foo>, but haven't changed to its new semantic. right. and yes, I'll go ahead and do : while I'm at it :-) I started ':' last night, but got too tired to trust the code I was writing. will make the grammars prettier pmichaud: speaking of zero-length-match, have any thoughts on fixing PGE repeatedly matching the same zero-length-match in the same place? okay, I'll do lunch, then take care of all of the subrule syntax changes. % iblechbot has joined #parrot % gryphon has left gryphon!~gryphon@dsl-209-221-185-54.zipcon.net Tene: so far my thought has been that it hasn't occurred enough to worry about :-) but perhaps I'm just not hearing about it. (yes, I've thought about it beyond that. more) I don't recall where it bit my last. Probably the first time I worked on cardinal. So, no, probably not often enough. I do need to get the repetition constructs to be smart enough to watch for zero-length recursions I had a few ideas about it and played with it a bit, but didn't like what I had and then discovered "not often enough" so it's kinda on the back-burner for now. okay, lunch, then PGE. Thought so. If it comes up again, I'll harass you about it. please do. even just adding a note to the ticket that says "it came up again" will work. when I get enough of those, I'll fix it. :-) afk, lunch % davidfetter has joined #parrot % NotFound has joined #parrot Hello. % Theory has joined #parrot % Coke has joined #parrot . I'm looking #45989... it mentions src/objects.c but I don't see that file. I'm guessing it's pre-pdd15oo The date is oct 2007 -> oo.c Ah, right, thanks. % gryphon has joined #parrot % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.mn.comcast.net % Andy has joined #parrot % Theory has joined #parrot % japhb has left japhb!~geoff@76-191-190-8.dsl.static.sonic.net % ambs has joined #parrot I don't see how Parrot_add_attribute in src/oo.c can be used, all my attempts calls add_attribute in src/pmc/class.pmc this still for 45989? Yes. % Ivatar has joined #parrot I think it's dead code. It is quite possible that ParrotClass no longer exists. I agree (ParrotClass) that is only called in the addattribute opcode, and is conditional on the type of object, of which we had two, but now have one. Good catch. =-) is ParrotClass officially deprecated? (of course, if it doesn't exist I guess it doesn't matter) ls src/pmc/parrotclass* By the way, the exception in class.pmc does noy include the class name. No such file or directory s/noy/not notfound, do you have commit bits? I can't keep track anymore. =-) Coke: I'm a newbie. Try this: delete Parrot_add_attribute(). remove the code path in the addattribute op. rebuild everything do a make test. Ok. if that passes, attach that patch to the ticket and we'll apply it. (it's also in a .h somewhere.) that docs for that function read like dan. Parrot_single_subclass in oo.c uses it. did I miss a usage? hurm. might be the same thing, let's see if that's every used. =-) if (!parent_is_class) { /* we append one attribute to hold the PMC */ Parrot_add_attribute(interp, child_class, CONST_STRING(interp, "__value")); but I still agree that the code in the addattribute op should be fixed. yah. I think however that is adding attributes the other way is probably the right way, though. yes, likely. s/that is/that/ rip out one thing at a time. =-) some kind of c-level code coverage in the test suite would be nice. \o/ % AndyA has left AndyA!~andy@82.152.157.85 Parrot_single_subclass is used in default.pmc Yup. so for now, removing the parrotclass path in the opcode would be a help. % tedkat has joined #parrot Don't know how to do that. % AndyA has joined #parrot % desertmax has joined #parrot look at src/ops/object.c - look for the call to the function in question. it's in a code path that is conditional on something being equal to parrotclass, which it can never be. (since that PMC was apparently removed.) actually, src/ops/object.ops (src/ops/object.c is a generated file) % japhb has joined #parrot gah. yes. pmichaud++ What must I look for, VTABLE_subclass ? no, Parrot_add_attribute. (the function that started all this) - it's only used once in this file, and if we eliminate the dead conditional, we can eliminate it's usage, getting us one step closer to not having to fix it. =-) s/it's/its/ Ok. else if (VTABLE_isa(interp, $1, pclass_name)) Parrot_add_attribute(interp, $1, $2); s:1st/its/it's/ s:2nd/it's/its/ # revert and reapply pclass_name is ParrotClass so pclass_name could go, too? Looks like is tne only usage, yes. so try to rip that out, and then do a rebuild and see how make test does. Doing. (If you're bored, look for other usages of that value for pclass_name; might be a holdover in a few places. (none) Is a local variable in the function. All tests successful, 26 tests and 568 subtests skipped. Files=583, Tests=11121, 424 wallclock secs (288.78 cusr + 58.43 csys = 347.21 CPU) You have new mail in /var/mail/ambs ops, that last line should not be sent O:-) :) now, let me check for macOS lib/Parrot/Pmc2c/PMCEmitter.pm: use Parrot::Pmc2c::PMC::ParrotClass; not of pclass_name itself: the value it was checking against. value was just a local STRING ... yes, but if anything else is checking against that string... it is probably also wrong. lib/Parrot/Pmc2c/PMCEmitter.pm 'use's lib/Parrot/Pmc2c/PMC/ParrotClass.pm , but doesn't use it that i can see; kill the latter, drop the former 'use'? Files=583, Tests=11105, 449 wallclock secs ( 5.90 usr 0.95 sys + 306.94 cusr 68.94 csys = 382.73 CPU) Result: PASS spinclad: avoid unecessary uses, yes. ack shows no other "ParrotClass"es except editor/pir-mode.el mentions in a bunch of .pod's and comments... and MANIFEST. Files=583, Tests=11275, 552 wallclock secs ( 4.22 usr 3.77 sys + 245.53 cusr 91.12 csys = 344.64 CPU) Result: PASS for MacOS, nice :D Sending patch. NotFound++ Thanks. % admin has joined #parrot % Eevee has joined #parrot % admin is now known as Holly r27377 | chromatic++ | trunk: : [OO] Removed Parrot_single_subclass and Parrot_add_attribute, which were parts : of the old OO implementation (NotFound, RT #45989). diff: http://www.parrotvm.org/svn/parrot/revision?rev=27377 r27378 | pmichaud++ | trunk: : [pge]: : * Initial code refactor to handle zero-width matches. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27378 NotFound: ... heh. chromatic is always watching. =-) * Coke waves to chromatic. I see- I've finished work on #45015 some days ago. How do I get my code commited? :-) * Coke checks. NotFound: oooh. I didn't realize you tried to rip out *everything* and it still passed. awesome. NotFound++ NotFound++ % ambs has left ambs!~ambs@255.92.54.77.rev.vodafone.pt Well, not everything, the bits mentioned by spinclad not. desertmax: minor nits on the test file: print foo; print "\n" is more simply written as say foo. (just fyi) oh, okay. I am going to change this. (some commented out code at teh end of the test...) the uploaded files are not up to date, btw. I've made some changes. If this is a new test file, the copyright should just be 2008 ... Should probably have a test that runs if GMP is not installed to make sure it errors out cleanly. % grim_fandango has joined #parrot I'm not sure I would make an exception for version. Or perhaps have it return Undef instead of 0? (not sure if we have a standard to follow there.) % tedkat has left tedkat!~tedkat@24-119-222-126.cpe.cableone.net % rdice_ has left rdice_!~richard_d@CPE0014bfafbbd5-CM0011e6ecf48a.cpe.net.cable.rogers.com Thanks for the feedback. otherwise looks good, and those are mostly nits. You have another patch pending on that? I tried to match BigInt's behaviour, btw, because it uses GMP too. I've taken the ticket, so I'll see it when it gets updated. (bigint) fair enough. if that's how it does the versioning, makes sense to follow it. Okay, then I'll change all the other things and reupload the files. desertmax++ (don't forget a patch to CREDITS) r27379 | pmichaud++ | trunk: : [pge]: : * Remove form of enumerated character list -- it's : no longer allowed by S05. (The <+alpha-[jJ]> form still : exists, though.) diff: http://www.parrotvm.org/svn/parrot/revision?rev=27379 r27380 | pmichaud++ | trunk: : [pge]: : * Refactor handling of iscapture flag somewhat. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27380 % rdice has joined #parrot pmichaud, what does it takes to make +@( $ ) work? where are you trying to use it? null pmc in that case makes it look as though $ doesn't exist. (although that's a bit weird, as I would expect to get an Undef ) I try to parse [] and [ 1 ] I just add say( +@( $ ) ); below if $key eq '[ ]' { in the circumfix rule since is optional, it might not exist. in which case we can't find out how many elements it has :-) oh, it's not optional but I get it for [1,2] too hmmmm statement list contains an array of so you probably want +@( $ ) but truthfully, you don't want to be looking at the parse tree what you should be looking at is $( $ ) if that's an infix:<,> node, then you know it's a comma-separated list as opposed to a singleton or null % desertmax_ has joined #parrot % rdice has left rdice!~richard_d@CPE0019e063f3ed-CM0019474d4000.cpe.net.cable.rogers.com % particle[ventus] has joined #parrot ok % patrick has joined #parrot phone % Holly has left Holly!~me@83.105.31.111 % desertmax has left desertmax!~markus@62-47-160-212.adsl.highway.telekom.at % patrick has left patrick!~patrick@va-76-4-88-171.dhcp.embarqhsd.net watch coke lose weight :: http://coke-floats.blogspot.com/ % rdice has joined #parrot The exception for attribute already exists in class pmc does not include the class name, unlike the removed Parrot_add_attribute version. Must this be changed? might be helpful. "Attribute 'attr name' already exists in 'class name'" ? * Coke smacks chromatic. coke-floats++ % wknight8111 has joined #parrot allison@perl.org | Concurrency Tasks: link: http://www.perlfoundation.org/parrot/index.cgi?concurrency_tasks dalek's url is at http://xrl.us/bjqpy r27381 | pmichaud++ | trunk: : [pge]: : * S05 changes the non-capturing subrule syntax from to <.foo>. : * Fix a few more instances of this. There are still more to do, : especially in some of the languages/ subdirectories. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27381 % particle[ventus] has left particle[ventus]!~particle@c-24-19-3-148.hsd1.wa.comcast.net % desertmax_ has left desertmax_!~markus@212-183-123-211.adsl.highway.telekom.at r27382 | pmichaud++ | trunk: : [core]: : * Update DEPRECATED.pod with notes about current/upcoming PGE changes. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27382 % barney has joined #parrot pmichaud: when you say 'at which point', do you mean 'after which' ? (defining it in terms of the point makes it somewhat unclear what happens on releaase day.) % barney has left barney!~bernhard@dslb-084-058-148-172.pools.arcor-ip.net after which % rdice has left rdice!~richard_d@CPE0014bfafbbd5-CM0011e6ecf48a.cpe.net.cable.rogers.com % Theory has left Theory!~Theory@64.122.42.219 % teknomunk has joined #parrot % mj41 has left mj41!chatzilla@pc-jurosz.ro.vutbr.cz % mj41 has joined #parrot % particl1 has joined #parrot r27383 | pmichaud++ | trunk: : [pge]: : * Add syntax for capturing subrules. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27383 % iblechbot has left iblechbot!~iblechbot@41.18-dial.augustakom.net % peepsalot has left peepsalot!~peepsalot@bwext.kpimdp.com % Theory has joined #parrot Anyone here run Mac OSX? yes, why? Looking for confirmation/reproduction/analysis of a Rakudo bug that someone has reported on OSX, but works for me on Win32. ok which bug? hmmm... which bug is quiet, purl * purl listens as a hush falls over #perl http://use.perl.org/~JonathanWorthington/journal/36343 See comments. * masak reads Needs turning into a ticket, but I'm curious if anyone else can reproduce this. ok, so do I just `svn up` and do that `perl Configure.pl`? purl is confused, he doesn't even know where he is masak: Yes And then "make" then "make" is kind enough to create it again. * masak does so cd languages/perl6 && make # after the make of Parrot is done purl: hush. and this isn't #perl masak: what? purl: ssh ssh is non-free, all platforms: ftp://ftp.cs.hut.fi/pub/ssh/, UNIX (free): http://www.openssh.org/, MacOS: NiftyTelnetSSH http://www.lysator.liu.se/~jonasw/freeware/niftyssh/, Win: TeraTerm or puTTY (skip SecureCRT). Book: O'Reilly's http://www.snailbook.com/ or sing; sing; halt or (see the OpenSSH project at http://www.openssh.org) or NetBook: http://sourceforge.net/projects/syossh/ or BE QUIET! or ugly? :) doesn't print anything for me. Ivatar: What platform? Mac 10.5 Rev: 27376 make returned an error Did you use the perl6 executable? (that was for Ivatar) yep masak: make of Parrot? of languages/perl6 both as a script and interactively. % slightlyoff has joined #parrot Ivatar: Could you possibly try it with the PBC? (cd languages/perl6 && make, then run it as ../../parrot perl6.pbc) Jonathan: shall i nopaste the make error? masak: CAn you nopaste the...yes, please. :-) I can read books, too not just minds Jonathan: could the problem be that the user copied from the rendered html? your example cod needs more htmle scapes. Jonathan: http://paste2.org/p/26174 still nothing with parrot+pbc... :/ my example fingers need more keyboards. Tene: Oh!!! Godo catch. Godo enough for karma? Tene++ Tene++ Mmm. Karma. :) Jonathan-- # can write compilers, but HTML is too hard for him Jonathan: perhaps you need a compiler targeting HTML. I'm looking #46038 and #42374, that complains about a warning when freeing cstrings. I don't see that warnings, but I think that changing the signature of string_cstring_free to(ARGIN_NULLOK(char * const p)) will fix it. just typed out by hand, still nothing. Ivatar: Try with second line as it is now shown in that post (refresh, I edited it) Shoudl have been grammar Many is Loads { rule TOP { of } }; But the got eaten as HTML tags. I kind of thought it was missing something there :) (having done the squaak exercises) yeah, that was it Great. Phew, now we don't have an OSX-specific Rakudo bug to track down. goodie Thanks all. masak++, Tene++, Ivatar++ wee! Jonathan: looking forward to hearing your talk at NPW, btw Looking forward to being over in Stockholm. :-) And with Perl folks. Plus by then, I will have survived moving. * Jonathan is exhausted from an evening's packing. where are you moving to/from? wknight8111: south of Spain to Bratislava. oh okay Lived in Spain for the winter. I'm originally from the UK. I can't even get my fiance to go to spain for our honeymoon Too hot? Erm, not knowing where you're from that might be a silly question. :-) Come to the north, not so hot. True. The south has started to warm up some. Northwest, even better. * NotFound plays 'On the road to Santiago' % allison has left allison!~chatzilla@dsl-241-25-20.telkomadsl.co.za % mire has joined #parrot % masak has left masak!~user@130.238.45.242 % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com % contingencyplan has joined #parrot % tetragon has joined #parrot c:\usr\local\parrot\trunk>parrot t\pmc\orderedhash_9.pasm c:\usr\local\parrot\trunk>parrot -G t\pmc\orderedhash_9.pasm 101101 * particl1 found a gc bug :( society-- % particl1 is now known as particle[ventus] r27384 | pmichaud++ | trunk: : [pge]: : * Add quoted argument syntax (RT#53606, RT#53834). diff: http://www.parrotvm.org/svn/parrot/revision?rev=27384 particle[ventus]: same result for me with or without -G which result? 101101 you're not running msvc with a 32bit build on x64 * tetragon waits for her build to finish msvc++ # good at finding gc bugs Ops, no. % Ivatar has left Ivatar!~graham@tu055.demon.co.uk i bet if you ran the gcdebug runcore you'd see it parrot --runcore=gcdebug t/pmc/orderedhash_9.pasm $ ./parrot --runcore=gcdebug t/pmc/orderedhash_9.pasm src/gc/dod.c:180: failed assertion 'obj' there you go! chromatic++ gcdebug++ pobject_lives is called with a NULL obj. Looks like the object is a hash item. * tetragon compares the parrot exceptions.c backtrace against the crash reporter backtrace They differ The error happens at delete P0[P1] r27385 | pmichaud++ | trunk: : * Adjust wording in DEPRECATED.pod (Coke++) diff: http://www.parrotvm.org/svn/parrot/revision?rev=27385 r27386 | pmichaud++ | trunk: : [pge]: : * Go ahead and allow the syntax as well as . : It won't yet interpolate like normal double-quotes would in Perl 6, : but it lets us get rid of some backslashes and is more dwimmy. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27386 r27387 | pmichaud++ | trunk: : * Convert a variety of into . (RT#53606) diff: http://www.parrotvm.org/svn/parrot/revision?rev=27387 r27388 | chromatic++ | trunk: : [PMC] Removed long-deprecated subclass vtable entry from Default PMC (NotFound, : RT #45975). diff: http://www.parrotvm.org/svn/parrot/revision?rev=27388 % Patterner has left Patterner!~Psyche@e177238102.adsl.alicedsl.de r27389 | pmichaud++ | trunk: : [pge]: : * Update tools/dev/mk_language_shell.pl to use syntax. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27389 % Patterner has joined #parrot % slightlyoff has left slightlyoff!~slightlyo@204.14.154.209 % gryphon has left gryphon!~gryphon@dsl-209-221-185-54.zipcon.net /* TODO warn or fill if there are holes */ ---> Maybe this is related to the problem? hrmm, could be.... "svn status" is showing src/call_list.txt I'm not sure where that's coming from. r27390 | pmichaud++ | trunk: : [tools]: : * Update mk_language_shell.pl so that it doesn't attempt to configure : shells created outside of the languages/ directory. (RT#53418) diff: http://www.parrotvm.org/svn/parrot/revision?rev=27390 701 tickets. you guys have been kicking butt today (Parrot committers)++ % contingencyplan has left contingencyplan!~contingen@cpe-76-186-27-146.tx.res.rr.com 700 now, looks like. merged two tickets. 58+642 700 that counts. chaos-- clearly 700 was too easy of a goal. so, 650? I think 700 was a good initial goal. I'd hate to have one we couldn't reach. Inside parrot_mark_hash, if mark_key is true bucket->key must be non nulll, according the logic of the code. I put an assertion, and fails in the test case. % japhb has left japhb!~geoff@208.201.228.107 % particle[ventus] has left particle[ventus]!~particle@32.156.218.7 nah, not going to actually suggest another goal. but am going to keep digging. Dropping the assertion and not calling pobject_lives when null, the problem dissapear. Don't know if this is correct, however. Nobody's going to suggest 666 as the net goal? heh. I considered it then hushed up. =-) % kid51 has joined #parrot purl, is your messages service disabled? msg kid51 hi there. Message for kid51 stored. Is purl sleeping? * tetragon grumbles about connection latency Probably not, my wonderful Verizon DSL connection has dropped off twice in 15 minutes! % particl1 has joined #parrot For once, it's not a problem with the router's signal failing to reach my laptop. It's the Internet connection itself. maybe the tubes are blocked. I blame Senator Ted Stevens Looks like the problem is that delete_keyed_int sets the key in bucket to NULL, but mark_hash expects that it never be NULL. s/mark_hash/parrot_mark_hash % davidfetter has left davidfetter!~chatzilla@start.fetter.org % NotFound has left NotFound!~julian@50.Red-213-96-228.staticIP.rima-tde.net % japhb has joined #parrot % sjansen has left sjansen!~sjansen@hq-nat2.gurulabs.com % kid51 has left kid51!~jkeen@pool-71-247-38-145.nycmny.east.verizon.net % mire has left mire!~Frodo@252-172-222-85.adsl.verat.net r27391 | pmichaud++ | trunk: : [pge]: : * Remove outdated todo from RT#44449. diff: http://www.parrotvm.org/svn/parrot/revision?rev=27391 696. % Zaba has left Zaba!~zaba@ip102.148.adsl.wplus.ru % Zaba has joined #parrot * particl1 looks for more tickets to add... % janus has left janus!~janus@e182074115.adsl.alicedsl.de % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net % wknight8111 has joined #parrot % janus has joined #parrot % Psyche^ has joined #parrot % Patterner has left Patterner!~Psyche@e177238102.adsl.alicedsl.de % Psyche^ is now known as Patterner % wknight8111 has left wknight8111!~nobody@c-71-230-33-251.hsd1.pa.comcast.net % Theory has left Theory!~Theory@c-24-21-175-208.hsd1.mn.comcast.net % tetragon has left tetragon!~seneca@76-10-171-227.dsl.teksavvy.com % Theory has joined #parrot % grim_fandango has left grim_fandango!~matt@bas2-kingston08-1177655568.dsl.bell.ca % tetragon has joined #parrot % Eevee has left Eevee!~eevee@c-67-160-3-54.hsd1.wa.comcast.net % Psyche^ has joined #parrot % Patterner has left Patterner!~Psyche@e177228057.adsl.alicedsl.de % Psyche^ is now known as Patterner % Eevee has joined #parrot % petdance has joined #parrot #52976 has been resolved or at least worked around until the bug it exposed can be properly fixed the mmd code works, at any rate % particl1 has left particl1!~particle@c-24-19-3-148.hsd1.mn.comcast.net Is there any reason I shouldn't close that one myself? % uniejo has joined #parrot % masak has joined #parrot % masak has left masak!~user@nl119-201-209.student.uu.se % masak has joined #parrot Jonathan: er, why did `make` in fact fail yesterday? cotto_home: unless you want to keep the ticket around to track the *proper* fix, feel free to close it (but be sure to post about the current situation) % allison has joined #parrot % teknomunk has left teknomunk!~teknomunk@r74-195-239-111.stl1cmta01.stwrok.ok.dh.suddenlink.net % petdance has left petdance!~Andy@64.81.227.163