-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
there are three suspicious bugs #175
Comments
Technically the latter two would find the "last" entry .. although your finding is probably correct. I'll see about getting this in. Thanks for spotting these ;) |
And yes. The first one should be the next index being tested. It's collapsing data runs. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
hi, i'm porting DiscUtils to Java.
when i used SpotBugs, it pointed out 3 suspicious bugs.
1. RpC: Repeated conditional tests (RpC_REPEATED_CONDITIONAL_TEST)
DiscUtils/Library/DiscUtils.Ntfs/CookedDataRuns.cs
Line 284 in b3ffb51
_runs
at the second condition might bei + 1
, right ???2. This method DiscUtils.Core.LogicalDiskManager.DynamicDisk.GetPrivateHeader(VirtualDisk) continues a loop after finding an equality condition
DiscUtils/Library/DiscUtils.Core/LogicalDiskManager/DynamicDisk.cs
Lines 103 to 109 in b3ffb51
3. This method DiscUtils.Core.Partitions.BiosExtendedPartitionTable.GetMetadataDiskExtents() continues a loop after finding an equality condition
DiscUtils/Library/DiscUtils.Core/Partitions/BiosExtendedPartitionTable.cs
Lines 100 to 111 in b3ffb51
thanks,
The text was updated successfully, but these errors were encountered: