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.

Sign up for GitHub

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

Jump to bottom

Remove SArray class hierarchy #245

Open
smarr opened this issue Apr 24, 2018 · 0 comments
Open

Remove SArray class hierarchy #245

smarr opened this issue Apr 24, 2018 · 0 comments
Labels
enhancement Improves the implementation with something noteworthy good first issue Are you trying to have a good at SOMns? Start here! help wanted Would be great if you could help out here.
Milestone
later

Comments

@smarr
Copy link
Owner

smarr commented Apr 24, 2018

I think, we can do this cleaner without the hierarchy.

The hierarchy is problematic for guards, because the types are not leaf types.
And we do not really use the hierarchy to much advantage anyway.
Removing it will perhaps also give more flexibility when it comes to subclasses and the properties (value, transfer array, etc).
@smarr smarr added enhancement Improves the implementation with something noteworthy help wanted Would be great if you could help out here. good first issue Are you trying to have a good at SOMns? Start here! labels Apr 24, 2018
@smarr smarr added this to the later milestone Apr 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Improves the implementation with something noteworthy good first issue Are you trying to have a good at SOMns? Start here! help wanted Would be great if you could help out here.
Projects
None yet
Milestone
later
Development

No branches or pull requests
1 participant
@smarr