diff options
author | Andrew Morton <akpm@linux-foundation.org> | 2023-05-12 15:59:53 -0700 |
---|---|---|
committer | Andrew Morton <akpm@linux-foundation.org> | 2023-05-16 16:30:31 -0700 |
commit | 3e8c407e81545ad31b0b9e1355d8a4dab7861acf (patch) | |
tree | 9e4f0842b61fa128cbfa06c736b22222abeb1cef /lib | |
parent | ae8a1cd2d215dc6353dc9111f36e841edadc6fde (diff) | |
download | linux-next-3e8c407e81545ad31b0b9e1355d8a4dab7861acf.tar.gz |
maple_tree-add-mas_next_range-and-mas_find_range-interfaces-checkpatch-fixes
ERROR: Macros with complex values should be enclosed in parentheses
#58: FILE: include/linux/maple_tree.h:544:
+#define mas_contiguous(__mas, __entry, __max) \
+ while (((__entry) = mas_find_range((__mas), (__max))) != NULL)
WARNING: suspect code indent for conditional statements (8, 12)
#285: FILE: lib/maple_tree.c:6136:
+ if (mas_find_setup(mas, max, &entry))
+ return entry;
WARNING: Statements should start on a tabstop
#286: FILE: lib/maple_tree.c:6137:
+ return entry;
total: 1 errors, 2 warnings, 284 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
./patches/maple_tree-add-mas_next_range-and-mas_find_range-interfaces.patch has style problems, please review.
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
Please run checkpatch prior to sending patches
Cc: "Liam R. Howlett" <Liam.Howlett@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/maple_tree.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/maple_tree.c b/lib/maple_tree.c index 0b466f596179..41c152718000 100644 --- a/lib/maple_tree.c +++ b/lib/maple_tree.c @@ -6134,7 +6134,7 @@ void *mas_find(struct ma_state *mas, unsigned long max) void *entry = NULL; if (mas_find_setup(mas, max, &entry)) - return entry; + return entry; /* Retries on dead nodes handled by mas_next_slot */ return mas_next_slot(mas, max, false); |