openstack/liberasurecode
Code Issues Proposed changes

Make header files more stand-alone

Browse Source 
If a header file references erasurecode_backend.h structures, it ought
to include the header file.

Note that the already-present

   #ifndef _ERASURECODE_BACKEND_H_
   #define _ERASURECODE_BACKEND_H_
   ...
   #endif  // _ERASURECODE_BACKEND_H_

guards ensure we don't get multiple definitions.

Change-Id: I5b8b63452b5751295cf89236693f98378f949e18
This commit is contained in:
Tim Burke
2025-08-28 15:31:09 -07:00
parent edae314559
commit 7c7373b812
3 changed files with 6 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
include/erasurecode/erasurecode_postprocessing.h
View File

@@ -29,6 +29,8 @@
#ifndef _ERASURECODE_POSTPROCESSING_H_
#define _ERASURECODE_POSTPROCESSING_H_
#include "erasurecode_backend.h"
int finalize_fragments_after_encode(ec_backend_t instance,
int k, int m, int blocksize, uint64_t orig_data_size,
char **encoded_data, char **encoded_parity);

2
include/erasurecode/erasurecode_preprocessing.h
View File

@@ -29,6 +29,8 @@
#ifndef _ERASURECODE_PREPROCESSING_H_
#define _ERASURECODE_PREPROCESSING_H_
#include "erasurecode_backend.h"
int prepare_fragments_for_encode(
ec_backend_t instance,
int k, int m,

2
include/isa_l/isa_l_common.h
View File

@@ -27,6 +27,8 @@
* vi: set noai tw=79 ts=4 sw=4:
*/
#include "erasurecode_backend.h"
#define ISA_L_W 8
/* Forward declarations */