<html><body><div style="color:#000; background-color:#fff; font-family:arial, helvetica, sans-serif;font-size:10pt"><div><span>Hi Pete, </span></div><div>It just seems to me this 2 line code snippet is written so often, it would benefit to be a one code liner.<br></div><div><br></div><div><br></div> <div style="font-size: 10pt; font-family: arial, helvetica, sans-serif; "> <div style="font-size: 12pt; font-family: 'times new roman', 'new york', times, serif; "> <font size="2" face="Arial"> <hr size="1"> <b><span style="font-weight:bold;">De :</span></b> Pete Muir <pmuir@redhat.com><br> <b><span style="font-weight: bold;">À :</span></b> Adrian Gonzalez <adr_gonzalez@yahoo.fr> <br><b><span style="font-weight: bold;">Cc :</span></b> "cdi-dev@lists.jboss.org" <cdi-dev@lists.jboss.org> <br> <b><span style="font-weight: bold;">Envoyé le :</span></b> Vendredi 16 Décembre 2011 12h05<br> <b><span style="font-weight:
bold;">Objet :</span></b> Re: [cdi-dev] Question about conversation.begin if transient<br> </font> <br>I don't personally see much wrong with the code snippet today, it seems pretty clear…<br><br>On 15 Dec 2011, at 16:24, Adrian Gonzalez wrote:<br><br>> Hello,<br>> <br>> This question has surely already been asked, but I'm looking over and over the following code :<br>> if (conversation.isTransient()) {<br>> conversation.begin();<br>> }<br>> <br>> Is it possible to create a single method in Conversation class to handle this ?<br>> Something like conversation.beginIfTransient() ? (of course with a better name :) ).<br>> <br>> WDYT ?<br>> <br>> <br>> _______________________________________________<br>> cdi-dev mailing list<br>> <a ymailto="mailto:cdi-dev@lists.jboss.org"
href="mailto:cdi-dev@lists.jboss.org">cdi-dev@lists.jboss.org</a><br>> <a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br><br><br><br> </div> </div> </div></body></html>