Patchwork tweak lookup_feature() in target-i386/cpu.c

login
register
mail settings
Submitter Michael W. Bombardieri
Date March 30, 2013, 2:01 a.m.
Message ID <20130330020121.GA32412@bom.nom.co>
Download mbox | patch
Permalink /patch/232502/
State New
Headers show

Comments

Michael W. Bombardieri - March 30, 2013, 2:01 a.m.
Hi,

The following patch removes variable 'found' from lookup_feature().
We can just return true/false (found/not found) directly.
Does this look OK?

- Michael
Igor Mammedov - April 1, 2013, 12:13 p.m.
On Sat, 30 Mar 2013 10:01:21 +0800
"Michael W. Bombardieri" <mb@ii.net> wrote:

> Hi,
> 
> The following patch removes variable 'found' from lookup_feature().
> We can just return true/false (found/not found) directly.
> Does this look OK?
We are aiming to remove lookup_feature() and related feature tables,
so touching it without adding/fixing something looks like unnecessary code
movement.

> 
> - Michael
> 
> 
> diff --git a/target-i386/cpu.c b/target-i386/cpu.c
> index a0640db..9147aaf 100644
> --- a/target-i386/cpu.c
> +++ b/target-i386/cpu.c
> @@ -319,15 +319,14 @@ static bool lookup_feature(uint32_t *pval, const char *s, const char *e,
>  {
>      uint32_t mask;
>      const char **ppc;
> -    bool found = false;
>  
>      for (mask = 1, ppc = featureset; mask; mask <<= 1, ++ppc) {
>          if (*ppc && !altcmp(s, e, *ppc)) {
>              *pval |= mask;
> -            found = true;
> +            return true;
>          }
>      }
> -    return found;
> +    return false;
>  }
>  
>  static void add_flagname_to_bitmaps(const char *flagname,
>

Patch

diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index a0640db..9147aaf 100644
--- a/target-i386/cpu.c
+++ b/target-i386/cpu.c
@@ -319,15 +319,14 @@  static bool lookup_feature(uint32_t *pval, const char *s, const char *e,
 {
     uint32_t mask;
     const char **ppc;
-    bool found = false;
 
     for (mask = 1, ppc = featureset; mask; mask <<= 1, ++ppc) {
         if (*ppc && !altcmp(s, e, *ppc)) {
             *pval |= mask;
-            found = true;
+            return true;
         }
     }
-    return found;
+    return false;
 }
 
 static void add_flagname_to_bitmaps(const char *flagname,