Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

Filelocking bugfix #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

csm101
Copy link

@csm101 csm101 commented Oct 21, 2016

File sharing policy must be passed in the SECOND parameter of TFileStream.Create, OR-ed with the open mode flags.

the "Rights" parameter (the third) is meant for the "rwxrwxrwx" protection bits used by unix filesystems and is ignored under Windows (see note at the end of documentation page)

A secondary minor correction (not actually a bug): FreeAndNil(x) already checks for "if x=nil": there is no need to checking it before calling.

@csm101
Copy link
Author

csm101 commented Oct 21, 2016

opps... I noticed right now that this pull request contains also the changes I made for my other pull request... Sorry I am not very experienced with git, I am still trying to figure out some things..

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant