Hi Vu,
ack, review only. Minor comment below./Thanks HansN
On 07/30/2018 10:46 AM, Vu Minh Nguyen wrote:
When explicitly having <default-value> tag, but no value is given:
<default-value></default-value>, set NULL to attrDefaultValue.
---
src/imm/immloadd/imm_loader.cc | 3 ++-
src/imm/tools/imm_import.cc | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/imm/immloadd/imm_loader.cc b/src/imm/immloadd/imm_loader.cc
index de5a575e9..ad9785e92 100644
--- a/src/imm/immloadd/imm_loader.cc
+++ b/src/imm/immloadd/imm_loader.cc
@@ -1909,7 +1909,8 @@ void addClassAttributeDefinition(
attrDefinition.attrFlags = attrFlags;
/* Set the default value */
- if (attrDefaultValueBuffer) {
+ if (attrDefaultValueBuffer &&
[HansN] use static_cast<const char*>(attrDefaultValueBuffer) instead or
(strlen(attrDefaultValueBuffer) > 0)) { (instead of the reinterpret_cast, not
needed though)
+ (strlen(reinterpret_cast<char*>(attrDefaultValueBuffer)) > 0)) {
charsToValueHelper(&attrDefinition.attrDefaultValue, attrValueType,
(const char *)attrDefaultValueBuffer);
} else {
diff --git a/src/imm/tools/imm_import.cc b/src/imm/tools/imm_import.cc
index e2bdcba5c..8145ec572 100644
--- a/src/imm/tools/imm_import.cc
+++ b/src/imm/tools/imm_import.cc
@@ -2444,7 +2444,8 @@ static void addClassAttributeDefinition(ParserState
*state) {
}
/* Set the default value */
- if (state->attrDefaultValueSet) {
+ if (state->attrDefaultValueSet &&
[HansN] use static_cast<const char*>(attrDefaultValueBuffer) instead or
(strlen(attrDefaultValueBuffer) > 0)) { (instead of the reinterpret_cast, not
needed though)
+ (strlen(reinterpret_cast<char*>(state->attrDefaultValueBuffer)) > 0)) {
if (charsToValueHelper(&attrDefinition.attrDefaultValue,
state->attrValueType,
state->attrDefaultValueBuffer,
state->strictParse)) {
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel