This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH]: Implement fwide (was Re: swprintf() and friends?)


On Dec 10 15:40, Jeff Johnston wrote:
> Corinna Vinschen wrote:
>> Jeff?
>>
>> Did you have a look in the meantime?
>>
>>   
>
> I actually was looking at this today.
>
> Couple of immediate points:
>
> 1. In ORIENT, you do a fp->flags != __SORD (should be "|= ")

Ouch!  Thanks for spotting.

> 2. ORIENT should not check the value of ori.  It is only 0 in one case when 
> fwide
>    is used so fwide should check for 0 and not call ORIENT in that case.

Fine with me.

> 3. There doesn't seem to be orientation for the printf family while the 
> scanf family is covered
>     via vfscanf.

Hmm, you seem to be right.  I examined the FreeBSD sources again since I
was pretty sure that the code path will result in calling ORIENT when
calling any printf function.  But it really looks like printf doesn't
set the orientation.  A bug in the FreeBSD sources?  __sfvwrite_r might
be a good point to call ORIENT, right?

> I can fix these myself if you like or you can resubmit if you would like to 
> test.

I have no problems if you fix that.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Project Co-Leader
Red Hat


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