Show 23 quoted lines
> Phillip Wood <phillip.wood123@gmail.com> writes:
>> On 10/05/2025 14:46, Leon Michalak via GitGitGadget wrote:
>>
>>> @@ -78,6 +82,19 @@ void init_add_i_state(struct add_i_state *s, struct repository *r)
>>> repo_config_get_string(r, "diff.algorithm",
>>> &s->interactive_diff_algorithm);
>>> + if (!repo_config_get_int(r, "diff.context", &context)) {
>>> + if (context < 0)
>>> + die(_("%s cannot be negative"), "diff.context");
>>> + else
>>> + s->context = context;
>>> + };
>>> + if (!repo_config_get_int(r, "diff.interHunkContext", &interhunkcontext)) {
>>> + if (interhunkcontext < 0)
>>> + die(_("%s cannot be negative"), "diff.interHunkContext");
>>> + else
>>> + s->interhunkcontext = interhunkcontext;
>>> + };
>>
>> Thanks for changing this. This iteration of the code changes looks good
>
> Lose the ';' (semicolon) after closing {brace}s.
> This is C; you do not need an empty statement after a {block}.