update old links #276
Merged
update old links #276
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tcNickolas
reviewed
Oct 2, 2018
| @@ -18,7 +18,7 @@ The most important property of the condensation graph is that it is **acyclic**. | |||
| The algorithm described in the next section extracts all strongly connected components in a given graph. It is quite easy to build a condensation graph then. | |||
|
|
|||
| ## Description of the algorithm | |||
| Described algorithm was independently suggested by Kosaraju and Sharir at 1979. This is an easy-to-implement algorithm based on two series of [depth first search](http://e-maxx-eng.appspot.com/graph/depth-first-search.html), and working for $O(n + m)$ time. | |||
| Described algorithm was independently suggested by Kosaraju and Sharir at 1979. This is an easy-to-implement algorithm based on two series of [depth first search](https://cp-algorithms.com/graph/depth-first-search.html), and working for $O(n + m)$ time. | |||
There was a problem hiding this comment.
For this link and the next one, it would be better to use relative links instead of the absolute ones: depth first search
|
There is one more occurrence of old URL in the pages, at src/combinatorics/generating_combinations.md. Do you want to fix it to a relative link too? This way all related changes would be in one commit. |
|
Actually I already fixed it, while tcNickolas did the review. Maybe I shouldn't do all the work myself, and let the people fix their own mistakes? |
|
Thanks for the fixes @jakobkogler :) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
update old links (http://e-maxx-eng.appspot.com) with (https://cp-algorithms.com)