Skip to content
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

Fixed FFat::end. Fixes #3244 #3245

Merged
merged 3 commits into from Sep 24, 2019

Conversation

@lbernstone
Copy link
Contributor

commented Sep 22, 2019

_wl_handle was not being cleared properly

@lbernstone

This comment has been minimized.

Copy link
Contributor Author

commented Sep 24, 2019

I'll work on a way to do a second begin & cleaning up the file objects this weekend in a separate PR. Go ahead and merge this if there are no more issues.

@mouridis

This comment has been minimized.

Copy link
Contributor

commented Sep 24, 2019

@lbernstone Did you check my comment above about this PR breaking FFat.format()?

@lbernstone

This comment has been minimized.

Copy link
Contributor Author

commented Sep 24, 2019

@mouridis Yes, I put in another commit that fixed format.

@me-no-dev me-no-dev merged commit ca88fdc into espressif:master Sep 24, 2019
20 checks passed
20 checks passed
Build Arduino IDE Tests 0
Details
Build Arduino IDE Tests 1
Details
Build Arduino IDE Tests 2
Details
Build Arduino IDE Tests 3
Details
Build Arduino IDE Tests 4
Details
Build Arduino IDE Tests 5
Details
Build Arduino IDE Tests 6
Details
Build Arduino IDE Tests 7
Details
Build Arduino IDE Tests 8
Details
Build Arduino IDE Tests 9
Details
Build Arduino IDE Tests 10
Details
Build Arduino IDE Tests 11
Details
Build Arduino IDE Tests 12
Details
Build Arduino IDE Tests 13
Details
Build Arduino IDE Tests 14
Details
Build Arduino IDE Tests 15
Details
Build Arduino IDE Tests 16
Details
Build Arduino IDE Tests 17
Details
Build PlatformIO Tests
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.