diff mbox

btrfs-progs: make it static if function isn't called outside

Message ID 1384330779-16647-1-git-send-email-anand.jain@oracle.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Anand Jain Nov. 13, 2013, 8:19 a.m. UTC
Signed-off-by: Anand Jain <anand.jain@oracle.com>
---
 cmds-dedup.c |    2 +-
 disk-io.c    |    2 +-
 qgroup.c     |    6 +++---
 utils.c      |    2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

Comments

David Sterba Nov. 13, 2013, 3:50 p.m. UTC | #1
On Wed, Nov 13, 2013 at 04:19:39PM +0800, Anand Jain wrote:
> --- a/cmds-dedup.c
> +++ b/cmds-dedup.c
> @@ -31,7 +31,7 @@ static const char * const dedup_cmd_group_usage[] = {
>  	NULL
>  };
>  
> -int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
> +static int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
>  {
>  	int ret = 0;
>  	int fd;

FYI, this hunk modifies code in the the dedup patch that's floating
above the other patches that may get merged to master eventually, so
I've folded it into the dedup patch itself.

david
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Liu Bo Nov. 13, 2013, 3:55 p.m. UTC | #2
On Wed, Nov 13, 2013 at 04:50:30PM +0100, David Sterba wrote:
> On Wed, Nov 13, 2013 at 04:19:39PM +0800, Anand Jain wrote:
> > --- a/cmds-dedup.c
> > +++ b/cmds-dedup.c
> > @@ -31,7 +31,7 @@ static const char * const dedup_cmd_group_usage[] = {
> >  	NULL
> >  };
> >  
> > -int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
> > +static int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
> >  {
> >  	int ret = 0;
> >  	int fd;
> 
> FYI, this hunk modifies code in the the dedup patch that's floating
> above the other patches that may get merged to master eventually, so
> I've folded it into the dedup patch itself.
> 


Thanks for the efforts, Dave.

-liubo
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/cmds-dedup.c b/cmds-dedup.c
index beaadb3..67aa6ee 100644
--- a/cmds-dedup.c
+++ b/cmds-dedup.c
@@ -31,7 +31,7 @@  static const char * const dedup_cmd_group_usage[] = {
 	NULL
 };
 
-int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
+static int dedup_ctl(char *path, struct btrfs_ioctl_dedup_args *args)
 {
 	int ret = 0;
 	int fd;
diff --git a/disk-io.c b/disk-io.c
index 3d855d2..6a24772 100644
--- a/disk-io.c
+++ b/disk-io.c
@@ -344,7 +344,7 @@  int write_and_map_eb(struct btrfs_trans_handle *trans,
 	return 0;
 }
 
-int write_tree_block(struct btrfs_trans_handle *trans,
+static int write_tree_block(struct btrfs_trans_handle *trans,
 		     struct btrfs_root *root,
 		     struct extent_buffer *eb)
 {
diff --git a/qgroup.c b/qgroup.c
index 1c68046..94d1feb 100644
--- a/qgroup.c
+++ b/qgroup.c
@@ -76,7 +76,7 @@  struct btrfs_qgroup_list {
 /*
  * qgroupid,rfer,excl default to set
  */
-struct {
+static struct {
 	char *name;
 	char *column_name;
 	int need_print;
@@ -652,7 +652,7 @@  static int add_qgroup(struct qgroup_lookup *qgroup_lookup, u64 qgroupid,
 	return ret;
 }
 
-void __free_btrfs_qgroup(struct btrfs_qgroup *bq)
+static void __free_btrfs_qgroup(struct btrfs_qgroup *bq)
 {
 	struct btrfs_qgroup_list *list;
 	while (!list_empty(&bq->qgroups)) {
@@ -674,7 +674,7 @@  void __free_btrfs_qgroup(struct btrfs_qgroup *bq)
 	free(bq);
 }
 
-void __free_all_qgroups(struct qgroup_lookup *root_tree)
+static void __free_all_qgroups(struct qgroup_lookup *root_tree)
 {
 	struct btrfs_qgroup *entry;
 	struct rb_node *n;
diff --git a/utils.c b/utils.c
index 97859d9..2345f40 100644
--- a/utils.c
+++ b/utils.c
@@ -1939,7 +1939,7 @@  int test_dev_for_mkfs(char *file, int force_overwrite, char *estr)
 	return 0;
 }
 
-int test_skip_this_disk(char *path)
+static int test_skip_this_disk(char *path)
 {
 	int fd;