Re: [icu-design] Re: Wrap classes in structs for C API (PR#5128)

[Deborah G. <gol...@ap...>]

That's not supported in the API. In fact, the API doesn't say  
anything about a USet being the same as a UnicodeSet. If we ever do  
support it in the API, it's trivial to back out the explicit  
qualification.

As I said in the original bug, virtual calls cause a pipeline stall  
on many processors. It's not worth the performance hit to enable  
unsupported and undocumented behavior.

Deborah

On Apr 20, 2006, at 5:14 PM, Eric Mader wrote:

> Is there really no chance of polymorphism? Can't I do something like:
>
> class MyUnicodeSet : public UnicodeSet
> {
>   MyUnicodeSet(...);
>   ...
> }
>
> ...
> void foo()
> {
>   USet *uset = (USet *) new MyUnicodeSet(...);
>
>   uset_add(uset, 0x0915);
>   ...
> }
>
> Eric
>
> Deborah Goldsmith wrote:
>> I thought of a much simpler way to do this. Rather than replace:
>> ((UnicodeSet *)uset)->whatever();
>> by
>> struct USet {
>>     UnicodeSet  fSet;
>> };
>> ...
>> uset->fSet.whatever()
>> instead, just do this:
>> ((UnicodeSet *)uset)->UnicodeSet::whatever();
>> which makes the call non-virtual. Much less invasive, and much  
>> less work. :-)
>> -------------------------------------------------------
>> Using Tomcat but need to do more? Need to support web services,  
>> security?
>> Get stuff done quickly with pre-integrated technology to make your  
>> job easier
>> Download IBM WebSphere Application Server v.1.0.1 based on Apache  
>> Geronimo
>> http://sel.as-us.falkag.net/sel? 
>> cmd=lnk&kid=120709&bid=263057&dat=121642
>> _______________________________________________
>> icu-design mailing list
>> icu...@li...
>> To Un/Subscribe: https://lists.sourceforge.net/lists/listinfo/icu- 
>> design
>
>
>
> -------------------------------------------------------
> Using Tomcat but need to do more? Need to support web services,  
> security?
> Get stuff done quickly with pre-integrated technology to make your  
> job easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache  
> Geronimo
> http://sel.as-us.falkag.net/sel? 
> cmd=lnk&kid=120709&bid=263057&dat=121642
> _______________________________________________
> icu-design mailing list
> icu...@li...
> To Un/Subscribe: https://lists.sourceforge.net/lists/listinfo/icu- 
> design