Hi,

On 20/09/2022 18:42, Gert Doering wrote:
Hi,

On Mon, Sep 19, 2022 at 12:06:18AM +0200, Antonio Quartulli wrote:
+    switch (session->opt->push_peer_info_detail)
      {
-        /* push version */
-        buf_printf(&out, "IV_VER=%s\n", PACKAGE_VERSION);
+        case 3:
+        {
...
+        }
+ /* fall through */

These {} brackets are not needed by C or our style guide.

they are required when the first instruction after a case label is a variable declaration. So this is needed for case 3, but could be avoided for the other 2.

Cheers,

--
Antonio Quartulli


_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to