Move assignment out of inner loop -- only needs to be done once, and

fixes a compiler warning (and potential bug)

This commit was SVN r7540.

opal/mca/base/mca_base_components_open.c

@@ -275,10 +275,10 @@ static int distill(bool include_mode, const char *type_name,
              item = next) {
             next = opal_list_get_next(item);
             good = true;
-
-            for (i = 0; NULL != names[i]; ++i) {
             cli = (mca_base_component_list_item_t *) item;
             component = cli->cli_component;
+
+            for (i = 0; NULL != names[i]; ++i) {
                 if (0 == strcmp(names[i], component->mca_component_name)) {
                     good = false;
                     break;