Skip to content

Simplify code complexity - #296

Draft
llrs-roche wants to merge 2 commits into
mainfrom
295_cyclocomplexity
Draft

Simplify code complexity#296
llrs-roche wants to merge 2 commits into
mainfrom
295_cyclocomplexity

Conversation

@llrs-roche

Copy link
Copy Markdown
Contributor

Pull Request

Fixes #295

                         name cyclocomp
16         extract_occurrence        16
12        extract_assign_call        15
39 move_functions_after_arrow        10
36                 lang2calls         7
6                      c.qenv         6
20       fix_shifted_comments         6

Reduced in half the extract_occurrence complexity by moving a branch of code to the function extract_assign_call.
extract_dependency() was simplified to be out of the top 6 too.

Setting to draft as it might need more work to make it readable

@llrs-roche llrs-roche added the core label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@llrs-roche

Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: High cyclomatic complexity leds to bad readability

1 participant