Gitweb: http://git.fedorahosted.org/git/?p=gfs2-utils.git;a=commitdiff;h=7520428d1f7... Commit: 7520428d1f78a538243b92226335ce9d1551633c Parent: 6260b11af9222910550678f7c8933d4091870427 Author: Bob Peterson rpeterso@redhat.com AuthorDate: Thu Sep 24 10:15:46 2015 -0500 Committer: Bob Peterson rpeterso@redhat.com CommitterDate: Tue Sep 29 13:23:44 2015 -0500
fsck.gfs2: Combine remove_inode_eattr with its only caller
Since remove_inode_eattr() is now only called from one place, it is combined with its caller to make the code more readable and less confusing: function ask_remove_inode_eattr. --- gfs2/fsck/pass1.c | 23 +++++++---------------- 1 files changed, 7 insertions(+), 16 deletions(-)
diff --git a/gfs2/fsck/pass1.c b/gfs2/fsck/pass1.c index a515a62..895d1d1 100644 --- a/gfs2/fsck/pass1.c +++ b/gfs2/fsck/pass1.c @@ -553,17 +553,6 @@ static int check_data(struct gfs2_inode *ip, uint64_t metablock, return 0; }
-static int remove_inode_eattr(struct gfs2_inode *ip, struct block_count *bc) -{ - undo_reference(ip, ip->i_di.di_eattr, 0, bc); - ip->i_di.di_eattr = 0; - bc->ea_count = 0; - ip->i_di.di_blocks = 1 + bc->indir_count + bc->data_count; - ip->i_di.di_flags &= ~GFS2_DIF_EA_INDIRECT; - bmodified(ip->i_bh); - return 0; -} - static int ask_remove_inode_eattr(struct gfs2_inode *ip, struct block_count *bc) { @@ -573,11 +562,13 @@ static int ask_remove_inode_eattr(struct gfs2_inode *ip, "errors.\n"), (unsigned long long)ip->i_di.di_num.no_addr, (unsigned long long)ip->i_di.di_num.no_addr); if (query( _("Clear all Extended Attributes from the inode? (y/n) "))){ - if (!remove_inode_eattr(ip, bc)) - log_err( _("Extended attributes were removed.\n")); - else - log_err( _("Unable to remove inode eattr pointer; " - "the error remains.\n")); + undo_reference(ip, ip->i_di.di_eattr, 0, bc); + ip->i_di.di_eattr = 0; + bc->ea_count = 0; + ip->i_di.di_blocks = 1 + bc->indir_count + bc->data_count; + ip->i_di.di_flags &= ~GFS2_DIF_EA_INDIRECT; + bmodified(ip->i_bh); + log_err( _("Extended attributes were removed.\n")); } else { log_err( _("Extended attributes were not removed.\n")); }