chip: use a temporary variable in gpiod_chip_close()
authorBartosz Golaszewski <bartekgola@gmail.com>
Fri, 30 Jun 2017 14:28:30 +0000 (16:28 +0200)
committerBartosz Golaszewski <bartekgola@gmail.com>
Thu, 6 Jul 2017 09:11:36 +0000 (11:11 +0200)
The code looks better if we use a temporary pointer to gpiod_line.

Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>
src/lib/chip.c

index 51e64cccf5004bb83b8617701951d883d413ab87..fa82784c1c298d928d53feb45db9f3458474a2ae 100644 (file)
@@ -144,12 +144,15 @@ struct gpiod_chip * gpiod_chip_open_lookup(const char *descr)
 
 void gpiod_chip_close(struct gpiod_chip *chip)
 {
+       struct gpiod_line *line;
        unsigned int i;
 
        for (i = 0; i < chip->cinfo.lines; i++) {
-               if (chip->lines[i]) {
-                       gpiod_line_release(chip->lines[i]);
-                       line_free(chip->lines[i]);
+               line = chip->lines[i];
+
+               if (line) {
+                       gpiod_line_release(line);
+                       line_free(line);
                }
        }