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

append delete or copy delete #11

Closed
ggaaooppeenngg opened this issue Jan 5, 2015 · 2 comments
Closed

append delete or copy delete #11

ggaaooppeenngg opened this issue Jan 5, 2015 · 2 comments

Comments

@ggaaooppeenngg
Copy link

https://github.com/gyuho/goraph/blob/refactor/graph/graph.go#L280

why not just,d.outEdge[src] = append(d.OutEdges[src][:idx],d.OutEdges[src][idx+1:]...)?
Is there any reason for efficiency?
@gyuho
Copy link
Owner

gyuho commented Mar 13, 2015

I changed a lot of code base, with much simplification. Much can be simply done with map data structure.

@gyuho gyuho closed this as completed Mar 13, 2015
@gyuho
Copy link
Owner

gyuho commented Sep 8, 2015

Just finished a huge refactoring, if you are still interested.
#49

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ggaaooppeenngg @gyuho