zfs code - Jun 2009 - arc.c: variable types observation

* This comment is noted from reading 
*
http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/uts/common/fs/zfs/arc.c
* which may or may not be the current version.

I was reading through the code and I noticed that arc_no_grow and arc_dead are
declared as ''int''.  Looking around, they are only used as
boolean flags.  With that, would it be better to declare them as a
''boolean_t'' like arc_warm and others?

Is there some reason I may have missed as to why these are
''int''?

-Lares
-- 
This message posted from opensolaris.org

Marc Moreau wrote:
> * This comment is noted from reading 
> *
http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/uts/common/fs/zfs/arc.c
> * which may or may not be the current version.
> 
> I was reading through the code and I noticed that arc_no_grow and arc_dead
are declared as ''int''.  Looking around, they are only used as
boolean flags.  With that, would it be better to declare them as a
''boolean_t'' like arc_warm and others?
That sounds like a reasonable change to me.

--matt