On 5/21/11 4:30 AM, Panayotis Katsaloulis wrote:
>
> On May 13, 2011, at 8:46 PM, Arno Puder wrote:
>
>>
>> I'm OK with this patch. Needless to say you should only commit it when
>> the C side works as well. Please take a look at the current C
>> implementation of performSel... and ask me any questions you have in
>> understanding the inner workings. The one thing that might help you to
>> understand the C code is the dispatcherObject. Its purpose is to be used
>> as a stand-in to perform the actual performSelec... method on the
>> Objective-C side.
>>
>> Arno
>
> I have commit the patch, both backends work.
> What I actually did for the C backend, is more or less to copy the method of the older system.
> I am sure that some optimization could be performed, but I am a bit confused :)
>
> I didn't touch the old methods, I have only marked them as deprecated.
> If you agree, I'd go over the android demos and replace the obsolete code.
> :)
yes, your implementation is pretty inefficient. :) Please change the
Android Compat Lib and all other locations to make use of NSSelector.
Then remove the deprecated method. Once you have done that, I will
revise your implementation. I need to change the implementation in
DispatcherObject that will break the deprecated method, so I will wait
until you are done with your changes.
Arno
|