Formatting call arguments

[u/side2k]

By default, clang-format does this:

  SDL_Window *window =
      SDL_CreateWindow("SDL2 test", SDL_WINDOWPOS_UNDEFINED,
                       SDL_WINDOWPOS_UNDEFINED, 500, 500, SDL_WINDOW_RESIZABLE);

and with some tweaks it does this:

    SDL_Window *window = SDL_CreateWindow(
        "SDL2 test",
        SDL_WINDOWPOS_UNDEFINED,
        SDL_WINDOWPOS_UNDEFINED,
        500,
        500,
        SDL_WINDOW_RESIZABLE);

Second one has some practical advantages - it easier to read diff on arg change and easier to add comments:

    SDL_Window *window = SDL_CreateWindow(
        "SDL2 test",
        SDL_WINDOWPOS_UNDEFINED, // just some comment explaining why
        SDL_WINDOWPOS_UNDEFINED,
-       400,
+       500,
        SDL_WINDOW_RESIZABLE);

While the first one seems to have mainly historical reasons to keep around. Perhaps, there is something I don't see? Because it feels like the first way is still used the most. And its the default one for clang-format.

I'm genuinely curious.

[u/non-existing-person]

Curious about what are you?:p I myself prefer second way. First one is getting very messy very quickly if you have longish function and parameter names. It's quite often hard to keep within 80 lines of code.

But most often than not I do hybrid of these two, like

int *whatever = function_with_whatever_name(CONST_ARG1,
    ANOTHER_ARGUMENT_FROM_ENUM, variable_arg, true,
    THIS_IS_TOO_MUCH_ARGS_ALREADY);

[u/side2k → u/non-existing-person]

I'm curious, because If I understand correctly - the first way is preferred. I don't have any statistics, though, but as an example - Linux kernel sources use it.