Added REGEX patterns for string array initialization

Added custom Error for subprogram not found

Join procedures_dictionary lists into a single search_array

Refactored get_type_of_contents and get_structure_member for pointers to procedures

Check that it's not external and use vault.get_derived_type_constructor instead of vault.get_procedure_by_name

Get arguments from READ() by using the CallManager.find_call_arguments()

Fill the accessible_var dictionary for the main procedure

Avoid misleading errors by removing the indexing before searching the variable

[Issue#147]: Added flag to mark variables used in external calls

[Issue#147]: Function to fill vault has been excorporated. Now makeVault assigns ids to all real variables: these were lines that got lost in refactoring.

[Issue#147]: Modified main of Modify code. Now 1) read the banned_var list + vault 2) add var used in external calls to ban list 3) propagate dependency of the ban list

[Issue#147]: Moved function change_code form inserter to ModifyCode

[Issue#147]: Moved function from Inseter + var used in external have to be banned just if they are RPE type

[Issue#147]: Added additional parameter "var_type" for tracking function

[Issue#147]: Added incremental_id to variables, to be used during tracking.

[Issue#147]: Added var_type to function that propagate dependency. Now var_identifier uses var.incremental_number

[Issue#147]: Added tracking of RPE variables

[Issue#147]: Modified function load_sources to take extension as input, and to be able to read files in subdirectories

[Issue#147]: Extension is now passed to create ReadSourceFile objects, to properly set module_name

[Issue#147]: Updated rebuild_text, so as to split lines with \n marker in module.lines

[Issue#147]: Moved all function to deal with non preprocessed code into a single file

[Issue#147]: Added parameter to read file to avoid removing spaces when reading. Fixed bug in write_file that was adding a last empty line to every file.

[Issue#147]: Check that, when looking for allocation of a var  we did not match a member with the same name: a_i will also match %a_i

[Issue#147]: Storing precision with and without KIND keyword specification was useless: removed

[run_with_ICE]:  Issue sparonuz/AutoRPE#150, now fix_parameter_declaration has been refactored in somethin comprehensible

[run_with_ICE]:  Added ret val to EPSILON intrinsic

[run_with_ICE]: Added escape for . in numbers in function replace variables exact match

Check that it's not external and use vault.get_derived_type_constructor instead of vault.get_procedure_by_name

[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