Skip to content

Commit 29caf97

Browse files
chu11mergify[bot]
authored andcommitted
job-list: move include from .h to .c file
Problem: Headers shouldn't be included in header files unless they are needed for definitions. Solution: Move include for job-list.h from list.h to list.c.
1 parent 8ce4939 commit 29caf97

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

src/modules/job-list/list.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
#include "src/common/libutil/errno_safe.h"
2121
#include "src/common/libczmqcontainers/czmq_containers.h"
2222

23+
#include "job-list.h"
2324
#include "idsync.h"
2425
#include "list.h"
2526
#include "job_util.h"

src/modules/job-list/list.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@
1313

1414
#include <flux/core.h>
1515

16-
#include "job-list.h"
17-
1816
void list_cb (flux_t *h, flux_msg_handler_t *mh,
1917
const flux_msg_t *msg, void *arg);
2018

0 commit comments

Comments
 (0)