Message ID | 20240116-const-v1-11-17c87978f40b@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Constify all the things | expand |
Context | Check | Description |
---|---|---|
tedd_an/pre-ci_am | success | Success |
tedd_an/CheckPatch | warning | WARNING:LONG_LINE: line length of 90 exceeds 80 columns #102: FILE: gobex/gobex.c:1446: + gobex_debug = g_parse_debug_string(env, keys, G_N_ELEMENTS(keys)); /github/workspace/src/src/13520860.patch total: 0 errors, 1 warnings, 32 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /github/workspace/src/src/13520860.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. |
tedd_an/GitLint | success | Gitlint PASS |
tedd_an/IncrementalBuild | success | Incremental Build PASS |
diff --git a/gobex/gobex.c b/gobex/gobex.c index e9b89cead..8924e059f 100644 --- a/gobex/gobex.c +++ b/gobex/gobex.c @@ -122,7 +122,7 @@ struct setpath_data { guint8 constants; } __attribute__ ((packed)); -static struct error_code { +static const struct error_code { guint8 code; const char *name; } obex_errors[] = { @@ -169,7 +169,7 @@ static struct error_code { const char *g_obex_strerror(guint8 err_code) { - struct error_code *error; + const struct error_code *error; for (error = obex_errors; error->name != NULL; error++) { if (error->code == err_code) @@ -1423,7 +1423,7 @@ failed: return FALSE; } -static GDebugKey keys[] = { +static const GDebugKey keys[] = { { "error", G_OBEX_DEBUG_ERROR }, { "command", G_OBEX_DEBUG_COMMAND }, { "transfer", G_OBEX_DEBUG_TRANSFER }, @@ -1443,7 +1443,7 @@ GObex *g_obex_new(GIOChannel *io, GObexTransportType transport_type, const char *env = g_getenv("GOBEX_DEBUG"); if (env) { - gobex_debug = g_parse_debug_string(env, keys, 7); + gobex_debug = g_parse_debug_string(env, keys, G_N_ELEMENTS(keys)); g_setenv("G_MESSAGES_DEBUG", "gobex", FALSE); } else gobex_debug = G_OBEX_DEBUG_NONE;
From: Emil Velikov <emil.velikov@collabora.com> --- gobex/gobex.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)