Skip to content

Checking for size of outgoing and incoming edges of the vertex after removing a single edge#771

Merged
BanzaiMan merged 1 commit intojruby:masterfrom
prathamesh-sonpatki:vertex_specs
Jun 7, 2013
Merged

Checking for size of outgoing and incoming edges of the vertex after removing a single edge#771
BanzaiMan merged 1 commit intojruby:masterfrom
prathamesh-sonpatki:vertex_specs

Conversation

@prathamesh-sonpatki
Copy link
Contributor

  • After removing all incoming or outgoing edges, incoming or outgoing
    edges of a vertex are set to null
  • But after removing a single edge, it is not checked that outgoing of
    the source or incoming of the destination should be changed to null
    or not
  • This commit checks that condition and sets outgoing and incoming of
    source and destination respectively to null if the condition is met.

@ghost ghost assigned subbuss Jun 7, 2013
@BanzaiMan
Copy link
Member

We fixed CI. Could you rebase this PR off 0ccfa36 and try again?

(Also, @enebo and @subbuss, please review this PR.)

removing a single edge

- After removing all incoming or outgoing edges, incoming or outgoing
  edges of a vertex are set to null
- But after removing a single edge, it is not checked that outgoing of
  the source or incoming of the destination should be changed to null
  or not
- This commit checks that condition and sets outgoing and incoming of
  source and destination respectively to null if the condition is met.
@prathamesh-sonpatki
Copy link
Contributor Author

@BanzaiMan I rebased to master but it still failed for 1.9 https://travis-ci.org/jruby/jruby/jobs/7876718 & https://travis-ci.org/jruby/jruby/jobs/7876719

BanzaiMan added a commit that referenced this pull request Jun 7, 2013
Checking for size of outgoing and incoming edges of the vertex after removing a single edge
@BanzaiMan BanzaiMan merged commit fbd6c9d into jruby:master Jun 7, 2013
@BanzaiMan
Copy link
Member

On a second thought, the patch looks reasonable to me. I also added spec:ir task to Travis, so we run them in CI.

@prathamesh-sonpatki prathamesh-sonpatki deleted the vertex_specs branch June 8, 2013 05:59
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.

3 participants