diff mbox series

[v4,1/6] run-command.h: remove always unused "clean_on_exit_handler_cbdata"

Message ID patch-v4-1.6-fc55b203474-20220331T014349Z-avarab@gmail.com (mailing list archive)
State Accepted
Commit 551f5022019803cceb0fa3943f696fae32f093f9
Headers show
Series various: remove dead code | expand

Commit Message

Ævar Arnfjörð Bjarmason March 31, 2022, 1:45 a.m. UTC 
Remove a "struct child_process" member added in
ac2fbaa674c (run-command: add clean_on_exit_handler, 2016-10-16), but
which was never used.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 run-command.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Junio C Hamano March 31, 2022, 10:03 p.m. UTC | #1 
Ævar Arnfjörð Bjarmason  <avarab@gmail.com> writes:

> Remove a "struct child_process" member added in
> ac2fbaa674c (run-command: add clean_on_exit_handler, 2016-10-16), but
> which was never used.
>
> Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
> ---
>  run-command.h | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/run-command.h b/run-command.h
> index 07bed6c31b4..5bd0c933e80 100644
> --- a/run-command.h
> +++ b/run-command.h
> @@ -142,7 +142,6 @@ struct child_process {
>  	unsigned clean_on_exit:1;
>  	unsigned wait_after_clean:1;
>  	void (*clean_on_exit_handler)(struct child_process *process);
> -	void *clean_on_exit_handler_cbdata;
>  };
>  
>  #define CHILD_PROCESS_INIT { \

Thanks for digging.  Looks good.
diff mbox series

Patch

diff --git a/run-command.h b/run-command.h
index 07bed6c31b4..5bd0c933e80 100644
--- a/run-command.h
+++ b/run-command.h
@@ -142,7 +142,6 @@  struct child_process {
 	unsigned clean_on_exit:1;
 	unsigned wait_after_clean:1;
 	void (*clean_on_exit_handler)(struct child_process *process);
-	void *clean_on_exit_handler_cbdata;
 };
 
 #define CHILD_PROCESS_INIT { \