tree-wide: use !strv_isempty() instead of strv_length() > 0
This commit is contained in:
parent
bf0e0a4df2
commit
586290017d
|
@ -4162,19 +4162,19 @@ void exec_context_dump(ExecContext *c, FILE* f, const char *prefix) {
|
|||
if (c->pam_name)
|
||||
fprintf(f, "%sPAMName: %s\n", prefix, c->pam_name);
|
||||
|
||||
if (strv_length(c->read_write_paths) > 0) {
|
||||
if (!strv_isempty(c->read_write_paths)) {
|
||||
fprintf(f, "%sReadWritePaths:", prefix);
|
||||
strv_fprintf(f, c->read_write_paths);
|
||||
fputs("\n", f);
|
||||
}
|
||||
|
||||
if (strv_length(c->read_only_paths) > 0) {
|
||||
if (!strv_isempty(c->read_only_paths)) {
|
||||
fprintf(f, "%sReadOnlyPaths:", prefix);
|
||||
strv_fprintf(f, c->read_only_paths);
|
||||
fputs("\n", f);
|
||||
}
|
||||
|
||||
if (strv_length(c->inaccessible_paths) > 0) {
|
||||
if (!strv_isempty(c->inaccessible_paths)) {
|
||||
fprintf(f, "%sInaccessiblePaths:", prefix);
|
||||
strv_fprintf(f, c->inaccessible_paths);
|
||||
fputs("\n", f);
|
||||
|
|
|
@ -179,7 +179,7 @@ int main(int argc, char* argv[]) {
|
|||
goto finish;
|
||||
}
|
||||
|
||||
if (strv_length(final_env) <= 0) {
|
||||
if (strv_isempty(final_env)) {
|
||||
r = 0;
|
||||
goto finish;
|
||||
}
|
||||
|
|
|
@ -656,7 +656,7 @@ int ask_password_agent(
|
|||
goto finish;
|
||||
}
|
||||
|
||||
if (strv_length(l) <= 0) {
|
||||
if (strv_isempty(l)) {
|
||||
l = strv_free(l);
|
||||
log_debug("Invalid packet");
|
||||
continue;
|
||||
|
|
|
@ -1269,7 +1269,7 @@ int bus_append_unit_property_assignment(sd_bus_message *m, const char *assignmen
|
|||
if (r < 0)
|
||||
return r;
|
||||
|
||||
if (strv_length(l) > 0) {
|
||||
if (!strv_isempty(l)) {
|
||||
|
||||
r = sd_bus_message_open_container(m, 'r', "sasb");
|
||||
if (r < 0)
|
||||
|
|
Loading…
Reference in a new issue