<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style></head><body lang=EN-US link=blue vlink="#954F72"><div class=WordSection1><p class=MsoNormal>Once append* is deployed, let me know.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Thanks,</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>john</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Sent from <a href="https://go.microsoft.com/fwlink/?LinkId=550986">Mail</a> for Windows 10</p><p class=MsoNormal><o:p> </o:p></p><div style='mso-element:para-border-div;border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal style='border:none;padding:0in'><b>From: </b><a href="mailto:brutzman@nps.edu">Don Brutzman</a><br><b>Sent: </b>Tuesday, March 14, 2017 1:16 PM<br><b>To: </b><a href="mailto:yottzumm@gmail.com">yottzumm@gmail.com</a><br><b>Cc: </b><a href="mailto:roy.walmsley@ntlworld.com">Roy Walmsley</a>; <a href="mailto:x3d-public@web3d.org">X3D Graphics public mailing list</a><br><b>Subject: </b>Re: X3DJSAIL: Desired chaining methods. Modify return value orprovide new method.</p></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>This turns out to be trickier than first suspected. The conflicting issue is that the legacy SAI methods use a pattern that returns void (i.e. no value) instead of a copy of the object for subsequent method pipelining.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>So instead of modifying the existing methods, I added "append" methods that take a single child for appending to the subject array.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>http://www.web3d.org/specifications/java/javadoc/org/web3d/x3d/jsail/HAnim/HAnimHumanoidObject.html#method.summary</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>HAnimHumanoidObject appendJoints(HAnimJoint newValue)</p><p class=MsoNormal> Append single child joints node to array of existing nodes (if any).</p><p class=MsoNormal>HAnimHumanoidObject appendSegments(HAnimSegment newValue)</p><p class=MsoNormal> Append single child segments node to array of existing nodes (if any).</p><p class=MsoNormal>HAnimHumanoidObject appendSites(HAnimSite newValue)</p><p class=MsoNormal> Append single child sites node to array of existing nodes (if any).</p><p class=MsoNormal>HAnimHumanoidObject appendSkeleton(X3DNode newValue)</p><p class=MsoNormal> Append single child skeleton node to array of existing nodes (if any).</p><p class=MsoNormal>HAnimHumanoidObject appendSkin(X3DChildNode newValue)</p><p class=MsoNormal> Append single child skin node to array of existing nodes (if any).</p><p class=MsoNormal>HAnimHumanoidObject appendViewpoints(HAnimSite newValue)</p><p class=MsoNormal> Append single child viewpoints node to array of existing nodes (if any).</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Similar methods appear elsewhere for appending a single node to an MFNode array field.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>For clarity and to have a consistent pattern (which helps code generators like yours)</p><p class=MsoNormal>we should probably change utility methods like "addChild" to "appendChild" throughout.</p><p class=MsoNormal>What do you think?</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Specification considerations noted as follows:</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>X3dJavaSpecificationChangesAndIssues.txt 14 March 2017</p><p class=MsoNormal>http://www.web3d.org/x3d/stylesheets/java/X3dJavaSpecificationChangesAndIssues.txt</p><p class=MsoNormal>============================================================</p><p class=MsoNormal>3. SFNode/MFNode return types void</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Numerous interface return types are void, preventing a method from returning the</p><p class=MsoNormal>current object from returning a copy of itself to permit method pipelining.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Various utility methods have been added (typically named append* instead of add*)</p><p class=MsoNormal>in order to circumvent this restriction.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Need to consider changing SFNode/MFNode return signatures to support pipelining.</p><p class=MsoNormal>Of note is that a method returning a null value is essentially the same as a</p><p class=MsoNormal>void return, so no loss of functionality seems involved with this approach.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>============================================================</p><p class=MsoNormal>On 3/10/2017 1:14 PM, yottzumm@gmail.com wrote:</p><p class=MsoNormal>> Maybe all these methods of HAnimHumanoiObject should return an HAnimHumanoidObject for chaining:</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addJoints(HAnimJoint newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addJoints(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSegments(HAnimSegment newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSegments(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSites(HAnimSite newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSites(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSkeleton(X3DNode newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSkeleton(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSkin(X3DChildNode newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addSkin(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addViewpoints(HAnimSite newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> public void addViewpoints(X3DNode[] newValue)</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> John</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>all the best, Don</p><p class=MsoNormal>-- </p><p class=MsoNormal>Don Brutzman Naval Postgraduate School, Code USW/Br brutzman@nps.edu</p><p class=MsoNormal>Watkins 270, MOVES Institute, Monterey CA 93943-5000 USA +1.831.656.2149</p><p class=MsoNormal>X3D graphics, virtual worlds, navy robotics http://faculty.nps.edu/brutzman</p><p class=MsoNormal><o:p> </o:p></p></div></body></html>