diff --git a/Code/Mantid/Framework/Geometry/inc/MantidGeometry/Crystal/PointGroupFactory.h b/Code/Mantid/Framework/Geometry/inc/MantidGeometry/Crystal/PointGroupFactory.h index 4da9deebc59f..5ec5502a26df 100644 --- a/Code/Mantid/Framework/Geometry/inc/MantidGeometry/Crystal/PointGroupFactory.h +++ b/Code/Mantid/Framework/Geometry/inc/MantidGeometry/Crystal/PointGroupFactory.h @@ -46,14 +46,14 @@ namespace Geometry class MANTID_GEOMETRY_DLL PointGroupFactoryImpl : public Kernel::DynamicFactory { public: - PointGroup_sptr createPointgroup(const std::string &hmSymbol) const; + PointGroup_sptr createPointGroup(const std::string &hmSymbol) const; std::vector getAllPointGroupSymbols() const; - std::vector getAllPointGroupSymbols(const PointGroup::CrystalSystem &crystalSystem) const; + std::vector getPointGroupSymbols(const PointGroup::CrystalSystem &crystalSystem) const; /// Subscribes a point group into the factory template - void subscribePointgroup() + void subscribePointGroup() { Kernel::Instantiator *instantiator = new Kernel::Instantiator; PointGroup_sptr temporaryPointgroup = instantiator->createInstance(); @@ -65,7 +65,7 @@ namespace Geometry } /// Unsubscribes a point group from the factory - void unsubscribePointgroup(const std::string &hmSymbol) + void unsubscribePointGroup(const std::string &hmSymbol) { unsubscribe(hmSymbol); removeFromCrystalSystemMap(hmSymbol); @@ -95,7 +95,7 @@ typedef Mantid::Kernel::SingletonHolder PointGroupFactory #define DECLARE_POINTGROUP(classname) \ namespace { \ Mantid::Kernel::RegistrationHelper register_pointgroup_##classname( \ - ((Mantid::Geometry::PointGroupFactory::Instance().subscribePointgroup()) \ + ((Mantid::Geometry::PointGroupFactory::Instance().subscribePointGroup()) \ , 0)); \ } diff --git a/Code/Mantid/Framework/Geometry/src/Crystal/PointGroupFactory.cpp b/Code/Mantid/Framework/Geometry/src/Crystal/PointGroupFactory.cpp index d17a31d38230..7faf3c7ad43c 100644 --- a/Code/Mantid/Framework/Geometry/src/Crystal/PointGroupFactory.cpp +++ b/Code/Mantid/Framework/Geometry/src/Crystal/PointGroupFactory.cpp @@ -8,7 +8,7 @@ namespace Geometry { /// Creates a PointGroup object from its Hermann-Mauguin symbol. -PointGroup_sptr PointGroupFactoryImpl::createPointgroup(const std::string &hmSymbol) const +PointGroup_sptr PointGroupFactoryImpl::createPointGroup(const std::string &hmSymbol) const { return create(hmSymbol); } @@ -26,7 +26,7 @@ std::vector PointGroupFactoryImpl::getAllPointGroupSymbols() const } /// Returns the Hermann-Mauguin symbols of all point groups that belong to a certain crystal system. -std::vector PointGroupFactoryImpl::getAllPointGroupSymbols(const PointGroup::CrystalSystem &crystalSystem) const +std::vector PointGroupFactoryImpl::getPointGroupSymbols(const PointGroup::CrystalSystem &crystalSystem) const { std::vector pointGroups; diff --git a/Code/Mantid/Framework/Geometry/test/PointGroupFactoryTest.h b/Code/Mantid/Framework/Geometry/test/PointGroupFactoryTest.h index ec4414fb80ca..860e3abf2da4 100644 --- a/Code/Mantid/Framework/Geometry/test/PointGroupFactoryTest.h +++ b/Code/Mantid/Framework/Geometry/test/PointGroupFactoryTest.h @@ -91,18 +91,18 @@ class PointGroupFactoryTest : public CxxTest::TestSuite ~PointGroupFactoryTest() { // Unsubscribing the fake point groups - PointGroupFactory::Instance().unsubscribePointgroup("cubicA"); - PointGroupFactory::Instance().unsubscribePointgroup("cubicB"); - PointGroupFactory::Instance().unsubscribePointgroup("triclinic"); + PointGroupFactory::Instance().unsubscribePointGroup("cubicA"); + PointGroupFactory::Instance().unsubscribePointGroup("cubicB"); + PointGroupFactory::Instance().unsubscribePointGroup("triclinic"); } void testCreatePointGroup() { - TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointgroup("cubicA")); - TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointgroup("cubicB")); - TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointgroup("triclinic")); + TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointGroup("cubicA")); + TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointGroup("cubicB")); + TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointGroup("triclinic")); - TS_ASSERT_THROWS(PointGroupFactory::Instance().createPointgroup("cubicC"), Mantid::Kernel::Exception::NotFoundError); + TS_ASSERT_THROWS(PointGroupFactory::Instance().createPointGroup("cubicC"), Mantid::Kernel::Exception::NotFoundError); } void testGetAllPointGroupSymbols() @@ -116,23 +116,23 @@ class PointGroupFactoryTest : public CxxTest::TestSuite void testGetAllPointGroupSymbolsCrystalSystems() { - std::vector cubic = PointGroupFactory::Instance().getAllPointGroupSymbols(PointGroup::Cubic); + std::vector cubic = PointGroupFactory::Instance().getPointGroupSymbols(PointGroup::Cubic); TS_ASSERT_DIFFERS(findString(cubic, "cubicA"), cubic.end()); TS_ASSERT_DIFFERS(findString(cubic, "cubicB"), cubic.end()); - std::vector triclinic = PointGroupFactory::Instance().getAllPointGroupSymbols(PointGroup::Triclinic); + std::vector triclinic = PointGroupFactory::Instance().getPointGroupSymbols(PointGroup::Triclinic); TS_ASSERT_DIFFERS(findString(triclinic, "triclinic"), triclinic.end()); } void testUnsubscribePointGroup() { - TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointgroup("cubicA")); + TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointGroup("cubicA")); - PointGroupFactory::Instance().unsubscribePointgroup("cubicA"); + PointGroupFactory::Instance().unsubscribePointGroup("cubicA"); TS_ASSERT_THROWS(PointGroupFactory::Instance().create("cubicA"), Mantid::Kernel::Exception::NotFoundError); - PointGroupFactory::Instance().subscribePointgroup(); - TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointgroup("cubicA")); + PointGroupFactory::Instance().subscribePointGroup(); + TS_ASSERT_THROWS_NOTHING(PointGroupFactory::Instance().createPointGroup("cubicA")); } private: