Skip to content

Null pointer when reindexing edge list#111

Closed
raahlb wants to merge 4 commits into
boostorg:developfrom
raahlb:fix_remove_vertex
Closed

Null pointer when reindexing edge list#111
raahlb wants to merge 4 commits into
boostorg:developfrom
raahlb:fix_remove_vertex

Conversation

@raahlb

@raahlb raahlb commented Jul 30, 2018

Copy link
Copy Markdown

Don't copy value_type if not needed, as source's m_property will be set to null when copied.

See https://svn.boost.org/trac10/ticket/13544.

@jeremy-murphy

Copy link
Copy Markdown
Collaborator

Would it be relatively easy to add a test-case that fails without this fix?

@raahlb

raahlb commented Aug 23, 2018

Copy link
Copy Markdown
Author

I added a test case. Also added it to the Jamfile.v2. Not used to this test system, so please verify I did it in the right way.

@anadon

anadon commented Aug 31, 2018

Copy link
Copy Markdown
Contributor

I'm helping out with the PR backlog. Looks like you have a bugfix, test, and no new examples will be needed. This will be in the first batch of merged PR's. This is to let you know and help me prioritize PR's.

@anadon

anadon commented Sep 1, 2018

Copy link
Copy Markdown
Contributor

Your test file was not included in the test directory. Make sure you added it to git.

@anadon

anadon commented Sep 1, 2018

Copy link
Copy Markdown
Contributor

I see what happened. The test you added has a different file name than the test file you added. I'll just fix that up.

@raahlb

raahlb commented Sep 2, 2018

Copy link
Copy Markdown
Author

Missed that you wrote that you had fixed it yourself. Not super important, but the name "delete edge" is a bit misleading, it's actually a vertex that is being deleted in the test. Sorry about that.

Thanks!

@jzmaddock

Copy link
Copy Markdown
Contributor

Are these changes in #114? If so this can be closed now.

@anadon

anadon commented Oct 12, 2018

Copy link
Copy Markdown
Contributor

These changes are in #114 and so this should be closed.

@raahlb raahlb closed this Oct 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants