Re: [Pyobjc-dev] Re: [Pyobjc-checkins] CVS: pyobjc/Modules/objc OC_PythonString.[hm]

[Ronald O. <ous...@ci...>]

On Sunday, Feb 16, 2003, at 00:02 Europe/Amsterdam, Bill Bumgarner 
wrote:

> Oops.  I really didn't mean to check these in.  Instead of undoing it, 
> I'll leave it for now as this code is *not* used currently.
>
> They actually work -- all unit tests pass -- but do not work in that 
> they do not handle unicode correctly.
>
> I believe this is the direction we should move in when bridging 
> strings from Python into ObjC -- it can be made fast and it  will 
> often avoid copying the python buffers.   There are very likely 
> reference counting issues.   I'm not sure.
>
> If we don't use this stuff, OC_PythonString.[hm] should be removed 
> entirely from the project.

I'm in favor of OC_PythonInt.[hm] and OC_PythonString.[hm] that were in 
before you checked in a new OC_PythonString implementation.

I have my doubts regarding a speedup by not copying python strings. And 
as NSString is a unicode object the new-style OC_PythonString should 
probably only be used for python unicode objects.

Ronald