Format readjson
clang-format may potentially reformat the readJson calls if they may
have more keys or key names are longer. This makes formatting in a way
that's readable by forcing to break a line for each key using an
empty-comment (`//`) each line.
It also allows trivially alphabetizing the list such that new additions
are less likely to have merge conflicts.
Tested:
- Check whitespace only.
- Code compiles.
- Redfish Service Validator with the same results before this
Change-Id: I3824a8c4faa9fa7c820d5d2fab6b565404926e2c
Signed-off-by: Ed Tanous <etanous@nvidia.com>
Signed-off-by: Myung Bae <myungbae@us.ibm.com>
diff --git a/redfish-core/lib/virtual_media.hpp b/redfish-core/lib/virtual_media.hpp
index a18f1d0..cd0d972 100644
--- a/redfish-core/lib/virtual_media.hpp
+++ b/redfish-core/lib/virtual_media.hpp
@@ -708,13 +708,16 @@
InsertMediaActionParams actionParams;
// Read obligatory parameters (url of image)
- if (!json_util::readJsonAction(
- req, asyncResp->res, "Image", actionParams.imageUrl,
- "WriteProtected", actionParams.writeProtected, "UserName",
- actionParams.userName, "Password", actionParams.password,
- "Inserted", actionParams.inserted, "TransferMethod",
- actionParams.transferMethod, "TransferProtocolType",
- actionParams.transferProtocolType))
+ if (!json_util::readJsonAction( //
+ req, asyncResp->res, //
+ "Image", actionParams.imageUrl, //
+ "Inserted", actionParams.inserted, //
+ "Password", actionParams.password, //
+ "TransferMethod", actionParams.transferMethod, //
+ "TransferProtocolType", actionParams.transferProtocolType, //
+ "UserName", actionParams.userName, //
+ "WriteProtected", actionParams.writeProtected //
+ ))
{
return;
}