-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Microlib: Disable feof function usage in general filesystem test #12111
Conversation
@@ -105,7 +105,12 @@ int lfs_emubd_read(const struct lfs_config *cfg, lfs_block_t block, | |||
} | |||
|
|||
size_t res = fread(data, 1, size, f); | |||
if (res < size && !feof(f)) { | |||
if (res < size | |||
#if !defined(__MICROLIB) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if this && !feof(f)
test should ever be there.
Why would it be valid to attempt to read off the end of the file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed with Kevin this feof check seems to be wrong but I will revert this changes as build system ignores this emubd due to .mbedignore.
@rajkan01, thank you for your changes. |
a3c0670
to
9c513bc
Compare
9c513bc
to
890c692
Compare
feof function is not supported in Microlib library. so disabling all the feof function usage in the green tea test.
890c692
to
95f56ba
Compare
CI started |
Test run: SUCCESSSummary: 5 of 5 test jobs passed |
Summary of changes
feof function is not supported in Microlib library refer "http://arminfo.emea.arm.com/help/index.jsp?topic=/com.arm.doc.100073_0612_00_en/chr1358938938431.html" for more info. so disabling all the feof function usage in the green tea test.
Impact of changes
Migration actions required
Documentation
Pull request type
Test results
Reviewers
@evedon @hugueskamba