René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"revwalk hide callback not changeable?","articleBody":"The documentation for *git_revwalk_new* says\r\n\r\n\u003e This revision walker uses a custom memory pool and an internal commit cache, so it is relatively expensive to allocate.\r\n\u003e \r\n\u003e For maximum performance, this revision walker should be reused for different walks.\r\n\r\nThis is facilitated by *git_revwalk_reset* which removes all state on the revwalk object *except* (it seems) the hide callback function. This would be fine if I could just unset the callback myself, but *git_revwalk_add_hide_cb* errors out if a callback is already set, even if all I'm trying to do is to set it to `NULL`.\r\n\r\nSo it seems in the current state that the revwalk object can be reused but only if you want to use the same callback function. This seems a little restrictive, and I don't see any reason why we can't change the callback, either in *git_revwalk_add_hide_cb* or *git_revwalk_reset*, or both.\r\n\r\nI'd be happy to submit a fix so long as someone indicates which solution is preferable.","author":{"url":"https://github.com/TheBB","@type":"Person","name":"TheBB"},"datePublished":"2018-11-14T14:24:48.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/4887/libgit2/issues/4887"}