<div dir="ltr">Come on, Hardy, join me in supporting &lt;result&gt; which avoids whitespace and encompasses the full meaning of &quot;return value&quot;.  :)<br><br>Matt<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sun, Feb 10, 2013 at 4:42 AM, Hardy Ferentschik <span dir="ltr">&lt;<a href="mailto:hardy@hibernate.org" target="_blank">hardy@hibernate.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Personally I don&#39;t like the whitespace in &lt;return value&gt;. I would prefer &lt;return&gt;.<br>
However, if the majority agrees to  &lt;return value&gt; that&#39;s fine with me. I like the the use of &lt;&gt;.<br>
<span class="HOEnZb"><font color="#888888"><br>
--Hardy<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 9 Jan 2013, at 3:05 PM, Sebastian Thomschke &lt;<a href="mailto:sebastian.thomschke@web.de">sebastian.thomschke@web.de</a>&gt; wrote:<br>
<br>
&gt; +1 for &quot;&lt;return value&gt;&quot; too<br>
&gt;<br>
&gt; seb<br>
&gt;<br>
&gt; On 09.02.2013 12:03, Gunnar Morling wrote:<br>
&gt;&gt; +1 for &lt;return value&gt;.<br>
&gt;&gt;<br>
&gt;&gt; --Gunnar<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; 2013/2/8 Emmanuel Bernard &lt;<a href="mailto:emmanuel@hibernate.org">emmanuel@hibernate.org</a>&gt;<br>
&gt;&gt; I am not a big fan of the retval which reminds me of Gollum skeaping his<br>
&gt;&gt; name ;)<br>
&gt;&gt;<br>
&gt;&gt; - &lt;return value&gt;<br>
&gt;&gt; - &lt;return&gt;<br>
&gt;&gt; - (return value)<br>
&gt;&gt; - (return)<br>
&gt;&gt;<br>
&gt;&gt; I think I like &lt;return value&gt; the best.<br>
&gt;&gt;<br>
&gt;&gt; As Hardy said, the name is not critical as nodes are identified by their<br>
&gt;&gt; elementDescriptor.kind.<br>
&gt;&gt;<br>
&gt;&gt; Emmanuel<br>
&gt;&gt;<br>
&gt;&gt; On Fri 2013-02-08 11:19, Matt Benson wrote:<br>
&gt;&gt; &gt; I am somewhat attracted to Sebastian&#39;s suggestion of illegal identifier<br>
&gt;&gt; &gt; characters.  My suggestion would be &quot;&lt;result&gt;&quot;.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Matt<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; On Fri, Feb 8, 2013 at 9:21 AM, Hardy Ferentschik &lt;<a href="mailto:hardy@hibernate.org">hardy@hibernate.org</a>&gt;wrote:<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; &gt; I don&#39;t want to introduce a name for the return value to allow things like<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; if(node.getName.equals(&quot;retval&quot;)) {<br>
&gt;&gt; &gt; &gt;         ReturnValueDescriptor descriptor = (ReturnValueDescriptor)<br>
&gt;&gt; &gt; &gt; node.getElementDescriptor();<br>
&gt;&gt; &gt; &gt; }<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; The actual type of a node is still given by it ElementDescriptor. The name<br>
&gt;&gt; &gt; &gt; cannot be used for that. It is more for convenience<br>
&gt;&gt; &gt; &gt; and &quot;nice&quot; toString implementation. Yes it could be ambiguous, but I don&#39;t<br>
&gt;&gt; &gt; &gt; think it matters. Any code relying on the property path<br>
&gt;&gt; &gt; &gt; as string is potentially wrong anyways. A white space seems odd,<br>
&gt;&gt; &gt; &gt; especially in the light of toString.<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; In that light $retval might be a legal java identifier, but chances are<br>
&gt;&gt; &gt; &gt; slim someone uses it.<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; --Hardy<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; On 8 Jan 2013, at 4:07 PM, Sebastian Thomschke &lt;<a href="mailto:sebastian.thomschke@web.de">sebastian.thomschke@web.de</a>&gt;<br>
&gt;&gt; &gt; &gt; wrote:<br>
&gt;&gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; &gt; What if there is a property or method called &quot;returnValue&quot;? I think the<br>
&gt;&gt; &gt; &gt; constant string returned should contain a character that is not legal for<br>
&gt;&gt; &gt; &gt; java identifier names. E.g. a white space.<br>
&gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; &gt; seb<br>
&gt;&gt; &gt; &gt; &gt;<br>
&gt;&gt; &gt; &gt; &gt; On 08.02.2013 12:51, Gunnar Morling wrote:<br>
&gt;&gt; &gt; &gt; &gt;&gt; Experts,<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; another issue where we need some feedback is BVAL-368, which is about<br>
&gt;&gt; &gt; &gt; the name of path nodes representing return values.<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; As per the current draft, Node#getName() returns null in that case.<br>
&gt;&gt; &gt; &gt; Question is, whether we should return something more meaningful, and if so,<br>
&gt;&gt; &gt; &gt; which value.<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; The RI used to return &quot;$retval&quot; before we change this to match the<br>
&gt;&gt; &gt; &gt; spec. Another obvious option would be &quot;returnValue&quot;. Having a standardized<br>
&gt;&gt; &gt; &gt; node name for return value nodes would also help with better toString()<br>
&gt;&gt; &gt; &gt; implementations for j.v.Path (although that&#39;s not standardized).<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; Any thoughts?<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; Thanks,<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; --Gunnar<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt; [1] <a href="https://hibernate.onjira.com/browse/BVAL-368" target="_blank">https://hibernate.onjira.com/browse/BVAL-368</a><br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;&gt; &gt; &gt; &gt;&gt;<br>
&gt;<br>
</div></div><div class="HOEnZb"><div class="h5">&gt; _______________________________________________<br>
&gt; beanvalidation-dev mailing list<br>
&gt; <a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
&gt; <a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
<br>
<br>
_______________________________________________<br>
beanvalidation-dev mailing list<br>
<a href="mailto:beanvalidation-dev@lists.jboss.org">beanvalidation-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/beanvalidation-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/beanvalidation-dev</a><br>
</div></div></blockquote></div><br></div>