[Issue#77]: Fix bug in update_blocks: the father block was not correctely set

[Issue#77]: Remove CodeCleaner.py

[Issue#77]: function read_file removed from ReadSourceFile class. class ReadSourceFile moved to SourceFile

[Issue#77]: Removed SourceManager: it was containing functions that belonged somwhere else.

[Issue#77]: Fix for issue sparonuz/AutoRPE#147: these scripts are now useless

[Issue#77]: Added parameter working precision to function that checks if two types are equivalent

[Issue#77]: FIX: get_type_of_content was giving char for argument that are function with char argument, but the function per se was not char

[Issue#77]: Module class was loading again lines from source file, which is kind of overkill, since this is already done by SourceFile class. This is moreover wrong is some changes were done in between

[Issue#77]: SubprogramCall has now a member to store the block were the call is done

[Issue#77]: Functions to add cast to non coherent calls: for the moment has been tested on the firs calls found

[Issue#77]: Added function to create a regex for name occurrence: was failing if name contained parenthesis

[Issue#77]: Now adds the #  include \"single_precision_substitute.h90\" clause

[Issue#77]: Added fix for reshapes

[Issue#77]: Dependece were not propagated correctely because the id property was used: it did't make much sense

[Issue#77]: For the moment commented out the part that should suggest to create an interface

[Issue#77]: added function to escape string to put them in regex

[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

Create a function to read the argument params

Get the variable from namelist

Search the variable based on the namelist name

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]: 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

[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

[Issue#104]: Function merge_lines was basically defined twice: removed one and update the one inside Inserter. Removed more useless functions inside InterfaceGenerator.py.

[Issue#104]: Starting from the question "Is it Finder.py the best file to have that function?" I decided that the file was useless, and ridistribute the function in it.

[refactor_parseSources]: The idea is to always read the sources into load_source_files routine, and remove a lot of useless steps

[refactor_parseSources]: Managed case in which two function with the same name exist

[develop]: moved and improved  regex for capturing subprogram calls

[develop]: Fix get_procedure_by_name that was a bit of a mess

[develop]: mv source_file to source_file_info + fixed storage of interface (everything was doubled) + imports

[develop]: Now basic structures have module info

[develop]: Intruduced function to get derived type constructor

[develop]: Now call_to_function does not recognize structures as functions

[develop]: New regex for  split_read_arguments

[develop]: Introduced get_procedure function: is meant to replace calls_to_subprogram and the great number of fixed within. It does not identify vectors as funcions.

[develop]: _fix_functions has been refactored, it was a mess of useless call

[develop]: Now module also have the lines saved in it, and a function to rebuild text

[develop]: Save vault added as a function to vault

[develop]: Moved load vault and save vault to Vault file. Now parse source is called inside the Vault constructor

[develop]: Added function to find var used in namelist, function split_by_appearance_in_namelist was taking around 3/4 minutes. Now we parse this info in parse sources, and this function takes around 1 sec

[develop]: removed old function + simplified regex for finding function/srb declaration

[develop]: Moved function clean_contents from cleaner to Getter

[develop]: Simplified function to detect subrutine calls

[develop]: Refactored function fix subroutine

[develop]: Added function to fix externals

[develop]: Refactored ImplementRPE, now this class is used just for Implementing RPE

[develop]: Added function to store external subroutine used

[develop]: Removed functions fix_functions fix_subroutines fix_external_subroutines , and replaced by fix_subprogram

[develop]: Now also used external functions are stored in vault. Bettered regex for call_to_functions

[develop]: Removed functions to parse implicit functions, that are no more used, anyway they are store in https://earth.bsc.es/gitlab/otinto/AutoRPE/-/snippets/73

[develop]: Removed from Inserter all the duplicated function that were searching variables in sbr call to truncate then the var.

[Issue#107]: Removed DerivedTypeDealer, and moved functions elsewhere

[Issue#107]: Now the get_real_in_line uses the get_procedure, and does a check on the pointer. Removed _clean_from_intrinsics, now uses strip_from_intrinsics.

[develop]: Most of the code in TrackDependency was duplicated, removed and put the right call.  Now MakeVault works properly.

[develop]: Moved load vault and save vault to Vault file. Now parse source is called inside the Vault constructor

[develop]: fix module module.main() call to module.main + added module name info to  vault.get_procedure_by_name when available

[refactor_parseSources]: The idea is to always read the sources into load_source_files routine, and remove a lot of useless steps

i[refactor_parseSources]: The idea is to always read the sources into load_source_files routine, and remove a lot of useless steps

[otinto/AutoRPE#91]: CleanCode cleans code, and has to be used to produce sources for ImplementRPEtoSources and MakeVault scripts

[otinto/AutoRPE#91]: Function preprocess_sources now divided between the part that cleans the code and the one that creates files with RPE type declaration
[otinto/AutoRPE#91]: No tmp folder are required in MakeVault.
[otinto/AutoRPE#91]: Removed some unused function, and removed useless layer for getting members of structures
[otinto/AutoRPE#91]: BugFix for allocate with '::'symbol detected as var declaration
[otinto/AutoRPE#91]: Update of the test with refactoring, and fix for imports`
[otinto/AutoRPE#91]: Added array for external types used and manage the case a external type is used
[otinto/AutoRPE#91]: Test for the find_function_type_from_header were failing, fixed a bug. Closing Issue #93
[otinto/AutoRPE#91]: Fixed for: functions as arugments, external variables as arguments, a call to get_type_of_contents was made without specifying the routine.
[otinto/AutoRPE#91]: Introduction of get_function_type , to shorten a bit get_type_of_contents, that is still too long.
[otinto/AutoRPE#91]:  Regex for hardcoded array was matching with array inside function. Closes #95 .
[otinto/AutoRPE#91]: check for keyword argument was missing in  get_dimension_of_contents
[otinto/AutoRPE#91]: Partially fix the case a write statement was using an array. Added corresponding test
[otinto/AutoRPE#91]: Changed the name of replace_real to replace_real_with_RPE_declaration + refactoring
[otinto/AutoRPE#91]: Fixed and moved regex for real declaration (Issue #96 ). Small renaming
[otinto/AutoRPE#91]:  This solves the variables duplication
[otinto/AutoRPE#91]: Function/Subroutine/Interfaces are subprograms, Function/Subroutine/Interfaces/Main/DerivedType are procedures. Everything was mixed, and called routine, sometimes procedure.
[otinto/AutoRPE#91]: Renamed function that fill dictionary of var, and fix the syntax.
[otinto/AutoRPE#91]: Trying to speed up with dictionary otinto/AutoRPE#41
[otinto/AutoRPE#91]: DerivedType class moved to BasicStructure for coherency. Changes for adding the dict of accessible_var
[otinto/AutoRPE#91]: is_call_to_function_vault added check for intrinsics + added check for array  to is_call_to_function+Formatting
[otinto/AutoRPE#91]:fix import + get_variable_by_name: now properly uses dictionary accessible_var, in the correct order + function for filling those dict