Parrot 6.3.0 "Black-cheeked Lovebird" | parrot.org/ | Log: irclog.perlgeek.de/parrot | #parrotsketch meeting Tuesday 19:30 UTC
Set by moderator on 15 April 2014.
01:07 woosley joined 01:18 FROGGS_ joined 01:58 basiliscos joined 02:21 plobsing joined 04:30 Psyche^ joined 06:46 FROGGS joined 07:59 cxreg joined 08:45 Tene joined 09:03 basiliscos joined 14:16 bluescreen joined 15:42 bluescreen_ joined 16:12 FROGGS joined 17:53 Chirag joined 18:03 benabik joined
Chirag rurban: Hey! 18:11
rurban Hey 18:14
Chirag I came across VTABLE_elements (INTERP, array) in arrayiterator .. I am assuming this method/macro is different from VTABLE INTVAL elements() or is it?? I cant find its definition anywhere.. 18:15
rurban This is calling the VTABLE method elements on the arrayiterator object. When you derive from the arrayiterator class it would call the arrayiterator.elements method, if the derived class does not have it's own elements method. 18:17
Chirag but elements() does not take any arguments.. 18:18
rurban I've set it to :no_wb but on further consideration SELF attributes are changed depending on the array PMC so I guess the :no_wb attribute is wrong. the WB should be added automatically at the end 18:19
array is SELF 18:20
every vtable method has at least 2 args, interp and self
Chirag The definition for VTABLE INTVAL elements() gave me an impression that it takes no args 18:22
rurban because in our pmc files we skip the first 2 default args and let the pmc2c compiler do that
Chirag ohk.. got it now .. that would give set_integer_native() manual_wb 18:23
because it calls VTABLE_elements
rurban no, since it's void it's easier to let pmc2c add the WB
there's no return to consider
Chirag oh 18:24
rurban but you can also add the WB manually at the end
I think the WB is needed here, even we do not create a new PMC. But I'm not sure. Could be a :no_wb also 18:25
Chirag i was putting it becuse it was making a call to elements() which has a WB .. so it implied a hidden WB .. hence the attribute manual_wb 18:26
but if I consider a void return then maybe I should put no_wb 18:27
rurban no, not needed when calling other VTABLE methods. only when it's hidden in macros
Chirag alright! 18:28
understood
I have added WBs for - ArrayIterator, CallContext, Key, Task 20:31
I will try to finish the remaining over the weekend...
Bye! :) 20:32
20:54 stryx` joined 21:15 basiliscos joined 21:30 bluescreen_ joined