Fix unreleased use of POCO::XML document and nodelists in Algorithms code

[Karl Palmen]

In the code of framework/Algorithms I found three cases of POCO::XML document or nodelist objects being created, but not released. This could cause memory leakage. Fix by using AutPtr for the object.

The three cases I founnd were CreateDummyCalFile, ReadGroupsFromFile, DetectorEfficiencyCorTest.

[Karl Palmen]

AutoPtr is preferred to release().

[Nick Draper]

bulk move to assigned at the into of the triage step

[Federico Montesino Pouzols]

use AutoPtr to handle mem in poco xml objects, cut headers, re #8484

Changeset: c6a90e9e81a407850f940b2a339e5e4790bd92ed

[Federico M Pouzols]

This is a maintenance / leak cleaning ticket. The modified cpp files/classes are:

• CreateDummyCalFile.cpp
• ReadGroupsFromFile.cpp

I could not find any issue in the current DetectorEfficiencyCorTest or anywhere else.

To test:

• review code changes
• check that tests pass, especially all the algorithm tests

[Harry Jeffery]

Merge remote-tracking branch 'origin/bugfix/8484_fix_unreleased_libpoco_xml_objects_in_algorithms'

Full changeset: ​4427370d51ef86a00b8ef90bf00acf4ffd4e9db7

[Stuart Campbell]

This ticket has been transferred to github issue ​9328