Skip to content
This repository has been archived by the owner on Feb 4, 2021. It is now read-only.

Consolidate adjacency list methods under central unexported functions #17

Closed
sdboyer opened this issue Apr 18, 2014 · 1 comment
Closed
Assignees

Comments

@sdboyer
Copy link
Owner

sdboyer commented Apr 18, 2014

there's a shitton of code duplication with the existing adjacency lists. a lot of that could be done away with (along with the shitty pseudoinheritance) by replacing the guts of the logic with some unexported functions with type switches.

while this would shift the internal logic out of a single place for a struct, it would centralize the deviations for any given operation into a single place - an overall complexity reduction, methinks. plus, organizing a struct in one file is kind of an artificial thing in go, anyway.

@sdboyer
Copy link
Owner Author

sdboyer commented Apr 28, 2014

done a fair bit of this, but it's increasingly seeming like it's better to reduce the N factor is by reducing the complexity in other areas. closing this for now, may reopen later.

@sdboyer sdboyer closed this as completed Apr 28, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant