switch to 1MB buffer and fix panic mentioned in #65

sselph · Feb 3, 2016 · 9498e60 · 9498e60
1 parent 5d34d1b
commit 9498e60
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/ds/hasher.go b/ds/hasher.go
@@ -80,7 +80,7 @@ func NewHasher(hashFunc func() hash.Hash, threads int) (*Hasher, error) {
 	l := make(map[string]*sync.Mutex)
 	b := make(chan []byte, threads)
 	for i := 0; i < threads; i++ {
-		b <- make([]byte, 4*1024*1024)
+		b <- make([]byte, 1*1024*1024)
 	}
 	return &Hasher{h: hashFunc, c: c, cl: &sync.Mutex{}, l: l, b: b}, nil
 }
diff --git a/rom/rom.go b/rom/rom.go
@@ -231,7 +231,7 @@ Loop:
 		game.Overview = strings.Map(stripChars, game.Overview)
 		game.GameTitle = strings.Map(stripChars, game.GameTitle)
 	}
-	if opts.OverviewLen != 0 && opts.OverviewLen > 0 && len(game.Overview) > opts.OverviewLen + 3 {
+	if opts.OverviewLen != 0 && opts.OverviewLen > 0 && len(game.Overview) > opts.OverviewLen+3 {
 		game.Overview = game.Overview[:opts.OverviewLen] + "..."
 	}
 	r.Game = game

diff --git a/scraper.go b/scraper.go
@@ -522,11 +522,11 @@ func main() {
 	}
 	if *mame || *scrapeAll {
 		mds, err := ds.NewMAME("")
-		defer mds.Close()
 		if err != nil {
 			fmt.Println(err)
 			return
 		}
+		defer mds.Close()
 		arcadeSources = append(arcadeSources, mds)
 	}
 	var hasher *ds.Hasher