diff options
author | NeilBrown <neilb@suse.de> | 2015-02-12 03:46:53 +0100 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2015-02-12 03:46:53 +0100 |
commit | 7a862a020f04b10f17f6dcae23bf60c52cbe190b (patch) | |
tree | 8134a9a06b1aaaa1ee640b20f3b00e95755d4f49 /policy.c | |
parent | Consistently print program Name and __func__ in debug messages. (diff) | |
download | mdadm-7a862a020f04b10f17f6dcae23bf60c52cbe190b.tar.xz mdadm-7a862a020f04b10f17f6dcae23bf60c52cbe190b.zip |
Don't break long strings onto multiple lines.
It is best to keep strings all together so that they
are easier to search for in the source code.
If a string is so long that it looks ugly one line,
them maybe it should be broken into multiple lines
for display too.
Only strings which contain a newline can be broken
into multiple lines:
"It is OK to\n"
"break this string\n"
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'policy.c')
-rw-r--r-- | policy.c | 9 |
1 files changed, 3 insertions, 6 deletions
@@ -727,16 +727,14 @@ void policy_save_path(char *id_path, struct map_ent *array) FILE *f = NULL; if (mkdir(FAILED_SLOTS_DIR, S_IRWXU) < 0 && errno != EEXIST) { - pr_err("can't create file to save path " - "to old disk: %s\n", strerror(errno)); + pr_err("can't create file to save path to old disk: %s\n", strerror(errno)); return; } snprintf(path, PATH_MAX, FAILED_SLOTS_DIR "/%s", id_path); f = fopen(path, "w"); if (!f) { - pr_err("can't create file to" - " save path to old disk: %s\n", + pr_err("can't create file to save path to old disk: %s\n", strerror(errno)); return; } @@ -745,8 +743,7 @@ void policy_save_path(char *id_path, struct map_ent *array) array->metadata, array->uuid[0], array->uuid[1], array->uuid[2], array->uuid[3]) <= 0) - pr_err("Failed to write to " - "<id_path> cookie\n"); + pr_err("Failed to write to <id_path> cookie\n"); fclose(f); } |