Orleans: PVS-Studio: excessive prefix check - StartsWith("boot") check also covers "bootstrap" cases

Created on 6 Feb 2016  路  3Comments  路  Source: dotnet/orleans

We can get rid of first check as second check covers all cases which first one does.

V3053 An excessive expression. Examine the substrings '/bootstrap' and '/boot'. ClientGenerator.cs 310

 else if (arg.StartsWith("/bootstrap") || arg.StartsWith("/boot"))
hacktoberfest help wanted

All 3 comments

This is very much benign, but we should get rid of one and probably be more strict in parsing codegen args.

I think this issue no longer exits. :)

Thanks, @noamyogev84!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

bwanner picture bwanner  路  5Comments

Vlad-Stryapko picture Vlad-Stryapko  路  3Comments

jt4000 picture jt4000  路  3Comments

DixonDs picture DixonDs  路  4Comments

danvanderboom picture danvanderboom  路  3Comments