Skip to content

Conversation

@louisjdmartin
Copy link
Member

No description provided.

jverdugo-aqclab and others added 4 commits January 14, 2025 16:27
* Bug fix: parsing errors (COM.NAME.Homonymy: Location unreachable or IndexOutOfBoundsException) were produced when analyzed files contained declaration of functions or subroutines with modifiers Recursive, Elemental or Pure

* Bug fix: Parsing error (IndexOutOfBoundsException) was produced when a file only contains comments, no source code. Also a JFlexException was thrown in the case of a file without code nor comments. The throw for this Exception has been removed to avoid parsing errors in the results.

* Bug fix: a parsing error (IndexOutOfBoundsException) was produced when analyzing files that included sentences like "PROCEDURE(proc_interface), POINTER :: proc"

* Rule for checking that lines of code in a file don't exceed a threshold of LOC (set at 1000). Only lines of code are taken into account, comments and blank lines are ignored.

* Rule for checking that lines of code in procedures don't exceed a threshold of LOC (set at 150). Only lines of code are taken into account, comments and blank lines are ignored.

* Rule ArgumentsProcedure

Rule to check the maximum number of arguments in each procedure, which must be at most 7.

* Rule CyclomaticComplexity

Rule to check the maximum value of the cyclomatic complexity in each function, which at most must be 15.

* Rule LogicUnit

Rule to check that only one module or program is declared in each file.

* Rule PercentageComment

Rule to check that in each file there are at least 30% of lines of comments with respect to the total number of lines.

* Rule CommentVar

* Rule Header

* Change new rules names to RNC rules

* Change name in comment

* Update fortran77-rules/src/main/resources/lex/COMFLOWCheckArguments.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMFLOWCheckArguments.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETComplexitySimplified.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETComplexitySimplified.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETLineOfCode.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETLineOfCode.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETLineOfCode.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMFLOWCheckArguments.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETRatioComment.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran77-rules/src/main/resources/lex/COMMETRatioComment.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETRatioComment.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETRatioComment.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETLineOfCode.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETComplexitySimplified.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMMETComplexitySimplified.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Update fortran90-rules/src/main/resources/lex/COMFLOWCheckArguments.lex

Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>

* Remove unused variable numTotal

* Header updated to reflect this rule is not defined in RNC.

* Test cases for new rules

---------

Co-authored-by: Ana Díaz Muñoz <adiaz@aqclab.es>
Co-authored-by: Ana Díaz Muñoz <120630965+adiaz-aqclab@users.noreply.github.com>
Co-authored-by: Diego <diego.rodriguez@cnes.fr>
Co-authored-by: Diego Rodriguez <63045276+diegorodriguez31@users.noreply.github.com>
Bumps ant from 1.10.4 to 1.10.11.

---
updated-dependencies:
- dependency-name: org.apache.ant:ant
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [org.json:json](https://github.com/douglascrockford/JSON-java) from 20190722 to 20231013.
- [Release notes](https://github.com/douglascrockford/JSON-java/releases)
- [Changelog](https://github.com/stleary/JSON-java/blob/master/docs/RELEASES.md)
- [Commits](https://github.com/douglascrockford/JSON-java/commits)

---
updated-dependencies:
- dependency-name: org.json:json
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@louisjdmartin louisjdmartin added this to the 5.1.0 milestone Jan 15, 2025
@dev31520 dev31520 merged commit f7bf365 into master Jan 15, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants