Closed Bug 1510614 Opened 6 years ago Closed 8 months ago

Lint Warning: Default Parcel Class Loader

Categories

(GeckoView :: General, enhancement, P5)

Unspecified
Android
enhancement

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: fluffyemily, Unassigned)

References

(Blocks 1 open bug)

Details

The documentation for Parcel#readParcelable(ClassLoader) (and its variations) says that you can pass in null to pick up the default class loader. However, that ClassLoader is a system class loader and is not able to find classes in your own application. If you are writing your own classes into the Parcel (not just SDK classes like String and so on), then you should supply a ClassLoader for your application instead; a simple way to obtain one is to just call getClass().getClassLoader() from your own class. http://developer.android.com/reference/android/os/Parcel.html This issue currently affects: FormatParam.java
Keywords: good-first-bug
OS: Unspecified → Android
Priority: -- → P5
Product: Firefox for Android → GeckoView
Severity: normal → S3

Enhancements should have severity N/A.

Severity: S3 → N/A

We think this was causing lint issues 6 years ago, but the list of lint issues has changed in that time.
We are making another meta that covers a more up to date list of lint issues: https://bugzilla.mozilla.org/show_bug.cgi?id=1906885
Closing this one out as WONTFIX.

Status: NEW → RESOLVED
Closed: 8 months ago
Keywords: good-first-bug
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.