Santuario change

Cantor, Scott cantor.2 at osu.edu
Thu Nov 5 10:23:43 EST 2015


On 11/5/15, 2:55 AM, "dev on behalf of Brent Putman" <dev-bounces at shibboleth.net on behalf of putmanb at georgetown.edu> wrote:



>I was even thinking perhaps we needed new options on the clone methods to just do that by default, since seems a common case.  But for the record here's what I did:
>
>AuthnStatement statement = XMLObjectSupport.cloneXMLObject(sourceStatement);
>statement.releaseDOM();
>statement.releaseChildrenDOM(true);
>return statement;

We have 3-4 places in the code doing this, so I'd say it's a little awkward to have that sequence hanging out all over. A brief comment isn't very clarifying.

-- Scott



More information about the dev mailing list