Messages in this thread | | | Date | Wed, 2 Mar 2011 22:37:41 +0300 | From | Dan Carpenter <> | Subject | Re: [PATCH] Staging: keucr: Remove all type defines |
| |
On Wed, Mar 02, 2011 at 08:20:23PM +0100, Roel Van Nyen wrote: > On Tue, Mar 01, 2011 at 11:00:06PM +0300, Dan Carpenter wrote: > > On Tue, Mar 01, 2011 at 08:29:50PM +0100, Roel Van Nyen wrote: > > > -typedef u8 BOOLEAN; > > > > It might be better to use bool type or int for boolean variables. But > > that can't be done automatically, it has to be audited carefully. I > > don't feel strongly about this, I'm just throwing it out there as food > > for thought. You are using the remove-typedef program btw? > > I did not use the remove-typedef program, I used perl regex. (btw: where is that, i could not find it)
I just saw Luis R. Rodriguez mention it. I hadn't heard of it before.
http://www.kernel.org/pub/linux/kernel/people/mcgrof/scripts/remove-typedef
> I will post a patchset later tonight wich contains the removal of all the types from this driver. > The first patch will contain the removal of all types except the BOOLEAN. > In the second one I will remove the boolean and check where to make it an integer and where a bool.
Sounds good.
regards, dan carpenter
| |