Recursive inheritance of classes #30
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When there is something like: ``` package PDL::Transform; our @EXPORT_OK = qw(apply invert map map unmap t_inverse t_compose t_wrap t_identity t_lookup t_linear t_scale t_offset t_rot t_fits t_code t_cylindrical t_radial t_quadratic t_cubic t_quadratic t_spherical t_projective ); our %EXPORT_TAGS = (Func=>\@EXPORT_OK); use PDL::Core; use PDL::Exporter; use DynaLoader; our @isa = ( 'PDL::Exporter','DynaLoader' ); push @pdl::Core::PP, __PACKAGE__; bootstrap PDL::Transform ; ``` in the code and later on: ``` .... { package PDL::Transform::Linear; our @isa = ('PDL::Transform'); *_opt = \&PDL::Transform::_opt; ... ``` this results in the line: ``` namespace PDL { class Transform: public ::PDL::Exporter, public ::DynaLoader, public ::PDL::Transform { ``` i.e. recursive inheritance, which has now been excluded by means of the `if` clause. Note: the meaning of the `{` here is not clean whether this is some sort of innerclass (in Cpp terms) and might have to be handled differently. The pach just removes the problem.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When there is something like:
in the code and later on:
this results in the line:
i.e. recursive inheritance, which has now been excluded by means of the
ifclause.Note: the meaning of the
{here is not clean whether this is some sort of innerclass (in Cpp terms) and might have to be handled differently. The pach just removes the problem.The source code of the file where the problem occurred and its translation by the filter: transform.tar.gz
Original repository: https://github.com/PDLPorters/pdl
Found by Fossies