<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>It's definitely better to add warning suppression with a comment for why than add broken serial version id.</div><div><br></div><div>At least for the case of entities (so should do an updated hibernate tools release or extend the used default template). </div><div><br></div><div>Pete, got example what you put as comment in the past?</div><div><br>/max (sent from my phone)<div><br></div></div><div><br>On 29/05/2013, at 18.16, "Lincoln Baxter, III" <<a href="mailto:lincolnbaxter@gmail.com">lincolnbaxter@gmail.com</a>> wrote:<br><br></div><blockquote type="cite"><div><div dir="ltr">Is this something that you think we should be doing by default?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 29, 2013 at 12:14 PM, Pete Muir <span dir="ltr"><<a href="mailto:pmuir@redhat.com" target="_blank">pmuir@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Adding forge-dev<br>
<br>
The approach we took in the quickstarts was to add @SuppressWarnings("serial") in this case, as it adds a visual warning to the class that you haven't taken this step, without the annoying warning in the IDE. We also try to add a comment to the effect you need to do this before you go into production.<br>
<br>
On 23 May 2013, at 23:44, Max Andersen <<a href="mailto:manderse@redhat.com">manderse@redhat.com</a>> wrote:<br>
<br>
> On Thu, May 23, 2013 at 03:40:42PM -0400, Burr Sutter wrote:<br>
>> Lots of warnings when Forge generates a JPA entity<br>
>><br>
>> "The serializable class Actor does not declare a static final serialVersionUID field of type long"<br>
><br>
> we can disable the warninng in JBDS I believe.<br>
><br>
> It is eclipse being anal - they are correct in that you should add such field, but if you do then<br>
> you are declaring you are guaranteeing serializability between versions of your class which generated code<br>
> cannot guarantee nor does any user need this.<br>
><br>
> I've written about it in blogs and wikis - I can digg them out if you want the glory details :)<br>
><br>
> /max<br>
><br>
<br>
<br>
_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."
</div>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>forge-dev mailing list</span><br><span><a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a></span><br><span><a href="https://lists.jboss.org/mailman/listinfo/forge-dev">https://lists.jboss.org/mailman/listinfo/forge-dev</a></span></div></blockquote></body></html>