Issue with variable case sensitivity about jace

[pieterderycke]

This seems to be bug related to casing. I have already done the analysis and it is only occurring when you construct a Func accepting a dictionary. When the calculate is called directly on the CalculationEngine object, the bug is not occurring.

CalculationEngine engine = new CalculationEngine();

Dictionary<string, double> variables = new Dictionary<string, double>();
variables.Add("var1", 2);
variables.Add("otherVariable", 4.2);

double result = engine.Calculate("var1+2/(3*otherVariable)", variables);

I will do the necessary modifications to fix the bug and release it as part of version 0.8.4. Currently, all variables in Jace should be case insensitive. Should I also add an option to make them case sensitive? Or are case insensitive variables sufficient for you?

[jjneuhart]

Pieter,

I don't really see a need for case sensitivity, at least on the project
that I am working on. Thanks for fixing the bug, and thanks for all of
your great work.

Jordan Neuhart

On Mon, Dec 9, 2013 at 12:05 AM, Pieter De Rycke
[email protected]:

This seems to be bug related to casing. I have already done the analysis
and it is only occurring when you construct a Func accepting a dictionary.
When the calculate is called directly on the CalculationEngine object, the
bug is not occurring.

CalculationEngine engine = new CalculationEngine();
Dictionary<string, double> variables = new Dictionary<string, double>();variables.Add("var1", 2);variables.Add("otherVariable", 4.2);
double result = engine.Calculate("var1+2/(3*otherVariable)", variables);

I will do the necessary modifications to fix the bug and release it as
part of version 0.8.4. Currently, all variables in Jace should be case
insensitive. Should I also add an option to make them case sensitive? Or
are case insensitive variables sufficient for you?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-30107471
.

email: [email protected]
cell: 402-217-1377

[pieterderycke]

Jordan,

Jace 0.8.4 has been released. It fixes your bug.

Kind regards,
Pieter

Date: Mon, 9 Dec 2013 06:57:56 -0800
From: [email protected]
To: [email protected]
CC: [email protected]
Subject: Re: [Jace] Issue with variable case sensitivity (#1)

Pieter,

I don't really see a need for case sensitivity, at least on the project

that I am working on. Thanks for fixing the bug, and thanks for all of

your great work.

Jordan Neuhart

On Mon, Dec 9, 2013 at 12:05 AM, Pieter De Rycke

[email protected]:

This seems to be bug related to casing. I have already done the analysis

and it is only occurring when you construct a Func accepting a dictionary.

When the calculate is called directly on the CalculationEngine object, the

bug is not occurring.

CalculationEngine engine = new CalculationEngine();

Dictionary<string, double> variables = new Dictionary<string, double>();variables.Add("var1", 2);variables.Add("otherVariable", 4.2);

double result = engine.Calculate("var1+2/(3*otherVariable)", variables);

I will do the necessary modifications to fix the bug and release it as

part of version 0.8.4. Currently, all variables in Jace should be case

insensitive. Should I also add an option to make them case sensitive? Or

are case insensitive variables sufficient for you?

—

Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-30107471

.

email: [email protected]

cell: 402-217-1377

—
Reply to this email directly or view it on GitHub.