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

some refactoring. #832

Closed
wants to merge 3 commits into from
Closed

some refactoring. #832

wants to merge 3 commits into from

Conversation

dsm
Copy link
Collaborator

@dsm dsm commented Jul 9, 2024

Hi, this PR do some refactoring.

@ra3xdh ra3xdh added this to the 24.3.0 milestone Jul 9, 2024
@dsm dsm marked this pull request as draft July 9, 2024 10:32
@dsm
Copy link
Collaborator Author

dsm commented Jul 9, 2024

@ra3xdh I'm beginning to refactoring some q3PtrList but it take some time and tomorrow I'll be on a 5-day holiday.I don't used sharedObjectList from qucs.

@ra3xdh
Copy link
Owner

ra3xdh commented Jul 9, 2024

I'm beginning to refactoring some q3PtrList but it take some time and tomorrow I'll be on a 5-day holiday

That is good news. I can assign #748 to you. I am planning to prepare v24.3.0 release within next two weeks. So, we can schedule this refactoring for the v24.4.0.

@ra3xdh ra3xdh linked an issue Jul 9, 2024 that may be closed by this pull request
4 tasks
@ra3xdh ra3xdh modified the milestones: 24.3.0, 24.4.0 Jul 9, 2024
@wawuwo wawuwo mentioned this pull request Jul 16, 2024
@ra3xdh
Copy link
Owner

ra3xdh commented Jul 16, 2024

@dsm, Please look at #845. This PR removes Q3PtrList from Node class. Please check for possible conflicts.

@dsm
Copy link
Collaborator Author

dsm commented Jul 16, 2024

@ra3xdh Hi, Maybe I'll close this PR, I need to refactor the codes step by step for my limited time. I might have to look at it from a different perspective.

@dsm dsm closed this Jul 22, 2024
@ra3xdh ra3xdh removed this from the 24.4.0 milestone Jul 22, 2024
@ra3xdh ra3xdh removed a link to an issue Jul 22, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants