Skip to content
This repository has been archived by the owner on Sep 2, 2021. It is now read-only.

Use stat on file systems that do not support d_type in file entry record returned from readdir. #369

Merged
merged 2 commits into from
Nov 6, 2013

Conversation

RaymondLim
Copy link
Contributor

This fixes adobe/brackets#5292.

@ingorichter
Copy link
Contributor

@RaymondLim which filesystem doesn't support d_type? I'd like to test and verify your PR. Thank you.

@RaymondLim
Copy link
Contributor Author

REISERFS https://github.com/adobe/brackets/issues/#5758 and SSHFS https://github.com/adobe/brackets/issues/#5544. Not sure how you're going to test with these filesystems. But you can always test with your existing filesystem by commenting out the original code and just use the new call stat for all file entries.

Update: http://en.wikipedia.org/wiki/Category:Linux_file_systems lists all the available file systems for Linux. Not sure what file systems are used for those reporting in adobe/brackets#5292.

@ghost ghost assigned ingorichter Nov 4, 2013
ingorichter added a commit that referenced this pull request Nov 6, 2013
Use stat on file systems that do not support d_type in file entry record returned from readdir.
@ingorichter ingorichter merged commit 7e1ae1a into master Nov 6, 2013
@ingorichter
Copy link
Contributor

Looks good. Thank you.

@ingorichter ingorichter deleted the rlim/readdir-issue branch November 6, 2013 20:30
ingorichter added a commit that referenced this pull request Dec 8, 2015
Use stat on file systems that do not support d_type in file entry record returned from readdir.
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.

No files in folder list
2 participants