<html>

  <head>

    <meta content="text/html; charset=windows-1252"

      http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <br>

    <br>

    <div class="moz-cite-prefix">On 11/5/15 12:41 PM, Cantor, Scott

      wrote:<br>

    </div>

    <blockquote cite="mid:2C94DA80-30FF-4A65-A7CC-EB38A1798F72@osu.edu"

      type="cite">

      <pre wrap="">On 11/5/15, 12:37 PM, "dev on behalf of Brent Putman" <a class="moz-txt-link-rfc2396E" href="mailto:dev-bounces@shibboleth.netonbehalfofputmanb@georgetown.edu"><dev-bounces@shibboleth.net on behalf of putmanb@georgetown.edu></a> wrote:

</pre>

      <blockquote type="cite">

        <pre wrap="">No, it's awkward.  I was thinking of adding something to the clone methods to just do that.  I don't know that there's any good reason to ever return a cloned object that has a non-rooted DOM.  We could just change the no-boolean clone(XMLObject) method to do that by default.  We'd be changing behavior, but probably not in a bad way.  But if we think that's not allowed or advisable, I was thinking of adding a new CloneOutputOption enum to support representing all 3 options, and a new clone(XMLObject, CloneOutputOption).

Then deprecate the clone(XMLObject, boolean) one.

</pre>

      </blockquote>

      <pre wrap="">

Up to you, doesn't feel like a particularly important change to the caller, but I'm not as familiar with this code.

</pre>

    </blockquote>

    <br>

    So I did both.  Added the new enum to represent all 3 options. 

    Changed the cloneXMLObject(XMLObject) behavior to DropDOM rather

    than the old UnrootedDOM behavior.  Deprecated

    cloneXMLObject(XMLObject, boolean) and removed all usage of it that

    Eclipse can find.<br>

  </body>

</html>