Bug(s) in src/vfvm_assemblydata.jl

[jotaraz]

In two functions in src/vfvm_assemblydata.jl the iparam-for-loop to be moved into some other loops such that all used variables are defined.

• line 337: @inline function assemble_res_jac(edge... the iparam-for-loop has to be inside the isnodespecies(system, ispec, K) if-statement, otherwise idofL is not defined.

• line 400: @inline function assemble_res_jac(bedge... the iparam-for-loop has to be inside idofK for-loop and the isnodespecies(system, ispec, K) if-statement, otherwise idofK and idofL are not defined.

In the first function:

[j-fu]

Well, Julia's scoping rules are different from C/C++...

This code is tested in Example430. It worked for two reasons:

• once idofL is set to some value, it is defined also outside the if/end clause, as this does not create a scope block

• The example is 1D has one species, so there is only one possible value of idofL

... need to add a multispecies test here, so I keep this open...