On Tue, Feb 08, 2022 at 09:34:56AM -0500, Emanuele Giuseppe Esposito wrote: > job_event_* functions can all be static, as they are not used > outside job.c. > > Same applies for job_txn_add_job(). > > Signed-off-by: Emanuele Giuseppe Esposito <eespo...@redhat.com> > --- > include/qemu/job.h | 18 ------------------ > job.c | 12 +++++++++--- > 2 files changed, 9 insertions(+), 21 deletions(-) > > diff --git a/include/qemu/job.h b/include/qemu/job.h > index 86ec46c09e..6000463126 100644 > --- a/include/qemu/job.h > +++ b/include/qemu/job.h > @@ -356,18 +356,6 @@ JobTxn *job_txn_new(void); > */ > void job_txn_unref(JobTxn *txn); > > -/** > - * @txn: The transaction (may be NULL) > - * @job: Job to add to the transaction > - * > - * Add @job to the transaction. The @job must not already be in a > transaction. > - * The caller must call either job_txn_unref() or job_completed() to release > - * the reference that is automatically grabbed here. > - * > - * If @txn is NULL, the function does nothing. > - */ > -void job_txn_add_job(JobTxn *txn, Job *job); ... > diff --git a/job.c b/job.c > index d603f9ad1e..f13939d3c6 100644 > --- a/job.c > +++ b/job.c > @@ -125,7 +125,7 @@ void job_txn_unref(JobTxn *txn) > } > } > > -void job_txn_add_job(JobTxn *txn, Job *job) > +static void job_txn_add_job(JobTxn *txn, Job *job)
Please move the doc comments into the .c file too.
signature.asc
Description: PGP signature