<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 22, 2020 at 3:43 PM Petr Vorel <<a href="mailto:petr.vorel@suse.com">petr.vorel@suse.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">From: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
<br>
Adding bogus return to keep compiler happy.<br>
<br>
Fixes: 3b716981b ("lib: add new cgroup test API")<br>
<br>
Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
Hi,<br>
<br>
IMHO using pragma would be better, but I don't know how to write it portable way.<br>
<br>
Kind regards,<br>
Petr<br>
<br>
 lib/tst_cgroup.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c<br>
index f55d8818d..0118dd7b2 100644<br>
--- a/lib/tst_cgroup.c<br>
+++ b/lib/tst_cgroup.c<br>
@@ -49,6 +49,7 @@ enum tst_cgroup_ver tst_cgroup_version(void)<br>
                return TST_CGROUP_V1;<br>
<br>
        tst_brk(TCONF, "Cgroup is not configured");<br>
+       return TST_CGROUP_V1; /* fix -Werror=return-type */<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">This return looks strange since it will never go to here.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">How about this?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">enum tst_cgroup_ver tst_cgroup_version(void)<br>{<br>        enum tst_cgroup_ver cg_ver;<br><br>        if (tst_cgroup_check("cgroup2")) {<br>                if (!tst_is_mounted("cgroup2") && tst_is_mounted("cgroup"))<br>                        cg_ver = TST_CGROUP_V1;<br>                else<br>                        cg_ver = TST_CGROUP_V2;<br><br>                goto out;<br>        }<br><br>        if (tst_cgroup_check("cgroup"))<br>                cg_ver = TST_CGROUP_V1;<br><br>        if (!tg_ver)<br>                tst_brk(TCONF, "Cgroup is not configured");<br><br>out:<br>        return cg_ver;<br>}<br></div><br></div><div> </div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>