diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2009-08-04 15:35:17 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-08-04 18:33:46 -0400 |
| commit | 07868201070d87484bd00610a4921e879be78746 (patch) | |
| tree | 262e32d1261bf05280a9711f6aa951550343979e | |
| parent | a40694a38a745af0dd7d8b796597ada1dd6caeb7 (diff) | |
flex_array: remove unneeded index calculation
flex_array_get() calculates an index value, then drops it on the floor;
simply remove it.
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Acked-by: Dave Hansen <dave@linux.vnet.ibm.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | lib/flex_array.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/flex_array.c b/lib/flex_array.c index 0e7894ce8882..08f1636d296a 100644 --- a/lib/flex_array.c +++ b/lib/flex_array.c | |||
| @@ -254,7 +254,6 @@ void *flex_array_get(struct flex_array *fa, int element_nr) | |||
| 254 | { | 254 | { |
| 255 | int part_nr = fa_element_to_part_nr(fa, element_nr); | 255 | int part_nr = fa_element_to_part_nr(fa, element_nr); |
| 256 | struct flex_array_part *part; | 256 | struct flex_array_part *part; |
| 257 | int index; | ||
| 258 | 257 | ||
| 259 | if (element_nr >= fa->total_nr_elements) | 258 | if (element_nr >= fa->total_nr_elements) |
| 260 | return NULL; | 259 | return NULL; |
| @@ -264,6 +263,5 @@ void *flex_array_get(struct flex_array *fa, int element_nr) | |||
| 264 | part = (struct flex_array_part *)&fa->parts[0]; | 263 | part = (struct flex_array_part *)&fa->parts[0]; |
| 265 | else | 264 | else |
| 266 | part = fa->parts[part_nr]; | 265 | part = fa->parts[part_nr]; |
| 267 | index = index_inside_part(fa, element_nr); | ||
| 268 | return &part->elements[index_inside_part(fa, element_nr)]; | 266 | return &part->elements[index_inside_part(fa, element_nr)]; |
| 269 | } | 267 | } |
