Wrong data returned [264729367]

Fixed

Bug

Status Update

No update yet.

Description

sc...@gmail.com

created issue #1

Jan 8, 2023 10:14AM

Component used: androidx.exifinterface:exifinterface
Version used: 1.3.5
Devices/Android versions reproduced on: Android Emulator API30

When using ExifIntreface on the attached picture, an exifinterface instance is returned, but further reading EXIF attributes yield wrong result, like getAttributeInt(ExifInterface.TAG_IMAGE_WIDTH, 0) returns 160, getAttributeInt(ExifInterface.TAG_IMAGE_LENGTH, 0) returns 120, and ExifIntreface.dateTimeOriginal returns null.

211014041-273c2a1d-74db-42fa-b050-67ded71be015.jpg

3.3 MB

View

Download

Comments

sc...@gmail.com <sc...@gmail.com> #2Feb 11, 2023 02:50PM

What about Enums? Enum is a type-safe way to pass navigation arguments.
Will Enums be covered in this issue as part of Serializable or will have own handled separately?
Because for enum you can provide a reasonable default value but for Parcelable or Serializable only @null

ib...@google.com <ib...@google.com> #3Feb 22, 2023 04:40PM

Please add support for Serializable objects so Enums can be used.

Message last modified on Feb 22, 2023 04:41PM

ap...@google.com <ap...@google.com> #4Mar 2, 2023 04:33PM

Support of Serializable objects is not enough for enums, you definitely need also the support of default values.

ap...@google.com <ap...@google.com> #5Mar 2, 2023 04:33PM

Project: platform/frameworks/support
Branch: androidx-master-dev

commit 6f59e3d74d0da5a4bf79c8480f56964e3af47f1c
Author: Wojtek Kaliciński <wkal@google.com>
Date: Mon Jul 23 20:17:19 2018

Support more types for args

Provides support for Serializable, enums,
as well as arrays of primitive types and Parcelables
in Navigation arguments.

Only enums support default values other than @null,
in the form of the enum literal (without class name),
e.g. app:defaultValue="READ"

Test: unit tests updated
BUG: 111487504
BUG: 111316353
Change-Id: I7e5abbc8ed0950bfdef292342dbe1fb34a5c4b17

M navigation/runtime/src/androidTest/java/androidx/navigation/NavInflaterTest.kt
A navigation/runtime/src/androidTest/java/androidx/navigation/test/TestEnum.java
M navigation/runtime/src/androidTest/res/navigation/nav_default_arguments.xml
M navigation/runtime/src/main/java/androidx/navigation/NavInflater.java
M navigation/safe-args-generator/src/main/kotlin/androidx/navigation/safe/args/generator/NavParser.kt
M navigation/safe-args-generator/src/main/kotlin/androidx/navigation/safe/args/generator/NavParserErrors.kt
M navigation/safe-args-generator/src/main/kotlin/androidx/navigation/safe/args/generator/NavWriter.kt
M navigation/safe-args-generator/src/main/kotlin/androidx/navigation/safe/args/generator/Types.kt
M navigation/safe-args-generator/src/tests/kotlin/androidx/navigation/safe/args/generator/NavParserTest.kt
M navigation/safe-args-generator/src/tests/kotlin/androidx/navigation/safe/args/generator/NavWriterTest.kt
M navigation/safe-args-generator/src/tests/test-data/expected/nav_writer_test/MainFragmentArgs.java
M navigation/safe-args-generator/src/tests/test-data/expected/nav_writer_test/Next.java
M navigation/safe-args-generator/src/tests/test-data/naive_test.xml

https://android-review.googlesource.com/720246

https://goto.google.com/android-sha1/6f59e3d74d0da5a4bf79c8480f56964e3af47f1c

ib...@google.com <ib...@google.com> Mar 2, 2023 04:39PM

Marked as fixed.

na...@google.com <na...@google.com> #6Dec 13, 2023 06:33PM

This is fixed internally and will be available in 1.0.0-alpha08

na...@google.com <na...@google.com> #7Nov 13, 2024 06:19PM

What happened if I declaire app:argType="androidx.navigation.test.TestEnum" and turn on proguard/R8? Because both ProGuard and R8 will optimize trivial enums into ints.