Re: [PATCH 08/15] sequencer: change complete_action() to use the refactored functions

2018-10-11 Thread Alban Gruin
Le 11/10/2018 à 15:51, Phillip Wood a écrit : > On 07/10/2018 20:54, Alban Gruin wrote: >> +    if (rewrite_file(todo_file, new_todo.buf.buf, new_todo.buf.len) < >> 0) { >> +    todo_list_release(_todo); >> +    return error_errno(_("could not write '%s'"), todo_file); >> +    } > >

Re: [PATCH 08/15] sequencer: change complete_action() to use the refactored functions

2018-10-11 Thread Phillip Wood
On 07/10/2018 20:54, Alban Gruin wrote: complete_action() used functions that read the todo-list file, made some changes to it, and wrote it back to the disk. The previous commits were dedicated to separate the part that deals with the file from the actual logic of these functions. Now that

[PATCH 08/15] sequencer: change complete_action() to use the refactored functions

2018-10-07 Thread Alban Gruin
complete_action() used functions that read the todo-list file, made some changes to it, and wrote it back to the disk. The previous commits were dedicated to separate the part that deals with the file from the actual logic of these functions. Now that this is done, we can call directly the