Hi Ralf

hi udai

I am having trouble with this add implementation

do you have time?

or should I just try more.......

anyway the problem the list

everytime I create an instance of add object a new list is being created

that's the expected behaviour

I think I just solved the problem

okay

I don't understand why this was happing ...let me try to explain

i'll see your solution when reviewing the patch

I was creating ArrayList in getMethod if the list property is found null

this way doesn't work

I just created a ArrayList at implementation (AbstractExp) and it begins to work as expected

okay

also I have created two list of exp one for additive and other for multiplicative

I think I should have just bugged you earlier as it solved my problem ;)

this doesn't sound rigth to me

i think there should only be one list

but i'll see what you did at review

first additive comes in () and then multiplicative in ()

((field <additives list>) <multiplicative list>)

how about: (a*b+c*d)*e+f

you assumed a specific order of operations

but add and mult expressions can occure in any order

like field.add(4).multiply(8).add(3)

will be ((field + 4 + 3) * 8)

so this is wrong

it should be (((field +4)*8)+3)