summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWerner Koch <wk@gnupg.org>2023-03-27 16:56:29 +0200
committerWerner Koch <wk@gnupg.org>2023-03-27 16:56:29 +0200
commit34cc7daba12a2a6baa3cfa337762656c210686fa (patch)
tree3312c41cf3fe9202b294392b77c9f148647124c9
parentb608c084b9220d8ed288eb916e88a236abac0707 (diff)
downloadgpgme-34cc7daba12a2a6baa3cfa337762656c210686fa.tar.gz
core: Minor code cleanup.
* src/engine-gpg.c (build_argv): Remove superfluous tes. -- gpg_error_from_syserror is guaranteed not to return 0 even if errno is not set.
-rw-r--r--src/engine-gpg.c75
1 files changed, 25 insertions, 50 deletions
diff --git a/src/engine-gpg.c b/src/engine-gpg.c
index 50497987..93db6115 100644
--- a/src/engine-gpg.c
+++ b/src/engine-gpg.c
@@ -958,8 +958,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!fd_data_map)
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc = datac = 0;
@@ -967,8 +966,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
if (need_special)
@@ -977,8 +975,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -988,8 +985,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1004,8 +1000,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1014,8 +1009,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1028,8 +1022,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1037,8 +1030,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1051,8 +1043,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1060,8 +1051,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1074,8 +1064,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1083,8 +1072,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1097,8 +1085,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1106,8 +1093,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1120,8 +1106,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1129,8 +1114,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1143,8 +1127,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1152,8 +1135,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1177,8 +1159,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1186,8 +1167,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1199,8 +1179,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1222,8 +1201,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
== -1)
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
if (_gpgme_io_set_close_notify (fds[0],
close_notify_handler, gpg)
@@ -1272,8 +1250,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
ptr = argv[argc];
@@ -1298,8 +1275,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}
@@ -1307,8 +1283,7 @@ build_argv (engine_gpg_t gpg, const char *pgmname)
if (!argv[argc])
{
err = gpg_error_from_syserror ();
- if (err)
- goto leave;
+ goto leave;
}
argc++;
}