[ReintroducingExceptionManager]: This changes were done in an old branch, and never merged into master

[Issue#142]: Separation between allocatable and non allocatables array was not exlusive, variable were duplicated ( introduced while fixing sparonuz/AutoRPE#132)

[Issue#142]: BUGIFX: some variables had the id not set, because it was added to an arg obj instead.

Create a function to read the argument params

Get the variable from namelist

Search the variable based on the namelist name

Transformed external_types to set

return when finding the first ALLOCATE

[Issue#128]: Removes duplicated module procedure inside interfaces interfaces generated by substitution of real with rpe_var

[Issue#130]:  Fixed error in assignation of rpe with an array reshape.

[Issue#130]: Removing multiple spaces from code makes it easier to fix things afterwards

[Issue#130]: Fix for intrinsic used as array indexing

[Issue#130]: Fixed problem of wrong subtitution with parameter declaration with reshape assignment.

Put sbits correctly for members after ALLOCATE and removed func find_module_and_line_of_allocation

Fixed error with getting the variable name and simplified functions

Use remove_if_conditionals

Loop the arguments searching for the variable we want

Split by allocatables or pointer variables

sparonuz authored Mar 19, 2021 and sparonuz committed Mar 25, 2021

[Issue#125]: Now the end of the sbr is searched by name, this implies changes in some source files: https://earth.bsc.es/gitlab/svn/nemo-mirror/-/issues/37

[Issue#125]:  Register if a subroutine is defining other subroutine, and if so add public var of this sbr to defined sbr accessible variables

[Issue#125]: get_variable does not have to find variables if they are inside intrinsics, we have to clean string in advance if we want the var to be found.

[Issue#125]: Fix for Issue sparonuz/AutoRPE#126

[Issue#125]: Fix for keyword argument which have the same name of the actual argument Issue sparonuz/AutoRPE#127

[Issue#122]: Data statement are no more used, I moved corresponding code for fix in code in https://earth.bsc.es/gitlab/otinto/AutoRPE/-/snippets/75

[Issue#122]: dummy_argument was a never used member of Procedure class. It can be filled and used + Issue #61

[develop]: Refactoring of get_procedure_inline. Now we are able to fix two call to same function on the same line

[develop]: Now output string is of the same length if input string, so not to brake regex.start index

[develop]: rpe is not a real type, and its inclusion in this list was causing wrong fixes in pointers assignations

[develop]: Removed script that was useless, this functions are already called in ImplementRPE + some refactoring

[Issue#114]: Variable's precision was not setted correctly: removed most of the array declared in VariablePrecision.py. Now  just a dictionary and a lookup table to encode the types defined in the code

[Issue#114]:  Added type to function that add cast to reals

[Issue#114]: Fixed codeChecker to work again, it was needed to change get_real_inline function to check for all INTENT type

[Issue#114]: BugFix: get_procedure_inline failed to indentify the call if a variable with a name similar to the called procedure preceded the call (x_solfrack(i,j) = solfrac([...]))

[Issue#114]: Removed horrible workaround to calculate type result of operation. There is now a dictionary containing the results

[develop]: ReplaceWrkAllocFunction file has been removed, still the code can be found here: https://earth.bsc.es/gitlab/otinto/AutoRPE/-/snippets/74

[develop]: Removed the list "procedures" from Vault and Module structures, now just procedures_dictionary for Vault is used

[Issue#111]: External subroutines and function calls are now stored inside normal subprogram structure, just with the external attribute . Same has to be done with external datatype

[Issue#111]: Issue #94, this function will replace clean_content, which is going to be deleted

[Issue#111]: Definitely removed clean_content

[Issue#111]: Fixed function has_intrinsics, now search for possible funcion instead of cycling over the 300 possible intrinsics

[Issue#111]: Now external structures stored along with others

[Issue#111]: Removed treatment of list_of_intrinsic as it were a propriety of vault.

[Issue#111]: Moved class Counter to Inserter, the only place were it is used.

[Issue#111]: Fixed load function that were using class ReadSourceFile when not needed. Moved class to basic struct

[Issue#111]: This was fixed back in otinto/AutoRPE#52, but wrong encoding was still present somewhere

[Issue#111]: Removed the class CurrentBlock, is now a function of module structure. Removed corresponding imports

[Issue#111]: CurrentBlock is now called just before loop line, and stores all the blocks in an array, and update it when necessary

[Issue#111]: Removed PointerDealer and redistrebuted functions

[Issue#111]:Storing the name of the namelist at parsing time, not to search for it later

[Issue#111]: Moved ObtainSourceFileInfo to functions collection, removed class

[Issue#111]: Unified the function to read the input sources using the class ReadSources