12:49 PM Sub, 21.03.2020. Philippe Mathieu-Daudé <phi...@redhat.com> је написао/ла: > > Fix warning reported by Clang static code analyzer: > > block.c:3167:5: warning: Value stored to 'ret' is never read > ret = bdrv_fill_options(&options, filename, &flags, &local_err); > ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > Reported-by: Clang Static Analyzer
Peter, and others, Is this allowed use of "Reported-by:" mark? I did not notice it being used this way before. And was under impression that all similar tags/marks must be followed by a person, not a tool. Regards, Aleksandar > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> > --- > block.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/block.c b/block.c > index a2542c977b..908c109a8c 100644 > --- a/block.c > +++ b/block.c > @@ -3164,7 +3164,7 @@ static BlockDriverState *bdrv_open_inherit(const char *filename, > parent->open_flags, parent->options); > } > > - ret = bdrv_fill_options(&options, filename, &flags, &local_err); > + bdrv_fill_options(&options, filename, &flags, &local_err); > if (local_err) { > goto fail; > } > -- > 2.21.1 > >