SE_NO_SERIALVERSIONID with Serializable records

[rovarga]

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.

[hazendaz]

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.

[gtoison]

My understanding is that serialVersionUID is not considered when loading a record:
the requirement for matching the serialVersionUID value is waived for record classes

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

	isEnum = "java.lang.Enum".equals(superClassname);
	if (isEnum) {
	return;
	}

It also seems to report SE_NO_SUITABLE_CONSTRUCTOR_FOR_EXTERNALIZATION which is not great because records are not externalizable anyway (so there might be another bug to report: a record should not implement Externalizable)

[hazendaz]

Fixed via #2795