fs: Use a temporary variable for the return value

[alejandro-colomar]

This avoids breaking a long line.

Cc: @ac000

[alejandro-colomar]

v2 changes:

• Keep the array the last.

$ git range-diff master gh/tmp tmp
1:  be380374 ! 1:  2419b055 Fs: Use a temporary variable for the return value
    @@ src/nxt_fs.c: static nxt_int_t nxt_fs_mkdir(const u_char *dir, mode_t mode);
     -    char    path[PATH_MAX];
     +    char       *start, *end, *dst;
     +    size_t     dirlen;
    -+    char       path[PATH_MAX];
     +    nxt_int_t  ret;
    ++    char       path[PATH_MAX];

          dirlen = nxt_strlen(dir);

[alejandro-colomar]

v2b changes:

• lowcase

$ git range-diff master gh/tmp tmp
1:  2419b055 ! 1:  69f56464 Fs: Use a temporary variable for the return value
    @@ Metadata
     Author: Alejandro Colomar <alx@kernel.org>

      ## Commit message ##
    -    Fs: Use a temporary variable for the return value
    +    Fs: use a temporary variable for the return value

         This avoids breaking a long line.

[alejandro-colomar]

v2c changes:

• case

$ git range-diff master gh/tmp tmp
1:  69f56464 ! 1:  a5289501 Fs: use a temporary variable for the return value
    @@ Metadata
     Author: Alejandro Colomar <alx@kernel.org>

      ## Commit message ##
    -    Fs: use a temporary variable for the return value
    +    fs: Use a temporary variable for the return value

         This avoids breaking a long line.

[alejandro-colomar]

How did that test fail?

https://github.com/nginx/unit/actions/runs/8805812704/job/24169230155

This is too trivial to have broken anything. :D

[ac000]

That's a common failure, I've set that job to run again...

[ac000]

Bingo!

[alejandro-colomar]

Closing in favour of https://github.com/nginx/unit/pull/1235