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

Liversage picture Liversage  路  4Comments

luciobemquerer picture luciobemquerer  路  4Comments

SebastianStehle picture SebastianStehle  路  4Comments

pherbel picture pherbel  路  4Comments

Vlad-Stryapko picture Vlad-Stryapko  路  3Comments