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

Prepare for CDL netlist export #1049

Merged
merged 2 commits into from
Nov 15, 2024
Merged

Prepare for CDL netlist export #1049

merged 2 commits into from
Nov 15, 2024

Conversation

ThomasZecha
Copy link

Basic introductory refactoring work for code parts affected/used by the new feature CDL netlist export (#876)
based on https://google.github.io/styleguide/cppguide.html:
-Prefix class attributes with a_ (much more better readability!)
-Class member initialization via constructor member initialization list
-No public class attributes

@ThomasZecha
Copy link
Author

@ra3xdh: Is there a particular reason why old Q3PtrList is used?
If not: I would recomment/switch to something like QList<QSharedPointer< ... >>

@ra3xdh
Copy link
Owner

ra3xdh commented Nov 8, 2024

Is there a particular reason why old Q3PtrList is used?

The total removing of the Q3PtrList wrapper is planned. See #748 The main concern is "auto delete" feature. Some internal logic may depend on it. It requires to carefully review every occurrence of Q3PtrList. The ideas and contributions are welcome. The same for other remaining Qt3 wrappers from qt3compat subdirectory. It the long perspective all qt3support leftover must be removed.

@ra3xdh
Copy link
Owner

ra3xdh commented Nov 8, 2024

Basic introductory refactoring work for code parts affected/used by the new feature CDL netlist export

The commit name and PR name are a bit misleading. Maybe better Prepare for CDL netlist export.

@ra3xdh ra3xdh changed the title New Feature: CDL netlist export Prepare for CDL netlist export Nov 8, 2024
@ra3xdh ra3xdh added this to the 25.1.0 milestone Nov 8, 2024
@ThomasZecha
Copy link
Author

New title Prepare for CDL netlist export is ok for me

@ThomasZecha
Copy link
Author

"The main concern is "auto delete" feature."
I think that can be fully assured by the shared-pointer logic.

iwbnwif added a commit to iwbnwif/qucs_s that referenced this pull request Nov 10, 2024
@ra3xdh
Copy link
Owner

ra3xdh commented Nov 10, 2024

I have planned merging. I will provide a review or merge in the next week.

@ThomasZecha
Copy link
Author

Thanks for the info!

@ra3xdh
Copy link
Owner

ra3xdh commented Nov 15, 2024

Please resolve conflict at externsimdialog.cpp and rebase if necessary. The conflict has appeared after merging #1057

ThomasZecha added 2 commits November 15, 2024 12:35
Introduced very basic c++ coding style for the extsimkernels subsystem
based on https://google.github.io/styleguide/cppguide.html:
-Prefix class attributes with a_ (much more better readability!)
-Class member initialization via constructor member initialization list
-No public class attributes

Signed-off-by: ThomasZecha <[email protected]>
Introduced very basic c++ coding style for the class Schematic and it's
parent-class QucsDoc based on https://google.github.io/styleguide/cppguide.html:
-Prefix class attributes with a_ (much more better readability!)
-Class member initialization via constructor member initialization list
-No public class attributes (where possible with small effort)

Signed-off-by: ThomasZecha <[email protected]>
@ThomasZecha
Copy link
Author

Rebased and resolved conflict.

@ra3xdh ra3xdh merged commit 1bbd4bd into ra3xdh:current Nov 15, 2024
7 checks passed
@ra3xdh
Copy link
Owner

ra3xdh commented Nov 15, 2024

Merged.

ThomasZecha pushed a commit to ThomasZecha/qucs_s that referenced this pull request Nov 22, 2024
Reviewed changes from ra3xdh#1049

SpiceLibCompDialog has been reworked to achieve the code quality
improvements introduced by ra3xdh#1049.

Tested the following dialogs under extsimkernel/:
-ExternSimDialog
-CustomSimDialog
-SimSettingsDialog
-SpiceLibCompDialog
ThomasZecha pushed a commit to ThomasZecha/qucs_s that referenced this pull request Nov 25, 2024
Reviewed changes from ra3xdh#1049

SpiceLibCompDialog has been reworked to achieve the code quality
improvements introduced by ra3xdh#1049.

Tested the following dialogs under extsimkernel/:
-ExternSimDialog
-CustomSimDialog
-SimSettingsDialog
-SpiceLibCompDialog

-rebased
ThomasZecha pushed a commit to ThomasZecha/qucs_s that referenced this pull request Nov 30, 2024
Reviewed changes from ra3xdh#1049

SpiceLibCompDialog has been reworked to achieve the code quality
improvements introduced by ra3xdh#1049.

Tested the following dialogs under extsimkernel/:
-ExternSimDialog
-CustomSimDialog
-SimSettingsDialog
-SpiceLibCompDialog

Rebased.
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