Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🩹 Fix regexes and memory leaks #47

Merged
merged 5 commits into from
Feb 27, 2024
Merged

🩹 Fix regexes and memory leaks #47

merged 5 commits into from
Feb 27, 2024

Conversation

Thomaltarix
Copy link
Collaborator

I fixed the parsing regexes so that this type of files can pass:

# Basic wire, direct link true to output.
#
# TRUE ---> OUTPUT

.chipsets:
                    input                   input1                  #b  o       njou        r
clock clock1

.links:
             input1     :     1        input1 :      1                      #helloworl d

And i also fixed all the memory leaks of the project.
It mostly consisted in deleting maps in the classes.

@Thomaltarix Thomaltarix added bug Something isn't working files File parsing related labels Feb 26, 2024
@Thomaltarix Thomaltarix self-assigned this Feb 26, 2024
@RenardFute
Copy link
Member

You sure "input1 : 1" should pass ?

@Thomaltarix
Copy link
Collaborator Author

I think so 👍

@Thomaltarix Thomaltarix merged commit d17feaa into main Feb 27, 2024
2 checks passed
@RenardFute RenardFute deleted the 3-file-parsing branch February 28, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working files File parsing related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants