diff options
author | Joseph Hunkeler <jhunkeler@gmail.com> | 2020-02-10 10:23:27 -0500 |
---|---|---|
committer | Joseph Hunkeler <jhunkeler@gmail.com> | 2020-02-10 10:23:27 -0500 |
commit | cb14cfde8b8154c5aa7a795d0ea846620030dd79 (patch) | |
tree | 9a541a916e2b9dd2ae4bce4559cdfce72ed5c611 /src/version_spec.c | |
parent | d32854a92c760c8b520938f91e2e127307293c7d (diff) | |
download | spmc-cb14cfde8b8154c5aa7a795d0ea846620030dd79.tar.gz |
Add a few comments
Diffstat (limited to 'src/version_spec.c')
-rw-r--r-- | src/version_spec.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/src/version_spec.c b/src/version_spec.c index 511cc73..9810869 100644 --- a/src/version_spec.c +++ b/src/version_spec.c @@ -359,10 +359,12 @@ ManifestPackage *find_by_strspec(Manifest *manifest, const char *_strspec) { } ManifestPackage **m = NULL; + // No operators found if (pos == NULL) { m = find_by_spec(manifest, name, ">=", NULL); } + // When `m` is still NULL after applying the default operator if (m == NULL) { for (size_t i = 0; *(pos + i) != '\0'; i++) { version[i] = *(pos + i); @@ -370,6 +372,7 @@ ManifestPackage *find_by_strspec(Manifest *manifest, const char *_strspec) { m = find_by_spec(manifest, name, op, version); } + // When `m` has been populated by either test above, return a COPY of the manifest if (m != NULL) { ManifestPackage *result = manifest_package_copy(m[0]); for (size_t i = 0; m[i] != NULL; i++) { @@ -378,5 +381,7 @@ ManifestPackage *find_by_strspec(Manifest *manifest, const char *_strspec) { free(m); return result; } + + // Obviously it didn't work out return NULL; } |