summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaul Elder <paul.elder@ideasonboard.com>2021-03-08 19:01:22 +0900
committerPaul Elder <paul.elder@ideasonboard.com>2021-03-10 12:30:51 +0900
commitd3cf0fed3e3139a8ae4b93fe661d4603e1b1115f (patch)
treed465cca36f29c2780344f0d0cad677b3da1cfe30
parenta3c9718cfcefbf5601b54ff1609cdc400c482744 (diff)
android: jpeg: exif: change GPS method encoding from ASCII to NoEncoding
According to the EXIF specification, the GPS method should be UNDEFINED, and the first 8 bytes will designate the type. However, CTS expects the first 8 bytes to be part of the data. Remove the 8-byte encoding designator by changing the encoding to NoEncoding to appease CTS. This is part of the fix that allows the following CTS test to pass: - android.hardware.cts.CameraTest#testJpegExif Signed-off-by: Paul Elder <paul.elder@ideasonboard.com> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
-rw-r--r--src/android/jpeg/exif.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/android/jpeg/exif.cpp b/src/android/jpeg/exif.cpp
index bd359f98..922086cd 100644
--- a/src/android/jpeg/exif.cpp
+++ b/src/android/jpeg/exif.cpp
@@ -405,7 +405,7 @@ void Exif::setGPSLocation(const double *coords)
void Exif::setGPSMethod(const std::string &method)
{
setString(EXIF_IFD_GPS, static_cast<ExifTag>(EXIF_TAG_GPS_PROCESSING_METHOD),
- EXIF_FORMAT_UNDEFINED, method, Unicode);
+ EXIF_FORMAT_UNDEFINED, method, NoEncoding);
}
void Exif::setOrientation(int orientation)