diff mbox series

FS: BTRFS: ulist.c: Fixed a brace coding style and space before tab

Message ID 20200522212902.20994-1-ethancarteredwards@gmail.com (mailing list archive)
State New, archived
Headers show
Series FS: BTRFS: ulist.c: Fixed a brace coding style and space before tab | expand

Commit Message

Ethan Edwards May 22, 2020, 9:29 p.m. UTC 
From: Ethan Edwards <ethancarteredwards@gmail.com

Fixed coding style

Signed-off-by: Ethan Edwards <ethancarteredwards@gmail.com>
---
 fs/btrfs/ulist.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

David Sterba May 25, 2020, 12:37 p.m. UTC | #1 
On Fri, May 22, 2020 at 05:29:02PM -0400, Ethan Edwards wrote:
> From: Ethan Edwards <ethancarteredwards@gmail.com
> 
> Fixed coding style

No thanks.

Long answer: https://btrfs.wiki.kernel.org/index.php/Developer%27s_FAQ#How_not_to_start
diff mbox series

Patch

diff --git a/fs/btrfs/ulist.c b/fs/btrfs/ulist.c
index 3374c9e9be67..1e0484c8d244 100644
--- a/fs/btrfs/ulist.c
+++ b/fs/btrfs/ulist.c
@@ -22,7 +22,7 @@ 
  * ULIST_ITER_INIT(&uiter);
  *
  * while ((elem = ulist_next(ulist, &uiter)) {
- * 	for (all child nodes n in elem)
+ *	for (all child nodes n in elem)
  *		ulist_add(ulist, n);
  *	do something useful with the node;
  * }
@@ -266,11 +266,11 @@  struct ulist_node *ulist_next(struct ulist *ulist, struct ulist_iterator *uiter)
 		return NULL;
 	if (uiter->cur_list && uiter->cur_list->next == &ulist->nodes)
 		return NULL;
-	if (uiter->cur_list) {
+	if (uiter->cur_list)
 		uiter->cur_list = uiter->cur_list->next;
-	} else {
+	else
 		uiter->cur_list = ulist->nodes.next;
-	}
+
 	node = list_entry(uiter->cur_list, struct ulist_node, list);
 	return node;
 }