{"thread":{"id":"51932","subject":"[PATCH v2] commit-graph: remove a duplicate assignment","startedAt":"2019-09-27T02:20:01Z","lastAt":"2019-09-27T12:45:31Z","messageCount":3,"participants":["Alex Henrie","Johannes Schindelin","Derrick Stolee"],"isPatch":true,"patchVersion":2,"patchTotal":null},"messages":[{"id":"383049","messageId":"20190927021952.364646-1-alexhenrie24@gmail.com","threadId":"51932","inReplyTo":null,"subject":"[PATCH v2] commit-graph: remove a duplicate assignment","fromName":"Alex Henrie","fromEmail":"alexhenrie24@gmail.com","sentAt":"2019-09-27T02:19:52Z","receivedAt":"2019-09-27T02:20:01Z","isPatch":true,"sender":{"key":"alexhenrie24@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5951993?v=4"},"body":"Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>\n---\n commit-graph.c | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/commit-graph.c b/commit-graph.c\nindex 9b02d2c426..d0e1f9e1f2 100644\n--- a/commit-graph.c\n+++ b/commit-graph.c\n@@ -1522,7 +1522,7 @@ static int write_commit_graph_file(struct write_commit_graph_context *ctx)\n \n static void split_graph_merge_strategy(struct write_commit_graph_context *ctx)\n {\n-\tstruct commit_graph *g = ctx->r->objects->commit_graph;\n+\tstruct commit_graph *g;\n \tuint32_t num_commits = ctx->commits.nr;\n \tuint32_t i;\n \n-- \n2.23.0\n\n"},{"id":"383061","messageId":"nycvar.QRO.7.76.6.1909271418170.15067@tvgsbejvaqbjf.bet","threadId":"51932","inReplyTo":"20190927021952.364646-1-alexhenrie24@gmail.com","subject":"Re: [PATCH v2] commit-graph: remove a duplicate assignment","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2019-09-27T12:21:00Z","receivedAt":"2019-09-27T12:21:19Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\n\nOn Thu, 26 Sep 2019, Alex Henrie wrote:\n\n> Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>\n\nHrm, the commit message should reflect Stolee's careful consideration,\nno? An empty commit message won't be helpful e.g. when cherry-picking on\ntop of a branch that already has your v1.\n\nCiao,\nJohannes\n\n> ---\n>  commit-graph.c | 2 +-\n>  1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/commit-graph.c b/commit-graph.c\n> index 9b02d2c426..d0e1f9e1f2 100644\n> --- a/commit-graph.c\n> +++ b/commit-graph.c\n> @@ -1522,7 +1522,7 @@ static int write_commit_graph_file(struct write_commit_graph_context *ctx)\n>\n>  static void split_graph_merge_strategy(struct write_commit_graph_context *ctx)\n>  {\n> -\tstruct commit_graph *g = ctx->r->objects->commit_graph;\n> +\tstruct commit_graph *g;\n>  \tuint32_t num_commits = ctx->commits.nr;\n>  \tuint32_t i;\n>\n> --\n> 2.23.0\n>\n>\n"},{"id":"383062","messageId":"adf9c41a-47af-04dd-d1ed-c181eba80dab@gmail.com","threadId":"51932","inReplyTo":"nycvar.QRO.7.76.6.1909271418170.15067@tvgsbejvaqbjf.bet","subject":"Re: [PATCH v2] commit-graph: remove a duplicate assignment","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2019-09-27T12:45:27Z","receivedAt":"2019-09-27T12:45:31Z","isPatch":true,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 9/27/2019 8:21 AM, Johannes Schindelin wrote:\n> Hi,\n> \n> \n> On Thu, 26 Sep 2019, Alex Henrie wrote:\n> \n>> Signed-off-by: Alex Henrie <alexhenrie24@gmail.com>\n> \n> Hrm, the commit message should reflect Stolee's careful consideration,\n> no? An empty commit message won't be helpful e.g. when cherry-picking on\n> top of a branch that already has your v1.\n\nI agree that we need more in the commit messages here. It is\nhelpful to include _why_ you are doing this. Keep in mind\nthat your cover letter is not recorded with the commit at all.\n\nFurther, when updating a patch series, you should re-roll the\nentire series as a v2 (preferably as a reply to the original\nseries). Keep in mind that this patch should NOT be applied\non top of your other series because then 'g' is not assigned\nat all. Mechanically, it COULD apply, but it would break\nin tests.\n\nPatches 2 and 3 in your series also have feedback that would\nbe good to apply in your v2.\n\nThanks,\n-Stolee\n"}]}