jffs2:jffs2_1pass.c: remove double braces

Clang interpretes an if condition like "if ((a = b) == NULL) as it tries to assign a value in a statement. Hence if you do "if ((something)) it warns you that you might be confused. Hence drop the double braces for plane if statements. Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>

jffs2:jffs2_1pass.c: remove double braces
Clang interpretes an if condition like "if ((a = b) == NULL) as it tries to assign a value in a statement. Hence if you do "if ((something)) it warns you that you might be confused. Hence drop the double braces for plane if statements. Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
Jeroen Hofstee · Tom Rini
1 parent 2716077cda
Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff
fs/jffs2/jffs2_1pass.c
@@ -724,7 +724,7 @@
 	for (b = pL->frag.listHead; b != NULL; b = b->next) {
 		jNode = (struct jffs2_raw_inode *) get_node_mem(b->offset,
 								pL->readbuf);
-		if ((inode == jNode->ino)) {
+		if (inode == jNode->ino) {
 #if 0
 			putLabeledWord("\r\n\r\nread_inode: totlen = ", jNode->totlen);
 			putLabeledWord("read_inode: inode = ", jNode->ino);
...	...	@@ -724,7 +724,7 @@
724	724	for (b = pL->frag.listHead; b != NULL; b = b->next) {
725	725	jNode = (struct jffs2_raw_inode *) get_node_mem(b->offset,
726	726	pL->readbuf);
727		- if ((inode == jNode->ino)) {
	727	+ if (inode == jNode->ino) {
728	728	#if 0
729	729	putLabeledWord("\r\n\r\nread_inode: totlen = ", jNode->totlen);
730	730	putLabeledWord("read_inode: inode = ", jNode->ino);