On Mon, Mar 02, 2020 at 09:07:14PM +0800, Chen Qun wrote: > The "again" assignment is meaningless before g_assert_not_reached. > In addition, the break statements no longer needs to be after > g_assert_not_reached. > > Clang static code analyzer show warning: > hw/usb/hcd-ehci.c:2108:13: warning: Value stored to 'again' is never read > again = -1; > ^ ~~ > > Reported-by: Euler Robot <euler.ro...@huawei.com> > Signed-off-by: Chen Qun <kuhn.chen...@huawei.com> > Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Acked-by: Gerd Hoffmann <kra...@redhat.com>