Hi, Looks good,
Steve. On Mon, 2012-04-30 at 18:29 +0100, Andrew Price wrote: > Fixes these warnings: > fs_geometry.c:80:47: warning: variable 'rlast2' set but not used > [-Wunused-but-set-variable] > meta.c:732:3: warning: large integer implicitly truncated to unsigned > type [-Woverflow] > > Signed-off-by: Andrew Price <[email protected]> > --- > gfs2/libgfs2/fs_geometry.c | 3 +-- > gfs2/libgfs2/meta.c | 2 +- > 2 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/gfs2/libgfs2/fs_geometry.c b/gfs2/libgfs2/fs_geometry.c > index fb55697..e248f7c 100644 > --- a/gfs2/libgfs2/fs_geometry.c > +++ b/gfs2/libgfs2/fs_geometry.c > @@ -77,7 +77,7 @@ void compute_rgrp_layout(struct gfs2_sbd *sdp, struct > osi_root *rgtree, > int rgsize_specified) > { > struct device *dev; > - struct rgrp_tree *rl, *rlast = NULL, *rlast2 = NULL; > + struct rgrp_tree *rl, *rlast = NULL; > struct osi_node *n, *next = NULL; > unsigned int rgrp = 0, nrgrp, rglength; > uint64_t rgaddr; > @@ -105,7 +105,6 @@ void compute_rgrp_layout(struct gfs2_sbd *sdp, struct > osi_root *rgtree, > PRIx64 "), length = %"PRIu64" (0x%" > PRIx64 ")\n", rgrp + 1, rl->start, rl->start, > rl->length, rl->length); > - rlast2 = rlast; > rlast = rl; > } > rlast->start = rlast->ri.ri_addr; > diff --git a/gfs2/libgfs2/meta.c b/gfs2/libgfs2/meta.c > index 87f8f31..1f796c8 100644 > --- a/gfs2/libgfs2/meta.c > +++ b/gfs2/libgfs2/meta.c > @@ -729,7 +729,7 @@ const struct lgfs2_metadata lgfs2_metadata[] = { > }, > [LGFS2_MT_FREE] = { > .gfs1 = 1, > - .gfs2 = 2, > + .gfs2 = 1, > .name = "free", > }, > };
