Skip to content

Unified Error Codes for libdfi #724

@PengZheng

Description

@PengZheng

Intro

Currently each component (by component we mean .h/.c pair, which is borrowed from Lakos' Large Scale C++) in libdfi has its own error code definition:

In dyn_function.c, we have:

static const int OK = 0;
static const int ERROR = 1;

In dyn_type.c, there is:

static const int OK = 0;
static const int ERROR = 1;
static const int MEM_ERROR = 2;
static const int PARSE_ERROR = 3;

Unified Error Codes

Given that celix_status_t now support a custom facility error part, I think the int return can be upgraded to a celix_status_t. And because celix_status_t is a typedef to int, this maybe is even backwards compatible.

But this would be nice as a future enhancement and port of this pull request.

Originally posted by @pnoltes in #699 (comment)

Metadata

Metadata

Assignees

Labels

component/dfiCategorizes an issue or PR relevant to the DFI implementation.kind/improvementCategorizes issue or PR as related to improvements.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions