core: forbid conversion real->int in some cases in FileStorage

sovrasov · sovrasov · commit 5b833db558a6 · 2017-07-12T15:50:57.000+03:00
diff --git a/modules/core/src/persistence.cpp b/modules/core/src/persistence.cpp
@@ -7374,22 +7374,21 @@ size_t FileNode::size() const
 void read(const FileNode& node, int& value, int default_value)
 {
     value = !node.node ? default_value :
-    CV_NODE_IS_INT(node.node->tag) ? node.node->data.i :
-    CV_NODE_IS_REAL(node.node->tag) ? cvRound(node.node->data.f) : 0x7fffffff;
+    CV_NODE_IS_INT(node.node->tag) ? node.node->data.i : std::numeric_limits<int>::max();
 }
 
 void read(const FileNode& node, float& value, float default_value)
 {
     value = !node.node ? default_value :
         CV_NODE_IS_INT(node.node->tag) ? (float)node.node->data.i :
-        CV_NODE_IS_REAL(node.node->tag) ? (float)node.node->data.f : 1e30f;
+        CV_NODE_IS_REAL(node.node->tag) ? saturate_cast<float>(node.node->data.f) : std::numeric_limits<float>::max();
 }
 
 void read(const FileNode& node, double& value, double default_value)
 {
     value = !node.node ? default_value :
         CV_NODE_IS_INT(node.node->tag) ? (double)node.node->data.i :
-        CV_NODE_IS_REAL(node.node->tag) ? node.node->data.f : 1e300;
+        CV_NODE_IS_REAL(node.node->tag) ? node.node->data.f : std::numeric_limits<double>::max();
 }
 
 void read(const FileNode& node, String& value, const String& default_value)

Original file line number	Diff line number	Diff line change
`@@ -7374,22 +7374,21 @@ size_t FileNode::size() const`
`7374`	`7374`	`void read(const FileNode& node, int& value, int default_value)`
`7375`	`7375`	`{`
`7376`	`7376`	`value = !node.node ? default_value :`
`7377`		`- CV_NODE_IS_INT(node.node->tag) ? node.node->data.i :`
`7378`		`- CV_NODE_IS_REAL(node.node->tag) ? cvRound(node.node->data.f) : 0x7fffffff;`
	`7377`	`+ CV_NODE_IS_INT(node.node->tag) ? node.node->data.i : std::numeric_limits<int>::max();`
`7379`	`7378`	`}`
`7380`	`7379`
`7381`	`7380`	`void read(const FileNode& node, float& value, float default_value)`
`7382`	`7381`	`{`
`7383`	`7382`	`value = !node.node ? default_value :`
`7384`	`7383`	`CV_NODE_IS_INT(node.node->tag) ? (float)node.node->data.i :`
`7385`		`- CV_NODE_IS_REAL(node.node->tag) ? (float)node.node->data.f : 1e30f;`
	`7384`	`+ CV_NODE_IS_REAL(node.node->tag) ? saturate_cast<float>(node.node->data.f) : std::numeric_limits<float>::max();`
`7386`	`7385`	`}`
`7387`	`7386`
`7388`	`7387`	`void read(const FileNode& node, double& value, double default_value)`
`7389`	`7388`	`{`
`7390`	`7389`	`value = !node.node ? default_value :`
`7391`	`7390`	`CV_NODE_IS_INT(node.node->tag) ? (double)node.node->data.i :`
`7392`		`- CV_NODE_IS_REAL(node.node->tag) ? node.node->data.f : 1e300;`
	`7391`	`+ CV_NODE_IS_REAL(node.node->tag) ? node.node->data.f : std::numeric_limits<double>::max();`
`7393`	`7392`	`}`
`7394`	`7393`
`7395`	`7394`	`void read(const FileNode& node, String& value, const String& default_value)`