style: remove redundant logs by returning msg in errors (#776)

encoding/kzg/pointsIO.go

@@ -100,8 +100,7 @@ func ReadG2PointOnPowerOf2(exponent uint64, g *KzgConfig) (bn254.G2Affine, error
 func ReadG1Points(filepath string, n uint64, numWorker uint64) ([]bn254.G1Affine, error) {
 	g1f, err := os.Open(filepath)
 	if err != nil {
-		log.Println("Cannot ReadG1Points", filepath, err)
-		return nil, fmt.Errorf("error cannot open g1 points file %w", err)
+		return nil, fmt.Errorf("error cannot open g1 points file %s: %w", filepath, err)
 	}
 
 	defer func() {
@@ -170,8 +169,7 @@ func ReadG1PointSection(filepath string, from, to uint64, numWorker uint64) ([]b
 	}
 	g1f, err := os.Open(filepath)
 	if err != nil {
-		log.Println("ReadG1PointSection.ERR.0", err)
-		return nil, err
+		return nil, fmt.Errorf("error cannot open g1 points file %w", err)
 	}
 
 	defer func() {

encoding/kzg/verifier/verifier.go

@@ -41,8 +41,7 @@ func NewVerifier(config *kzg.KzgConfig, loadG2Points bool) (*Verifier, error) {
 	// read the whole order, and treat it as entire SRS for low degree proof
 	s1, err := kzg.ReadG1Points(config.G1Path, config.SRSNumberToLoad, config.NumWorker)
 	if err != nil {
-		log.Println("failed to read G1 points", err)
-		return nil, err
+		return nil, fmt.Errorf("failed to read %d G1 points from %s: %v", config.SRSNumberToLoad, config.G1Path, err)
 	}
 
 	s2 := make([]bn254.G2Affine, 0)
@@ -56,8 +55,7 @@ func NewVerifier(config *kzg.KzgConfig, loadG2Points bool) (*Verifier, error) {
 
 		s2, err = kzg.ReadG2Points(config.G2Path, config.SRSNumberToLoad, config.NumWorker)
 		if err != nil {
-			log.Println("failed to read G2 points", err)
-			return nil, err
+			return nil, fmt.Errorf("failed to read %d G2 points from %s: %v", config.SRSNumberToLoad, config.G2Path, err)
 		}
 
 		g2Trailing, err = kzg.ReadG2PointSection(
@@ -67,7 +65,7 @@ func NewVerifier(config *kzg.KzgConfig, loadG2Points bool) (*Verifier, error) {
 			config.NumWorker,
 		)
 		if err != nil {
-			return nil, err
+			return nil, fmt.Errorf("failed to read trailing G2 points from %s: %v", config.G2Path, err)
 		}
 	} else {
 		if len(config.G2PowerOf2Path) == 0 && len(config.G2Path) == 0 {
@@ -90,8 +88,7 @@ func NewVerifier(config *kzg.KzgConfig, loadG2Points bool) (*Verifier, error) {
 	}
 	srs, err := kzg.NewSrs(s1, s2)
 	if err != nil {
-		log.Println("Could not create srs", err)
-		return nil, err
+		return nil, fmt.Errorf("failed to create SRS: %v", err)
 	}
 
 	fmt.Println("numthread", runtime.GOMAXPROCS(0))