Use more often uint64_t instead of uint32_t to better conform to XML

SWIG port RepresentationSetRepresentation::pushBack(RESQML2_NS::AbstractRepresentation* rep)

Author: philippeVerney

src/common/DataObjectRepository.h

@@ -558,15 +558,11 @@ namespace COMMON_NS
 		*/
 #define GETTER_DATAOBJECTS(returnedDataType, dataObjectName)\
 		DLL_IMPORT_OR_EXPORT std::vector<returnedDataType*> get##dataObjectName##Set() const;\
-		DLL_IMPORT_OR_EXPORT unsigned int get##dataObjectName##Count() const {\
-			const size_t result = get##dataObjectName##Set().size();\
-			if (result > (std::numeric_limits<unsigned int>::max)()) { throw std::range_error("The count is superior to unsigned int max"); }\
-			return static_cast<unsigned int>(result);\
+		DLL_IMPORT_OR_EXPORT uint64_t get##dataObjectName##Count() const {\
+			return get##dataObjectName##Set().size();\
 		}\
-		DLL_IMPORT_OR_EXPORT returnedDataType* get##dataObjectName(unsigned int index) const {\
-			std::vector<returnedDataType*> all = get##dataObjectName##Set();\
-			if (index >= all.size()) { throw std::out_of_range("The index is out of range"); }\
-			return all[index];\
+		DLL_IMPORT_OR_EXPORT returnedDataType* get##dataObjectName(uint64_t index) const {\
+			return get##dataObjectName##Set().at(index);\
 		}
 
 		GETTER_DATAOBJECTS(EML2_NS::AbstractHdfProxy, HdfProxy)

src/eml2_3/Activity.cpp

@@ -53,11 +53,9 @@ void Activity::pushBackParameter(const std::string& title, double value, gsoap_r
 		if (maxOccurs > -1 && static_cast<uint64_t>(maxOccurs) <= getParameterCount(title)) {
 			throw invalid_argument("The max number of occurrences has already been reached for parameter " + title);
 		}
-		if (dynamic_cast<EML2_NS::ActivityTemplate*>(activityTemplate) != nullptr) {
-			vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = static_cast<EML2_NS::ActivityTemplate*>(activityTemplate)->getParameterAllowedKinds(title);
-			if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::floatingPoint) == allowedKinds.end())
-				throw invalid_argument("The parameter template " + title + " does not allow a double datatype.");
-		}
+		vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = activityTemplate->getParameterAllowedKinds(title);
+		if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::floatingPoint) == allowedKinds.end())
+			throw invalid_argument("The parameter template " + title + " does not allow a double datatype.");
 	}
 
 	_eml23__Activity* activity = static_cast<_eml23__Activity*>(gsoapProxy2_3);
@@ -80,11 +78,9 @@ void Activity::pushBackParameter(const std::string& title, const std::string & v
 		if (maxOccurs > -1 && static_cast<uint64_t>(maxOccurs) <= getParameterCount(title)) {
 			throw invalid_argument("The max number of occurrences has already been reached for parameter " + title);
 		}
-		if (dynamic_cast<EML2_NS::ActivityTemplate*>(activityTemplate) != nullptr) {
-			vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = static_cast<EML2_NS::ActivityTemplate*>(activityTemplate)->getParameterAllowedKinds(title);
-			if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::string) == allowedKinds.end())
-				throw invalid_argument("The parameter template " + title + " does not allow a string datatype.");
-		}
+		vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = activityTemplate->getParameterAllowedKinds(title);
+		if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::string) == allowedKinds.end())
+			throw invalid_argument("The parameter template " + title + " does not allow a string datatype.");
 	}
 
 	_eml23__Activity* activity = static_cast<_eml23__Activity*>(gsoapProxy2_3);
@@ -106,11 +102,9 @@ void Activity::pushBackParameter(const std::string& title, int64_t value)
 		if (maxOccurs > -1 && static_cast<uint64_t>(maxOccurs) <= getParameterCount(title)) {
 			throw invalid_argument("The max number of occurrences has already been reached for parameter " + title);
 		}
-		if (dynamic_cast<EML2_NS::ActivityTemplate*>(activityTemplate) != nullptr) {
-			vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = static_cast<EML2_NS::ActivityTemplate*>(activityTemplate)->getParameterAllowedKinds(title);
-			if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::integer) == allowedKinds.end())
-				throw invalid_argument("The parameter template " + title + " does not allow an integer datatype.");
-		}
+		vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = activityTemplate->getParameterAllowedKinds(title);
+		if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::integer) == allowedKinds.end())
+			throw invalid_argument("The parameter template " + title + " does not allow an integer datatype.");
 	}
 
 	_eml23__Activity* activity = static_cast<_eml23__Activity*>(gsoapProxy2_3);
@@ -134,11 +128,9 @@ void Activity::pushBackParameter(const std::string& title, AbstractObject* resqm
 		int64_t maxOccurs = activityTemplate->getParameterMaxOccurences(title);
 		if (maxOccurs > -1 && static_cast<uint64_t>(maxOccurs) <= getParameterCount(title))
 			throw invalid_argument("The max number of occurrences has already been reached for parameter " + title);
-		if (dynamic_cast<EML2_NS::ActivityTemplate*>(activityTemplate) != nullptr) {
-			vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = static_cast<EML2_NS::ActivityTemplate*>(activityTemplate)->getParameterAllowedKinds(title);
-			if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::dataObject) == allowedKinds.end())
-				throw invalid_argument("The parameter template " + title + " does not allow a data object datatype.");
-		}
+		vector<gsoap_resqml2_0_1::resqml20__ParameterKind> allowedKinds = activityTemplate->getParameterAllowedKinds(title);
+		if (allowedKinds.size() > 0 && find(allowedKinds.begin(), allowedKinds.end(), gsoap_resqml2_0_1::resqml20__ParameterKind::dataObject) == allowedKinds.end())
+			throw invalid_argument("The parameter template " + title + " does not allow a data object datatype.");
 	}
 
 	getRepository()->addRelationship(this, resqmlObject);
@@ -260,7 +252,7 @@ vector<gsoap_resqml2_0_1::resqml20__ResqmlUom> Activity::getFloatingPointQuantit
 		throw range_error("The parameter " + paramTitle + " is not in the parameter range.");
 
 	vector<gsoap_resqml2_0_1::resqml20__ResqmlUom> result;
-	for (unsigned int i = 0; i < param.size(); ++i)
+	for (size_t i = 0; i < param.size(); ++i)
 	{
 		if (param[i]->soap_type() != SOAP_TYPE_gsoap_eml2_3_eml23__DoubleQuantityParameter)
 			throw invalid_argument("The parameter " + paramTitle + " contains some non double values.");
@@ -373,8 +365,9 @@ vector<string> Activity::getStringParameterValue(const std::string & paramTitle)
 {
 	vector<gsoap_eml2_3::eml23__AbstractActivityParameter*> param = getParameterFromTitle(paramTitle);
 
-	if (param.size() < 1)
-		invalid_argument("There exists no " + paramTitle + " parameter in this activity.");
+	if (param.size() < 1) {
+		throw invalid_argument("There exists no " + paramTitle + " parameter in this activity.");
+	}
 
 	vector<string> result;
 	for (size_t i = 0; i < param.size(); ++i)
@@ -412,7 +405,7 @@ bool Activity::isAResqmlObjectParameter(const std::string & paramTitle) const
 		throw invalid_argument("There exists no " + paramTitle + " parameter in this activity.");
 	}
 
-	for (unsigned int i = 0; i < param.size(); ++i) {
+	for (size_t i = 0; i < param.size(); ++i) {
 		if (param[i]->soap_type() != SOAP_TYPE_gsoap_eml2_3_eml23__DataObjectParameter)
 			return false;
 	}
@@ -422,17 +415,16 @@ bool Activity::isAResqmlObjectParameter(const std::string & paramTitle) const
 
 bool Activity::isAResqmlObjectParameter(uint64_t index) const
 {
-	_eml23__Activity* activity = static_cast<_eml23__Activity*>(gsoapProxy2_3);
-
-	return activity->Parameter.at(index)->soap_type() == SOAP_TYPE_gsoap_eml2_3_eml23__DataObjectParameter;
+	return static_cast<_eml23__Activity*>(gsoapProxy2_3)->Parameter.at(index)->soap_type() == SOAP_TYPE_gsoap_eml2_3_eml23__DataObjectParameter;
 }
 
 vector<COMMON_NS::AbstractObject*> Activity::getResqmlObjectParameterValue(const std::string & paramTitle) const
 {
 	vector<gsoap_eml2_3::eml23__AbstractActivityParameter*> param = getParameterFromTitle(paramTitle);
 
-	if (param.size() < 1)
-		invalid_argument("There exists no " + paramTitle + " parameter in this activity.");
+	if (param.size() < 1) {
+		throw invalid_argument("There exists no " + paramTitle + " parameter in this activity.");
+	}
 
 	vector<COMMON_NS::AbstractObject*> result;
 	for (size_t i = 0; i < param.size(); ++i)

src/eml2_3/ActivityTemplate.cpp

@@ -37,7 +37,7 @@ void ActivityTemplate::pushBackParameter(const std::string & title,
 	uint64_t minOccurs, int64_t maxOccurs)
 {
 	// Preconditions
-	if (maxOccurs >= 0 && minOccurs > static_cast<unsigned int>(maxOccurs)) {
+	if (maxOccurs >= 0 && minOccurs > static_cast<uint64_t>(maxOccurs)) {
 		throw invalid_argument("Maximum occurences of a parameter template must be at least equal to the minimum occurences of this same parameter template.");
 	}
 

src/epc/FileCoreProperties.cpp

@@ -91,7 +91,7 @@ std::string FileCoreProperties::toString() const
 	oss << "xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">" << endl;
 
 	// content
-	for (unsigned int i = 0; i < CoreProperty::undefinedCoreProperty; ++i) {
+	for (size_t i = 0; i < CoreProperty::undefinedCoreProperty; ++i) {
 		if (!properties[i].isEmpty()) {
 			oss << "\t" << properties[i].toString() << endl;
 		}
@@ -185,7 +185,7 @@ void FileCoreProperties::setVersion(const std::string & value)
 
 void FileCoreProperties::readFromString(const string & textInput)
 {
-	for (unsigned int i = 0; i < CoreProperty::undefinedCoreProperty; ++i) {
+	for (size_t i = 0; i < CoreProperty::undefinedCoreProperty; ++i) {
 		properties[i].setTypeProperty(CoreProperty::undefinedCoreProperty);
 	}
 

src/resqml2/AbstractSeismicLineFeature.cpp

@@ -77,7 +77,7 @@ std::vector<std::string> AbstractSeismicLineFeature::getTraceLabels() const
 	if (gsoapProxy2_0_1 != nullptr) {
 		gsoap_resqml2_0_1::_resqml20__SeismicLineFeature* seismicLine = static_cast<gsoap_resqml2_0_1::_resqml20__SeismicLineFeature*>(gsoapProxy2_0_1);
 		const uint64_t traceCount = seismicLine->TraceCount;
-		if (traceCount > (std::numeric_limits<int64_t>::max)()) {
+		if (traceCount > static_cast<uint64_t>((std::numeric_limits<int64_t>::max)())) {
 			throw overflow_error("The trace count cannot be superior to int64 max");
 		}
 		for (int64_t incr = 0; incr < static_cast<int64_t>(traceCount); ++incr) {

src/resqml2/AbstractStratigraphicOrganizationInterpretation.cpp

@@ -27,36 +27,14 @@ under the License.
 using namespace RESQML2_NS;
 using namespace std;
 
-std::vector<AbstractGridRepresentation *> AbstractStratigraphicOrganizationInterpretation::getGridRepresentations() const
+std::vector<AbstractGridRepresentation*> AbstractStratigraphicOrganizationInterpretation::getGridRepresentationSet() const
 {
 	return getRepository()->getSourceObjects<AbstractGridRepresentation>(this);
 }
 
-unsigned int AbstractStratigraphicOrganizationInterpretation::getGridRepresentationCount() const
-{
-	const size_t count = getGridRepresentations().size();
-
-	if (count > (std::numeric_limits<unsigned int>::max)()) {
-		throw range_error("Too much associated grids");
-	}
-
-	return static_cast<unsigned int>(count);
-}
-
-AbstractGridRepresentation * AbstractStratigraphicOrganizationInterpretation::getGridRepresentation(unsigned int index) const
-{
-	const std::vector<AbstractGridRepresentation *>& gridRepresentationSet = getGridRepresentations();
-
-	if (index >= gridRepresentationSet.size()) {
-		throw out_of_range("The index of the grid representation to get is out of range.");
-	}
-
-	return gridRepresentationSet[index];
-}
-
 bool AbstractStratigraphicOrganizationInterpretation::isAssociatedToGridRepresentation(AbstractGridRepresentation* gridRep) const
 {
-	const std::vector<AbstractGridRepresentation *>& gridRepresentationSet = getGridRepresentations();
+	const std::vector<AbstractGridRepresentation*> gridRepresentationSet = getGridRepresentationSet();
 
 	return find(gridRepresentationSet.begin(), gridRepresentationSet.end(), gridRep) != gridRepresentationSet.end();
 }

src/resqml2/AbstractStratigraphicOrganizationInterpretation.h

@@ -42,37 +42,7 @@ namespace RESQML2_NS
 
 		bool isStratigraphic() const final { return true; }
 
-		/**
-		 * Gets the grid representations associated to this stratigraphic organization interpretation.
-		 *
-		 * @returns	A vector of pointers to all the grid representations associated to this stratigraphic
-		 * 			organization interpretation.
-		 */
-		DLL_IMPORT_OR_EXPORT std::vector<AbstractGridRepresentation*> getGridRepresentations() const;
-
-		/**
-		 * Gets the count of grid representations associated to this stratigraphic organization
-		 * interpretation.
-		 *
-		 * @exception	std::range_error	If the count of associated grid representations is strictly
-		 * 									greater than <tt>unsigned int</tt> max.
-		 *
-		 * @returns	The count of associated grid representations.
-		 */
-		DLL_IMPORT_OR_EXPORT unsigned int getGridRepresentationCount() const;
-
-		/**
-		 * Gets a grid representation associated to this stratigraphic organization interpretation by
-		 * means of its index.
-		 *
-		 * @exception	std::out_of_range	If @p index is out of range.
-		 *
-		 * @param 	index	The index of the grid representation to get in the array of grid
-		 * 					representations of this stratigraphic organization interpretations.
-		 *
-		 * @returns	The associated grid representation at index @p index.
-		 */
-		DLL_IMPORT_OR_EXPORT AbstractGridRepresentation * getGridRepresentation(unsigned int index) const;
+		GETTER_DATAOBJECTS(AbstractGridRepresentation, GridRepresentation)
 
 		/**
 		 * Checks whether a given grid representation is associated to this stratigraphic organization

src/resqml2/AbstractSurfaceRepresentation.cpp

@@ -27,15 +27,15 @@ void AbstractSurfaceRepresentation::loadTargetRelationships()
 {
 	AbstractRepresentation::loadTargetRelationships();
 
-	for (unsigned int i = 0; i < getBoundariesCount(); ++i) {
+	for (uint64_t i = 0; i < getBoundariesCount(); ++i) {
 		COMMON_NS::DataObjectReference dor = getOuterRingDor(i);
 		if (!dor.isEmpty()) {
 			convertDorIntoRel(dor);
 		}
 	}
 }
 
-unsigned int AbstractSurfaceRepresentation::getBoundariesCount() const
+uint64_t AbstractSurfaceRepresentation::getBoundariesCount() const
 {
 	size_t result;
 
@@ -56,7 +56,7 @@ unsigned int AbstractSurfaceRepresentation::getBoundariesCount() const
 	return static_cast<unsigned int>(result);
 }
 
-COMMON_NS::DataObjectReference AbstractSurfaceRepresentation::getOuterRingDor(unsigned int index) const
+COMMON_NS::DataObjectReference AbstractSurfaceRepresentation::getOuterRingDor(uint64_t index) const
 {
 	if (index >= getBoundariesCount()) {
 		throw std::out_of_range("The index is out of range");

src/resqml2/AbstractSurfaceRepresentation.h

@@ -43,7 +43,7 @@ namespace RESQML2_NS
 		 *
 		 * @returns	The boundaries count.
 		 */
-		unsigned int getBoundariesCount() const;
+		uint64_t getBoundariesCount() const;
 
 		/**
 		 * Gets the DOR of an outer ring at a particular index.
@@ -55,7 +55,7 @@ namespace RESQML2_NS
 		 *
 		 * @returns	The DOR of the outer ring at position @p index.
 		 */
-		COMMON_NS::DataObjectReference getOuterRingDor(unsigned int index) const;
+		COMMON_NS::DataObjectReference getOuterRingDor(uint64_t index) const;
 
 		/**
 		 * Pushes back an outer ring at the first available boundary index of this surface

src/resqml2/AbstractValuesProperty.h

@@ -1856,7 +1856,7 @@ namespace RESQML2_NS
 				if (gsoapProxy2_0_1->soap_type() == SOAP_TYPE_gsoap_resqml2_0_1_resqml20__obj_USCOREContinuousProperty) {
 					gsoap_resqml2_0_1::_resqml20__ContinuousProperty* prop = static_cast<gsoap_resqml2_0_1::_resqml20__ContinuousProperty*>(gsoapProxy2_0_1);
 					for (size_t i = 0; i < stats.getMinimumSize(); ++i) {
-						const double minStat = stats.getMinimum(i);
+						const double minStat = static_cast<double>(stats.getMinimum(i));
 						if (prop->MinimumValue.size() > i) {
 							if (minStat == minStat && prop->MinimumValue[i] > minStat) {
 								prop->MinimumValue[i] = minStat;
@@ -1867,7 +1867,7 @@ namespace RESQML2_NS
 						}
 					}
 					for (size_t i = 0; i < stats.getMaximumSize(); ++i) {
-						const double maxStat = stats.getMaximum(i);
+						const double maxStat = static_cast<double>(stats.getMaximum(i));
 						if (prop->MaximumValue.size() > i) {
 							if (maxStat == maxStat && prop->MaximumValue[i] < maxStat) {
 								prop->MaximumValue[i] = maxStat;
@@ -1881,7 +1881,7 @@ namespace RESQML2_NS
 				else if (gsoapProxy2_0_1->soap_type() == SOAP_TYPE_gsoap_resqml2_0_1_resqml20__obj_USCOREDiscreteProperty) {
 					gsoap_resqml2_0_1::_resqml20__DiscreteProperty* prop = static_cast<gsoap_resqml2_0_1::_resqml20__DiscreteProperty*>(gsoapProxy2_0_1);
 					for (size_t i = 0; i < stats.getMinimumSize(); ++i) {
-						const auto minStat = stats.getMinimum(i);
+						const auto minStat = static_cast<int64_t>(stats.getMinimum(i));
 						if (prop->MinimumValue.size() > i) {
 							if (minStat == minStat && prop->MinimumValue[i] > minStat) {
 								prop->MinimumValue[i] = minStat;
@@ -1892,7 +1892,7 @@ namespace RESQML2_NS
 						}
 					}
 					for (size_t i = 0; i < stats.getMaximumSize(); ++i) {
-						const auto maxStat = stats.getMaximum(i);
+						const auto maxStat = static_cast<int64_t>(stats.getMaximum(i));
 						if (prop->MaximumValue.size() > i) {
 							if (maxStat == maxStat && prop->MaximumValue[i] < maxStat) {
 								prop->MaximumValue[i] = maxStat;