Reviewbot does not provide a very actionable message for clang-format. It is confusing because it gives a command that does not point to a real file. I would prefer a single command that I could copy, paste, and run from my terminal to fix the problem. For example, right now it lists:
./mach clang-format -s -p path/to/file.cpp
This command is confusing for two reasons:
- The path/to/file.cpp is not a real file, but looks like one.
- The command outputs a patch, but does not actually fix the problem.
At this point I have no single command I can run to just fix the problem.
I would prefer this command listed:
./mach clang-format -p toolkit/xre/nsEmbedFunctions.cpp
./mach clang-format -p js/xpconnect/src/XPCShellImpl.cpp
Or perhaps a one liner for the
hg import command.