Remove useless check

[goredo.git] / dot.go

diff --git a/dot.go b/dot.go
index d1433b593444b8610e7335a5ff12944a34e482da..9199f7dd0e43099c010e972911831fdf8fd715bf 100644 (file)
--- a/dot.go
+++ b/dot.go
@@ -1,6 +1,6 @@
 /*
 goredo -- djb's redo implementation on pure Go
-Copyright (C) 2020-2022 Sergey Matveev <stargrave@stargrave.org>
+Copyright (C) 2020-2023 Sergey Matveev <stargrave@stargrave.org>
 
 This program is free software: you can redistribute it and/or modify
 it under the terms of the GNU General Public License as published by
@@ -34,15 +34,13 @@ type DotNodes struct {
        to   string
 }
 
-func dotWalker(data map[DotNodes]bool, tgtOrig string) (map[DotNodes]bool, error) {
-       cwd, tgt := cwdAndTgt(tgtOrig)
-       depPath := path.Join(cwd, RedoDir, tgt+DepSuffix)
-       fdDep, err := os.Open(depPath)
+func dotWalker(data map[DotNodes]bool, tgt *Tgt) (map[DotNodes]bool, error) {
+       fdDep, err := os.Open(tgt.Dep())
        if err != nil {
-               return nil, err
+               return nil, ErrLine(err)
        }
        defer fdDep.Close()
-       var dep string
+       var dep *Tgt
        r := recfile.NewReader(fdDep)
        for {
                m, err := r.NextMap()
@@ -50,18 +48,21 @@ func dotWalker(data map[DotNodes]bool, tgtOrig string) (map[DotNodes]bool, error
                        if errors.Is(err, io.EOF) {
                                break
                        }
-                       return nil, err
+                       return nil, ErrLine(err)
                }
                switch m["Type"] {
                case DepTypeIfcreate:
-                       data[DotNodes{tgtOrig, cwdMustRel(cwd, m["Target"])}] = true
+                       data[DotNodes{tgt.String(), NewTgt(m["Target"]).String()}] = true
                case DepTypeIfchange:
-                       dep = m["Target"]
-                       data[DotNodes{tgtOrig, cwdMustRel(cwd, dep)}] = false
-                       if isSrc(cwd, dep) || dep == tgt {
+                       dep = NewTgt(path.Join(tgt.h, m["Target"]))
+                       if dep.a == tgt.a {
                                continue
                        }
-                       data, err = dotWalker(data, cwdMustRel(cwd, dep))
+                       data[DotNodes{tgt.String(), dep.String()}] = false
+                       if isSrc(dep) {
+                               continue
+                       }
+                       data, err = dotWalker(data, dep)
                        if err != nil {
                                return nil, err
                        }
@@ -70,7 +71,7 @@ func dotWalker(data map[DotNodes]bool, tgtOrig string) (map[DotNodes]bool, error
        return data, nil
 }
 
-func dotPrint(tgts []string) error {
+func dotPrint(tgts []*Tgt) error {
        data := map[DotNodes]bool{}
        var err error
        for _, tgt := range tgts {