-
Notifications
You must be signed in to change notification settings - Fork 578
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
SE_NO_SERIALVERSIONID with Serializable records #2793
Comments
I guess in theory true its not an error, but it could be a proper warning. It clearly states in there that sometimes you do need to do so in certain situations or that is how I read it. Maybe it needs its own classification though since it sets to 0L if not listed. |
My understanding is that There's already a special case to ignore enums, I think it should be the same records spotbugs/spotbugs/src/main/java/edu/umd/cs/findbugs/detect/SerializableIdiom.java Lines 172 to 175 in 3da060e
It also seems to report |
Fixed via #2795 |
Having a Serializable record without a serialVersionUID constant triggers SE_NO_SERIALVERSIONID.
This is essentially a false positive, as records are special to serialization -- SpotBugs should not report this warning for records.
The text was updated successfully, but these errors were encountered: