Skip to contentSkip to conversation

mm: fixed whitespace style errors in failslab.c

PATCH In review Derek Robson 2 participants 2 messages Apr 27, 2015

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 mm: fixed whitespace style errors in failslab.c

This patch fixes a white space issue found with checkpatch.pl in failslab.c
ERROR: code indent should use tabs where possible

Added a tab to replace the spaces to meet the preferred style.

---
 mm/failslab.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
mm/failslab.c +1 -1
index fefaaba..2064225 100644
@@ -16,7 +16,7 @@ bool should_failslab(size_t size, gfp_t gfpflags, unsigned long cache_flags)
 	if (gfpflags & __GFP_NOFAIL)
 		return false;
 
-        if (failslab.ignore_gfp_wait && (gfpflags & __GFP_WAIT))
+	if (failslab.ignore_gfp_wait && (gfpflags & __GFP_WAIT))
 		return false;
 
 	if (failslab.cache_filter && !(cache_flags & SLAB_FAILSLAB))
Signature · 7 lines
2.3.6

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to [email protected].  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>
#1 lore raw
Quoted 9 lines · Derek Robson

This patch fixes a white space issue found with checkpatch.pl in failslab.c
ERROR: code indent should use tabs where possible

Added a tab to replace the spaces to meet the preferred style.

---
 mm/failslab.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Running checkpatch.pl against in-tree files is not really useful. Especially if you "fix" only whitespace stuff.
Most maintainers agree that it is not worth the maintenance overhead and the git history pollution.

Please stick to drivers/staging/ or fix real issues. :-)

Signature · 8 lines
Thanks,
//richard

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to [email protected].  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>