internal: phase out DecorateError in favor of errors.Wrap

The former seems to have been a thing in the old evaluator code deleted in 2020, but a single call remained in the subsume package. All that a decorated error did was augment an errors.Error value with a Go error so that errors.Is would work with either. errors.Wrap gives us the same property, and is more consistent with the rest of the codebase nowadays. Signed-off-by: Daniel Martí <[email protected]> Change-Id: I5a3acf0381c83e0e36c18708cfb586ceccc7e2f2
cue-lang · Sep 12, 2024 · cb680f2 · cb680f2
1 parent f947403
commit cb680f2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/internal/core/subsume/subsume.go b/internal/core/subsume/subsume.go
@@ -138,7 +138,7 @@ func (s *subsumer) getError() (err errors.Error) {
 	}
 	err = s.errs
 	if s.inexact {
-		err = internal.DecorateError(internal.ErrInexact, err)
+		err = errors.Wrap(err, internal.ErrInexact)
 	}
 	return err
 }
diff --git a/internal/internal.go b/internal/internal.go
@@ -430,19 +430,3 @@ func GenPath(root string) string {
 }
 
 var ErrInexact = errors.New("inexact subsumption")
-
-func DecorateError(info error, err errors.Error) errors.Error {
-	return &decorated{cueError: err, info: info}
-}
-
-type cueError = errors.Error
-
-type decorated struct {
-	cueError
-
-	info error
-}
-
-func (e *decorated) Is(err error) bool {
-	return errors.Is(e.info, err) || errors.Is(e.cueError, err)
-}