Fixed tautological error conditions
authorChristian Muehlhaeuser <muesli@gmail.com>
Fri, 2 Aug 2019 15:20:36 +0000 (15:20 +0000)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Fri, 9 Aug 2019 23:04:43 +0000 (01:04 +0200)
Drop error & nil checks where the value can not have changed.

common/herrors/error_locator.go
deploy/deploy.go
resources/smartcrop.go

index 15de6d318f1b379a3cae81ed51094cdee3ce594a..3778a3729bdc0d3ed13b5bb9848c7e299b087061 100644 (file)
@@ -206,7 +206,7 @@ func locateError(r io.Reader, le FileError, matches LineMatcherFn) ErrorContext
 
        lines := strings.Split(string(b), "\n")
 
-       if le != nil && lepos.ColumnNumber >= 0 {
+       if lepos.ColumnNumber >= 0 {
                pos.ColumnNumber = lepos.ColumnNumber
        }
 
index 0512e352e95bbc03cb00f8ed52807f3cd89f55c0..6abb9e24e5c695f81d455b04ce22cbbc6eca48d8 100644 (file)
@@ -143,9 +143,6 @@ func (d *Deployer) Deploy(ctx context.Context) error {
 
        // Diff local vs remote to see what changes need to be applied.
        uploads, deletes := findDiffs(local, remote, d.force)
-       if err != nil {
-               return err
-       }
        d.summary.NumUploads = len(uploads)
        d.summary.NumDeletes = len(deletes)
        if len(uploads)+len(deletes) == 0 {
index 05bc55cd727bba07d9ea6beb9ba80e748904092c..d28a8dd03878245657ea7e302b614d5b4ec3485c 100644 (file)
@@ -70,10 +70,7 @@ func smartCrop(img image.Image, width, height int, anchor imaging.Anchor, filter
 
        b := img.Bounds().Intersect(rect)
 
-       cropped, err := imaging.Crop(img, b), nil
-       if err != nil {
-               return nil, err
-       }
+       cropped := imaging.Crop(img, b)
 
        return imaging.Resize(cropped, width, height, filter), nil