all: comply with linter recommendations #2

HTTP request now implicitly create request with context.

Any false positive related to not closing HTTP response body has been
annotated with "nolint:bodyclose".

In the example code, use consistent "// Output:" comment format, by
prefixing with single space.
Any comment on code now also prefixing with single space.

An error returned without variables now use [errors.New] instead of
[fmt.Errorf].
Any error returned using [fmt.Errorf] now wrapped using "%w" instead of
"%s".
Also, replace error checking using [errors.Is] or [errors.As], instead
of using equal/not-equal operator.

Any statement like "x = x OP y" now replaced with "x OP= y".
Also, swap statement is simplified using "x, y = y, x".
Any switch statement with single case now replaced with if-condition.

Any call to defer on function or program that call [os.Exit], now
replaced by calling the deferred function directly.

Any if-else condition now replaced with switch statement, if possible.

1 files changed, 4 insertions, 3 deletions

diff --git a/lib/os/exec/exec.go b/lib/os/exec/exec.go
index c67640b3..b7c44c5c 100644
--- a/lib/os/exec/exec.go
+++ b/lib/os/exec/exec.go
@@ -27,7 +27,8 @@ func ParseCommandArgs(in string) (cmd string, args []string) {
 
 	for _, r := range in {
 		if quote > 0 {
-			if r == quote {
+			switch r {
+			case quote:
 				if prev == '\\' {
 					sb.WriteRune(r)
 					prev = r
@@ -39,14 +40,14 @@ func ParseCommandArgs(in string) (cmd string, args []string) {
 					sb.Reset()
 					quote = 0
 				}
-			} else if r == '\\' {
+			case '\\':
 				if prev == '\\' {
 					sb.WriteRune(r)
 					prev = 0
 				} else {
 					prev = r
 				}
-			} else {
+			default:
 				if prev == '\\' {
 					sb.WriteRune('\\')
 				}