Skip to content

File size is not actually checked on Windows #52

Open

Description

@Sh4rK

I was checking out the source of this library purely out of interest, and while the comments and docs say at a few places that file size is compared on windows to avoid some false positives, this does not actually seem to be the case.

The key used for comparison on Windows is

struct Key {
    volume: u64,
    index: u64,
}

which does not include the file size, nor could I find any other place where file size is taken into account.

Activity

  1. BurntSushi commented on May 18, 2021

    @BurntSushi
    Owner

    Hmmm. It looks like it got dropped a long time ago in this commit: bbf6959

    Sadly, I did not mention it in the commit message and I have no idea now why I dropped the file size comparison. That is not up to my usual standards, sorry about that.

    I think in principle I'd be okay adding it back. I can't think of a reason why I dropped it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions