fix missing braces for for statements

Signed-off-by: Sheldon Wu <[email protected]>

Author: sheldnwu

generated/dpll-user.cpp

@@ -1117,18 +1117,21 @@ int dpll_pin_set(ynl_cpp::ynl_socket& ys, dpll_pin_set_req& req)
 	if (req.state.has_value()) {
 		ynl_attr_put_u32(nlh, DPLL_A_PIN_STATE, req.state.value());
 	}
-	for (unsigned int i = 0; i < req.parent_device.size(); i++)
+	for (unsigned int i = 0; i < req.parent_device.size(); i++) {
 		dpll_pin_parent_device_put(nlh, DPLL_A_PIN_PARENT_DEVICE, req.parent_device[i]);
-	for (unsigned int i = 0; i < req.parent_pin.size(); i++)
+	}
+	for (unsigned int i = 0; i < req.parent_pin.size(); i++) {
 		dpll_pin_parent_pin_put(nlh, DPLL_A_PIN_PARENT_PIN, req.parent_pin[i]);
+	}
 	if (req.phase_adjust.has_value()) {
 		ynl_attr_put_s32(nlh, DPLL_A_PIN_PHASE_ADJUST, req.phase_adjust.value());
 	}
 	if (req.esync_frequency.has_value()) {
 		ynl_attr_put_u64(nlh, DPLL_A_PIN_ESYNC_FREQUENCY, req.esync_frequency.value());
 	}
-	for (unsigned int i = 0; i < req.reference_sync.size(); i++)
+	for (unsigned int i = 0; i < req.reference_sync.size(); i++) {
 		dpll_reference_sync_put(nlh, DPLL_A_PIN_REFERENCE_SYNC, req.reference_sync[i]);
+	}
 
 	err = ynl_exec(ys, nlh, &yrs);
 	if (err < 0) {

generated/ethtool-user.cpp

@@ -2492,8 +2492,9 @@ int ethtool_profile_put(struct nlmsghdr *nlh, unsigned int attr_type,
 	struct nlattr *nest;
 
 	nest = ynl_attr_nest_start(nlh, attr_type);
-	for (unsigned int i = 0; i < obj.irq_moderation.size(); i++)
+	for (unsigned int i = 0; i < obj.irq_moderation.size(); i++) {
 		ethtool_irq_moderation_put(nlh, ETHTOOL_A_PROFILE_IRQ_MODERATION, obj.irq_moderation[i]);
+	}
 	ynl_attr_nest_end(nlh, nest);
 
 	return 0;
@@ -2652,8 +2653,9 @@ int ethtool_bitset_bits_put(struct nlmsghdr *nlh, unsigned int attr_type,
 	struct nlattr *nest;
 
 	nest = ynl_attr_nest_start(nlh, attr_type);
-	for (unsigned int i = 0; i < obj.bit.size(); i++)
+	for (unsigned int i = 0; i < obj.bit.size(); i++) {
 		ethtool_bitset_bit_put(nlh, ETHTOOL_A_BITSET_BITS_BIT, obj.bit[i]);
+	}
 	ynl_attr_nest_end(nlh, nest);
 
 	return 0;
@@ -2706,8 +2708,9 @@ int ethtool_strings_put(struct nlmsghdr *nlh, unsigned int attr_type,
 	struct nlattr *nest;
 
 	nest = ynl_attr_nest_start(nlh, attr_type);
-	for (unsigned int i = 0; i < obj.string.size(); i++)
+	for (unsigned int i = 0; i < obj.string.size(); i++) {
 		ethtool_string_put(nlh, ETHTOOL_A_STRINGS_STRING, obj.string[i]);
+	}
 	ynl_attr_nest_end(nlh, nest);
 
 	return 0;
@@ -2843,8 +2846,9 @@ int ethtool_stringset_put(struct nlmsghdr *nlh, unsigned int attr_type,
 	if (obj.count.has_value()) {
 		ynl_attr_put_u32(nlh, ETHTOOL_A_STRINGSET_COUNT, obj.count.value());
 	}
-	for (unsigned int i = 0; i < obj.strings.size(); i++)
+	for (unsigned int i = 0; i < obj.strings.size(); i++) {
 		ethtool_strings_put(nlh, ETHTOOL_A_STRINGSET_STRINGS, obj.strings[i]);
+	}
 	ynl_attr_nest_end(nlh, nest);
 
 	return 0;
@@ -2963,8 +2967,9 @@ int ethtool_stringsets_put(struct nlmsghdr *nlh, unsigned int attr_type,
 	struct nlattr *nest;
 
 	nest = ynl_attr_nest_start(nlh, attr_type);
-	for (unsigned int i = 0; i < obj.stringset.size(); i++)
+	for (unsigned int i = 0; i < obj.stringset.size(); i++) {
 		ethtool_stringset_put(nlh, ETHTOOL_A_STRINGSETS_STRINGSET, obj.stringset[i]);
+	}
 	ynl_attr_nest_end(nlh, nest);
 
 	return 0;

generated/handshake-user.cpp

@@ -302,8 +302,9 @@ int handshake_done(ynl_cpp::ynl_socket& ys, handshake_done_req& req)
 	if (req.sockfd.has_value()) {
 		ynl_attr_put_s32(nlh, HANDSHAKE_A_DONE_SOCKFD, req.sockfd.value());
 	}
-	for (unsigned int i = 0; i < req.remote_auth.size(); i++)
+	for (unsigned int i = 0; i < req.remote_auth.size(); i++) {
 		ynl_attr_put_u32(nlh, HANDSHAKE_A_DONE_REMOTE_AUTH, req.remote_auth[i]);
+	}
 
 	err = ynl_exec(ys, nlh, &yrs);
 	if (err < 0) {

generated/net_shaper-user.cpp

@@ -506,8 +506,9 @@ net_shaper_group(ynl_cpp::ynl_socket& ys, net_shaper_group_req& req)
 	if (req.weight.has_value()) {
 		ynl_attr_put_u32(nlh, NET_SHAPER_A_WEIGHT, req.weight.value());
 	}
-	for (unsigned int i = 0; i < req.leaves.size(); i++)
+	for (unsigned int i = 0; i < req.leaves.size(); i++) {
 		net_shaper_leaf_info_put(nlh, NET_SHAPER_A_LEAVES, req.leaves[i]);
+	}
 
 	rsp.reset(new net_shaper_group_rsp());
 	yrs.yarg.data = rsp.get();

generated/netdev-user.cpp

@@ -1249,8 +1249,9 @@ netdev_bind_rx(ynl_cpp::ynl_socket& ys, netdev_bind_rx_req& req)
 	if (req.fd.has_value()) {
 		ynl_attr_put_u32(nlh, NETDEV_A_DMABUF_FD, req.fd.value());
 	}
-	for (unsigned int i = 0; i < req.queues.size(); i++)
+	for (unsigned int i = 0; i < req.queues.size(); i++) {
 		netdev_queue_id_put(nlh, NETDEV_A_DMABUF_QUEUES, req.queues[i]);
+	}
 
 	rsp.reset(new netdev_bind_rx_rsp());
 	yrs.yarg.data = rsp.get();

generated/nfsd-user.cpp

@@ -393,8 +393,9 @@ int nfsd_threads_set(ynl_cpp::ynl_socket& ys, nfsd_threads_set_req& req)
 	nlh = ynl_gemsg_start_req(ys, ((struct ynl_sock*)ys)->family_id, NFSD_CMD_THREADS_SET, 1);
 	((struct ynl_sock*)ys)->req_policy = &nfsd_server_nest;
 
-	for (unsigned int i = 0; i < req.threads.size(); i++)
+	for (unsigned int i = 0; i < req.threads.size(); i++) {
 		ynl_attr_put_u32(nlh, NFSD_A_SERVER_THREADS, req.threads[i]);
+	}
 	if (req.gracetime.has_value()) {
 		ynl_attr_put_u32(nlh, NFSD_A_SERVER_GRACETIME, req.gracetime.value());
 	}
@@ -501,8 +502,9 @@ int nfsd_version_set(ynl_cpp::ynl_socket& ys, nfsd_version_set_req& req)
 	nlh = ynl_gemsg_start_req(ys, ((struct ynl_sock*)ys)->family_id, NFSD_CMD_VERSION_SET, 1);
 	((struct ynl_sock*)ys)->req_policy = &nfsd_server_proto_nest;
 
-	for (unsigned int i = 0; i < req.version.size(); i++)
+	for (unsigned int i = 0; i < req.version.size(); i++) {
 		nfsd_version_put(nlh, NFSD_A_SERVER_PROTO_VERSION, req.version[i]);
+	}
 
 	err = ynl_exec(ys, nlh, &yrs);
 	if (err < 0) {
@@ -591,8 +593,9 @@ int nfsd_listener_set(ynl_cpp::ynl_socket& ys, nfsd_listener_set_req& req)
 	nlh = ynl_gemsg_start_req(ys, ((struct ynl_sock*)ys)->family_id, NFSD_CMD_LISTENER_SET, 1);
 	((struct ynl_sock*)ys)->req_policy = &nfsd_server_sock_nest;
 
-	for (unsigned int i = 0; i < req.addr.size(); i++)
+	for (unsigned int i = 0; i < req.addr.size(); i++) {
 		nfsd_sock_put(nlh, NFSD_A_SERVER_SOCK_ADDR, req.addr[i]);
+	}
 
 	err = ynl_exec(ys, nlh, &yrs);
 	if (err < 0) {

ynl-gen-cpp.py

@@ -649,18 +649,24 @@ def _attr_get(self, ri, var):
     def attr_put(self, ri, var):
         if self.attr["type"] in scalars:
             put_type = self.type
-            ri.cw.p(f"for (unsigned int i = 0; i < {var}.{self.c_name}.size(); i++)")
+            ri.cw.block_start(
+                line=f"for (unsigned int i = 0; i < {var}.{self.c_name}.size(); i++)"
+            )
             ri.cw.p(
                 f"ynl_attr_put_{put_type}(nlh, {self.enum_name}, {var}.{self.c_name}[i]);"
             )
+            ri.cw.block_end()
         elif "type" not in self.attr or self.attr["type"] == "nest":
-            ri.cw.p(f"for (unsigned int i = 0; i < {var}.{self.c_name}.size(); i++)")
+            ri.cw.block_start(
+                line=f"for (unsigned int i = 0; i < {var}.{self.c_name}.size(); i++)"
+            )
             self._attr_put_line(
                 ri,
                 var,
                 f"{self.nested_render_name}_put(nlh, "
                 + f"{self.enum_name}, {var}.{self.c_name}[i])",
             )
+            ri.cw.block_end()
         else:
             raise Exception(
                 f"Put of MultiAttr sub-type {self.attr['type']} not supported yet"