This is the mail archive of the libstdc++@gcc.gnu.org mailing list for the libstdc++ project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Prevent macro expansion of mblen


On Sun, Dec 31, 2000 at 07:54:18PM -0200, Alexandre Oliva wrote:
> I still fail to understand why [std::mblen and ::mblen] should be 
> the same entity.  

As has been pointed out several times, ISO 14882 requires it.

> We'd lose the optimization opportunity that the header files of the 
> system attempt to offer ...

We already know how to avoid losing the optimization opportunity...

> , and failing to work in case the name is only
> available as a macro (which is not the case at hand, but it might be
> on other systems).

The name is, apparently, never available only as a macro.

> These two problems are absent with the
> implementation in my latest patch, as proposed by Ben.  

I don't see why the treatment of mblen should be different from
any other C library symbol.  Many of them are commonly shadowed
as macros, and the same arguments and treatment apply to all.  
That is the point of much of (what was) the cshadow/ apparatus.
 
> But, if you have a good argument against the patch I last proposed,
> here's one that does it as you suggest.  Ok to install?

Therefore: no, IMO.

Nathan Myers
ncm at cantrip dot org

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]