Slight cleanup by using a for() loop instead of while(). This makes it
clearer what is the iteration and what is the actual work done.

Signed-off-by: Peter Zijlstra (Intel) <pet...@infradead.org>
---
 scripts/mod/modpost.c |    8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

--- a/scripts/mod/modpost.c
+++ b/scripts/mod/modpost.c
@@ -1595,12 +1595,10 @@ static void read_symbols(const char *mod
                        license = get_next_modinfo(&info, "license", license);
                }
 
-               namespace = get_modinfo(&info, "import_ns");
-               while (namespace) {
+               for (namespace = get_modinfo(&info, "import_ns");
+                    namespace;
+                    namespace = get_next_modinfo(&info, "import_ns", 
namespace))
                        add_namespace(&mod->imported_namespaces, namespace);
-                       namespace = get_next_modinfo(&info, "import_ns",
-                                                    namespace);
-               }
 
                if (!get_modinfo(&info, "description"))
                        warn("missing MODULE_DESCRIPTION() in %s\n", modname);



Reply via email to