CLEANUP/MINOR: tighten three comments carried over from v1.2.0

Same pass as the surrounding commits, applied to comments that predate this
branch: state the intent and drop the restatement of the code below.

Author: dkorunic

daemon.go

@@ -114,9 +114,8 @@ var errServiceReplaced = fmt.Errorf("service replaced or removed by reload")
 // rather than double-forking. Callers must not treat it as fatal.
 var errAlreadyRunning = fmt.Errorf("service already running")
 
-// errConditionUnmet is returned by startService when a file condition blocks
-// the start. startService already logged the skip; callers must treat it as a
-// benign non-start, never as a failure.
+// errConditionUnmet reports a start blocked by a file condition. startService
+// has already logged the skip; it is a benign non-start, not a failure.
 var errConditionUnmet = fmt.Errorf("start condition not met")
 
 // daemon holds all mutable daemon state so reload can update it.

internal/yml/config.go

@@ -568,10 +568,9 @@ func parseProcess(n *Node, env map[string]string) (service.Process, error) {
 // ({{file}}, {{.VAR}}). {{cpu}}/{{mem}} expand to integers, so excluded.
 var argSecretTemplateRe = regexp.MustCompile(`\{\{\s*(?:file\b|\.)`)
 
-// requireMapping rejects a value that cannot hold a key-value table. A scalar
-// or a list parses to an empty map, so without this the whole setting would be
-// silently ignored until a child exits. An absent key, or a bare "key:" with no
-// entries, stays legal.
+// requireMapping rejects a value that cannot hold a key-value table: a scalar
+// or list parses to an empty map, so the setting would be ignored in silence
+// until a child exits. An absent key, or a bare "key:", stays legal.
 func requireMapping(n *Node, key, procName, command string) error {
 	v := n.Get(key)
 	if v == nil || v.kind == kindMapping {

service/condition.go

@@ -25,10 +25,10 @@ import (
 // human-readable reason when a start should be skipped, or "" to proceed.
 //
 // os.Stat follows symlinks so a k8s ..data mount resolves to its target and a
-// dangling symlink counts as missing. The check is advisory: the file state
-// can change between this probe and the exec (inherent TOCTOU). A Stat error
-// other than not-exist leaves the condition unmet with the error in the
-// reason, so e.g. a permission problem never masquerades as a missing file.
+// dangling symlink counts as missing. The check is advisory: the state can
+// change between this probe and the exec. A Stat error other than not-exist
+// leaves the condition unmet and names the error, so a permission problem
+// never masquerades as a missing file.
 func (p *Process) UnmetCondition() string {
 	if p.ConditionFileExists != "" {
 		exists, err := fileExists(p.ConditionFileExists)