Skip to content

fix 'fileExists' check by using stat directly#7039

Merged
vladima merged 2 commits into
masterfrom
fixFileExists
Feb 12, 2016
Merged

fix 'fileExists' check by using stat directly#7039
vladima merged 2 commits into
masterfrom
fixFileExists

Conversation

@vladima
Copy link
Copy Markdown
Contributor

@vladima vladima commented Feb 12, 2016

fixes #7037

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Feb 12, 2016

any perf impact?

Comment thread src/compiler/sys.ts
File,
Directory
}
function fileSystemEntryExists(path: string, entryKind: FileSystemEntryKind): boolean {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a newline

@vladima
Copy link
Copy Markdown
Contributor Author

vladima commented Feb 12, 2016

got an offline signoff by @mhegazy

vladima added a commit that referenced this pull request Feb 12, 2016
fix 'fileExists' check by using stat directly
@vladima vladima merged commit 35eff19 into master Feb 12, 2016
@vladima vladima deleted the fixFileExists branch February 12, 2016 04:47
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
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.

'big.js': EISDIR: illegal operation on a directory, read

4 participants