Added global logger

astits/main.go

@@ -40,6 +40,9 @@ func main() {
 	cmd := astikit.FlagCmd()
 	flag.Parse()
 
+	// Set logger
+	astits.SetLogger(log.New(log.Writer(), log.Prefix(), log.Flags()))
+
 	// Handle signals
 	handleSignals()
 

data_psi.go

@@ -2,7 +2,6 @@ package astits
 
 import (
 	"fmt"
-	"log"
 
 	"github.com/asticode/go-astikit"
 )
@@ -293,7 +292,7 @@ func psiTableType(tableID int) string {
 	case tableID == 0x73:
 		return PSITableTypeTOT
 	default:
-		log.Printf("astits: unlisted PSI table ID %d\n", tableID)
+		logger.Debugf("astits: unlisted PSI table ID %d\n", tableID)
 	}
 	return PSITableTypeUnknown
 }

descriptor.go

@@ -2,7 +2,6 @@ package astits
 
 import (
 	"fmt"
-	"log"
 	"time"
 
 	"github.com/asticode/go-astikit"
@@ -634,7 +633,7 @@ func newDescriptorExtension(i *astikit.BytesIterator, offsetEnd int) (d *Descrip
 			return
 		}
 	default:
-		log.Printf("astits: unlisted extension tag 0x%x\n", d.Tag)
+		logger.Debugf("astits: unlisted extension tag 0x%x\n", d.Tag)
 	}
 	return
 }
@@ -1395,7 +1394,7 @@ func parseDescriptors(i *astikit.BytesIterator) (o []*Descriptor, err error) {
 							return
 						}
 					default:
-						log.Printf("astits: unlisted descriptor tag 0x%x\n", d.Tag)
+						logger.Debugf("astits: unlisted descriptor tag 0x%x\n", d.Tag)
 					}
 				}
 

logger.go

@@ -0,0 +1,10 @@
+package astits
+
+import "github.com/asticode/go-astikit"
+
+// Right now we use a global logger because it feels weird to inject a logger in pure functions
+// Indeed, logger is only needed to let the developer know when an unhandled descriptor or id has been found
+// in the stream
+var logger = astikit.AdaptStdLogger(nil)
+
+func SetLogger(l astikit.StdLogger) { logger = astikit.AdaptStdLogger(l) }