Stephen Smith writes:
> I can see three solutions and could support any of the three:
> 1) Move the free calls to run_status() and cmd_status().
> 2) Move the calls calls to wt_status_print since that is the last function
> from wt_status.c that is called befor the structure goes out of scope in
On Friday, September 7, 2018 3:31:55 PM MST Junio C Hamano wrote:
> For example, I noticed that both of the old
> callsites of wt_status_get_state() have free() of a few fiedls in
> the structure, and I kept the code as close to the original, but I
> suspect they should not be freed there in the fu
"Stephen P. Smith" writes:
> On Friday, September 7, 2018 3:31:55 PM MST you wrote:
>> Junio C Hamano writes:
>
>> The patch is mostly for illustration of the idea.
>>
>> The result seems to compile and pass the test suite, but I haven't
>> carefully thought about what else I may be breaking wi
On Friday, September 7, 2018 3:31:55 PM MST you wrote:
> Junio C Hamano writes:
> The patch is mostly for illustration of the idea.
>
> The result seems to compile and pass the test suite, but I haven't
> carefully thought about what else I may be breaking with this
> mechanical change. For exa
Junio C Hamano writes:
> "Stephen P. Smith" writes:
>
>> void wt_status_collect(struct wt_status *s)
>> {
>> +struct wt_status_state state;
>> wt_status_collect_changes_worktree(s);
>>
>> if (s->is_initial)
>> @@ -746,6 +752,11 @@ void wt_status_collect(struct wt_status *s)
>>
"Stephen P. Smith" writes:
> void wt_status_collect(struct wt_status *s)
> {
> + struct wt_status_state state;
> wt_status_collect_changes_worktree(s);
>
> if (s->is_initial)
> @@ -746,6 +752,11 @@ void wt_status_collect(struct wt_status *s)
> else
> wt_sta
6 matches
Mail list logo
