fix build against Android NDK (API Level 21 is required)
ClosedPublic

Authored by mgallien on Apr 27 2017, 7:32 PM.

Details

Summary

fix build against Android NDK (API Level 21 is required)

Test Plan

Test compilation under linux and Android

Diff Detail

Repository
R286 KFileMetaData
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mgallien created this revision.Apr 27 2017, 7:32 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptApr 27 2017, 7:32 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
apol added a subscriber: apol.Apr 27 2017, 10:00 PM

+1 LGTM.

Maybe you can specify on the commit message that attr/xattr.h isn't present on the Android NDK and add a comment on the ifdef wouldn't hurt either.

mgallien updated this revision to Diff 13901.Apr 28 2017, 5:06 AM

added a comment and also modify the commit message

apol accepted this revision.May 8 2017, 10:43 AM
This revision is now accepted and ready to land.May 8 2017, 10:43 AM
This revision was automatically updated to reflect the committed changes.