Patch: Change uint32 to uint32_t in vfs.h

Richard Sharpe realrichardsharpe at gmail.com
Fri Apr 17 14:58:44 MDT 2015


On Fri, Apr 17, 2015 at 1:17 PM, Jeremy Allison <jra at samba.org> wrote:
> On Tue, Apr 14, 2015 at 08:54:34AM +0200, Andreas Schneider wrote:
>> On Monday 13 April 2015 17:03:06 Richard Sharpe wrote:
>> > On Mon, Apr 13, 2015 at 5:01 PM, Richard Sharpe
>> >
>> > <realrichardsharpe at gmail.com> wrote:
>> > > Hi folks,
>> > >
>> > > I was extremely shocked to find that there are around 104 instances of
>> > > uint32 in our code.
>> > >
>> > > Here is a patch that changes 17 of them to uint32_t.
>> > >
>> > > If this sort of thing is welcome, I will fix the rest.
>> >
>> > So shocked, in fact, that I forgot to include the patch:
>>
>> RB+
>>
>> And yes, we should fix them all. We just fixed them slowly when we worked on
>> code which still does not use stdint types.
>
> NAK on this one, sorry. It changes the vfs.h types, but
> not the ones in the .c files. Can you do both at once ?
> I know they end up as the same underlying type, but doing
> only .h and not .c's leaves the code looking 'strange' :-).

Yes, I will resubmit with both. I don't know why I did that. Will look
at it this weekend.

-- 
Regards,
Richard Sharpe
(何以解憂?唯有杜康。--曹操)


More information about the samba-technical mailing list