Browse Source

Fix the new binary logic in Burndown

Vadim Markovtsev 6 năm trước cách đây
mục cha
commit
83ce4101eb
1 tập tin đã thay đổi với 4 bổ sung3 xóa
  1. 4 3
      leaves/burndown.go

+ 4 - 3
leaves/burndown.go

@@ -1080,7 +1080,7 @@ func (analyser *BurndownAnalysis) handleModification(
 	// Check for binary changes
 	blobFrom := cache[change.From.TreeEntry.Hash]
 	_, errFrom := blobFrom.CountLines()
-	blobTo := cache[change.From.TreeEntry.Hash]
+	blobTo := cache[change.To.TreeEntry.Hash]
 	_, errTo := blobTo.CountLines()
 	if errFrom != errTo {
 		if errFrom != nil {
@@ -1174,8 +1174,9 @@ func (analyser *BurndownAnalysis) handleModification(
 		pending.Text = ""
 	}
 	if file.Len() != thisDiffs.NewLinesOfCode {
-		return fmt.Errorf("%s: internal integrity error dst %d != %d",
-			change.To.Name, thisDiffs.NewLinesOfCode, file.Len())
+		return fmt.Errorf("%s: internal integrity error dst %d != %d %s -> %s",
+			change.To.Name, thisDiffs.NewLinesOfCode, file.Len(),
+			change.From.TreeEntry.Hash.String(), change.To.TreeEntry.Hash.String())
 	}
 	return nil
 }