Skip to contentSkip to conversation

staging: android: sync_debug.c: remove unnecessary braces

PATCH RESEND In review Robin Krahl 2 participants 2 messages Jan 19, 2016 – Feb 7, 2016 Staging

lore
Share & tools
lore
#0 lore raw

PATCH RESEND 1/1 staging: android: sync_debug.c: remove unnecessary braces

Remove unnecessary braces {} around a single statement within a for loop. Indicated by checkpatch.

---
 drivers/staging/android/sync_debug.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/staging/android/sync_debug.c +1 -2
index f45d13c..02a1649 100644
@@ -158,9 +158,8 @@ static void sync_print_fence(struct seq_file *s, struct sync_fence *fence)
 	seq_printf(s, "[%p] %s: %s\n", fence, fence->name,
 		   sync_status_str(atomic_read(&fence->status)));
 
-	for (i = 0; i < fence->num_fences; ++i) {
+	for (i = 0; i < fence->num_fences; ++i)
 		sync_print_pt(s, fence->cbs[i].sync_pt, true);
-	}
 
 	spin_lock_irqsave(&fence->wq.lock, flags);
 	list_for_each_entry(pos, &fence->wq.task_list, task_list) {
Signature · 1 line
2.5.0
maintainer
#1 lore raw
Quoted 26 lines · Robin Krahl

Remove unnecessary braces {} around a single statement within a for loop. Indicated by checkpatch.

---
 drivers/staging/android/sync_debug.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
drivers/staging/android/sync_debug.c +1 -2
index f45d13c..02a1649 100644
@@ -158,9 +158,8 @@ static void sync_print_fence(struct seq_file *s, struct sync_fence *fence)
 	seq_printf(s, "[%p] %s: %s\n", fence, fence->name,
 		   sync_status_str(atomic_read(&fence->status)));
 
-	for (i = 0; i < fence->num_fences; ++i) {
+	for (i = 0; i < fence->num_fences; ++i)
 		sync_print_pt(s, fence->cbs[i].sync_pt, true);
-	}
 
 	spin_lock_irqsave(&fence->wq.lock, flags);
 	list_for_each_entry(pos, &fence->wq.task_list, task_list) {
Signature · 1 line
2.5.0

Someone sent a fixed up patch before this one :(