Menu ▾ ▴

#107 Procedure with whitespaces in their names may not return values

v1.0.x
fixed
Erik Hänel
critical (96)
v1.0.9
Bug Fix
2016-10-08
2016-09-11
Erik Hänel
No

Entering something like

$'C:\CPP\NumeRe\procedures\test'()

will result in an expression error, if the named procedure does not return void.

Analysis:
The error comes from the non-replacements of the path separators in the final numerical variable name, because then the parser doesn't recognize it as a variable. In the variable name all path specific chars (:, / and \\) must be replaced with ~.

Implementation:
The fix was implemented as proposed by the analysis. Implementation tests were passed successfully.

Documentation:
Not needed - bug fix.

Tests:
Final tests passed successfully. Bug fix accepted.

Discussion

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • status: open --> accepted
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -3,3 +3,11 @@
 $'C:\CPP\NumeRe\procedures\test'()
 ~~~
 will result in an expression error, if the named procedure does **not** return `void`.
+
+**Analysis:**
+
+**Implementation:**
+
+**Documentation:**
+
+**Tests:**
    • status: accepted --> analyzing
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -5,6 +5,7 @@
 will result in an expression error, if the named procedure does **not** return `void`.

 **Analysis:**
+The error comes from the non-replacements of the path separators in the final numerical variable name, because then the parser doesn't recognize it as a variable. In the variable name all path specific chars (`:`, `/` and `\\`) must be replaced with `~`.

 **Implementation:**
    • status: analyzing --> implementing
     

  • Erik Hänel

    Erik Hänel - 2016-09-11
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -8,7 +8,9 @@
 The error comes from the non-replacements of the path separators in the final numerical variable name, because then the parser doesn't recognize it as a variable. In the variable name all path specific chars (`:`, `/` and `\\`) must be replaced with `~`.

 **Implementation:**
+The fix was implemented as proposed by the analysis. Implementation tests were passed successfully.

 **Documentation:**
+Not needed - bug fix.

 **Tests:**
    • status: implementing --> testing
     

  • Erik Hänel

    Erik Hänel - 2016-09-26
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -14,3 +14,4 @@
 Not needed - bug fix.

 **Tests:**
+Final tests passed successfully. Bug fix accepted.
    • status: testing --> fixed
     

  • Erik Hänel

    Erik Hänel - 2016-10-08
    • Labels: --> critical
     

Anonymous
Anonymous

Add attachments
Cancel





MongoDB Logo MongoDB