diff options
author | Eric Sandeen <sandeen@redhat.com> | 2009-10-25 21:41:32 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2009-10-25 21:41:32 -0400 |
commit | e633b58ac75f2f544b7d6572e37d4b63da31e59c (patch) | |
tree | b9dc7c7905f1134bac197d86aadb934a9bf17eec /e2fsck/pass2.c | |
parent | 03b9dca63a75731711071e0a7ddef0475d6daf3a (diff) | |
download | android_external_e2fsprogs-e633b58ac75f2f544b7d6572e37d4b63da31e59c.tar.gz android_external_e2fsprogs-e633b58ac75f2f544b7d6572e37d4b63da31e59c.tar.bz2 android_external_e2fsprogs-e633b58ac75f2f544b7d6572e37d4b63da31e59c.zip |
libext2fs: clean up ext2fs_bg_flags_ interfaces
The ext2fs_bg_flag* functions were confusing.
Currently we have this:
void ext2fs_bg_flags_set(ext2_filsys fs, dgrp_t group, __u16 bg_flags);
void ext2fs_bg_flags_clear(ext2_filsys fs, dgrp_t group,__u16 bg_flags);
(_set (unused) sets exactly bg_flags; _clear clears all and ignores bg_flags)
and these, which can twiddle individual bits in bg_flags:
void ext2fs_bg_flag_set(ext2_filsys fs, dgrp_t group, __u16 bg_flag);
void ext2fs_bg_flag_clear(ext2_filsys fs, dgrp_t group, __u16 bg_flag);
A better interface, after the patch below, is just:
ext2fs_bg_flags_zap(fs, group) /* zeros bg_flags */
ext2fs_bg_flags_set(fs, group, flags) /* adds flags to bg_flags */
ext2fs_bg_flags_clear(fs, group, flags) /* clears flags in bg_flags */
and remove the original ext2fs_bg_flags_set / ext2fs_bg_flags_clear.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'e2fsck/pass2.c')
-rw-r--r-- | e2fsck/pass2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/e2fsck/pass2.c b/e2fsck/pass2.c index 067a9c71..d45ff744 100644 --- a/e2fsck/pass2.c +++ b/e2fsck/pass2.c @@ -991,7 +991,7 @@ out_htree: pctx.num = dirent->inode; if (fix_problem(ctx, PR_2_INOREF_BG_INO_UNINIT, &cd->pctx)){ - ext2fs_bg_flag_clear(fs, group, + ext2fs_bg_flags_clear(fs, group, EXT2_BG_INODE_UNINIT); ext2fs_mark_super_dirty(fs); ctx->flags |= E2F_FLAG_RESTART_LATER; |