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 <[email protected]>
> Signed-off-by: Chen Qun <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>

Acked-by: Gerd Hoffmann <[email protected]>


Reply via email to