Gian-Carlo Pascutto [:gcp] Reporter
	Description • 13 years ago

There's a whole bunch of those:

/home/morbo/hg/mozilla-central/mobile/android/base/GeckoSmsManager.java:583: warning: [serial] serializable class IdTooHighException has no definition of serialVersionUID
    class IdTooHighException extends Exception { }

They're warnings, not errors. But given that this is currently *the only thing* in the entire project that's spamming warnings, fixing it would be appreciated.

	Lucas Rocha (:lucasr)
	Comment 1 • 13 years ago

I'm not seeing those warnings anymore. Closing.

	Brad Lassey [:blassey] (use needinfo?)
	Comment 2 • 13 years ago

That's because the feature was disabled be default. You'd see them again if you enable it.

	Vicamo Yang [:vicamo][:vyang] Assignee
	Comment 4 • 12 years ago

Blocking bug 742790 for it has to be verified with WebSMS Android backend.

	Brad Lassey [:blassey] (use needinfo?)
	Comment 5 • 12 years ago

Comment on attachment 674174 [details] [diff] [review]
add serialVersionUID for serializable WebSMS exception classes

Review of attachment 674174 [details] [diff] [review]:
-----------------------------------------------------------------

::: mobile/android/base/GeckoSmsManager.java
@@ +883,5 @@
>      MessagesListManager.getInstance().clear();
>    }
> +
> +  class IdTooHighException extends Exception {
> +    private static final long serialVersionUID = 1L;

what is serialVersionUID for?

	Brad Lassey [:blassey] (use needinfo?)
	Comment 6 • 12 years ago

(In reply to Brad Lassey [:blassey] from comment #5)
> Comment on attachment 674174 [details] [diff] [review]
> add serialVersionUID for serializable WebSMS exception classes
> 
> Review of attachment 674174 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> ::: mobile/android/base/GeckoSmsManager.java
> @@ +883,5 @@
> >      MessagesListManager.getInstance().clear();
> >    }
> > +
> > +  class IdTooHighException extends Exception {
> > +    private static final long serialVersionUID = 1L;
> 
> what is serialVersionUID for?

nevermind, read up on it. If we're going to define these, let's at least make them unique.

	Vicamo Yang [:vicamo][:vyang] Assignee
	Comment 7 • 12 years ago

Randomize serialVersionUID. Addressed review comment #6.

	Vicamo Yang [:vicamo][:vyang] Assignee
	Comment 8 • 12 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/164d7dd54ee0

	Ryan VanderMeulen [:RyanVM]
	Comment 9 • 12 years ago

https://hg.mozilla.org/mozilla-central/rev/164d7dd54ee0