core: delete AditionalEventEmitters and package fuzzy (#21)

The maintenance cost outweights the use of it, especially now that we have better side-switching events.

Author: markus-wa

demoinfocs_test.go

@@ -19,7 +19,6 @@ import (
 	dem "github.com/markus-wa/demoinfocs-golang"
 	common "github.com/markus-wa/demoinfocs-golang/common"
 	events "github.com/markus-wa/demoinfocs-golang/events"
-	fuzzy "github.com/markus-wa/demoinfocs-golang/fuzzy"
 	msg "github.com/markus-wa/demoinfocs-golang/msg"
 )
 
@@ -178,50 +177,6 @@ func TestUnexpectedEndOfDemo(t *testing.T) {
 	}
 }
 
-func TestValveMatchmakingFuzzyEmitters(t *testing.T) {
-	f, err := os.Open(valveMatchmakingDemoPath)
-	if err != nil {
-		t.Fatal(err)
-	}
-	defer f.Close()
-
-	cfg := dem.DefaultParserConfig
-	cfg.AdditionalEventEmitters = []dem.EventEmitter{new(fuzzy.ValveMatchmakingTeamSwitchEmitter)}
-
-	p := dem.NewParserWithConfig(f, cfg)
-
-	teamSwitchDone := false
-	tScoreBeforeSwap, ctScoreBeforeSwap := -1, -1
-	p.RegisterEventHandler(func(ev events.RoundEnd) {
-		switch ev.Winner {
-		case common.TeamTerrorists:
-			tScoreBeforeSwap = p.GameState().TeamTerrorists().Score + 1
-
-		case common.TeamCounterTerrorists:
-			ctScoreBeforeSwap = p.GameState().TeamCounterTerrorists().Score + 1
-		}
-	})
-
-	p.RegisterEventHandler(func(fuzzy.TeamSwitchEvent) {
-		teamSwitchDone = true
-		if tScoreBeforeSwap != p.GameState().TeamCounterTerrorists().Score {
-			t.Error("T-Score before swap != CT-Score after swap")
-		}
-		if ctScoreBeforeSwap != p.GameState().TeamTerrorists().Score {
-			t.Error("CT-Score before swap != T-Score after swap")
-		}
-	})
-
-	err = p.ParseToEnd()
-	if err != nil {
-		t.Fatal("Parsing failed:", err)
-	}
-
-	if !teamSwitchDone {
-		t.Fatal("TeamSwitchEvent not received")
-	}
-}
-
 func TestCancelParseToEnd(t *testing.T) {
 	f, err := os.Open(defaultDemPath)
 	if err != nil {

fuzzy/doc.go

@@ -210,20 +210,6 @@ type ParserConfig struct {
 	// Check out demopacket.go to see which net-messages are already being parsed by default.
 	// This is a beta feature and may be changed or replaced without notice.
 	AdditionalNetMessageCreators map[int]NetMessageCreator
-
-	// AdditionalEventEmitters contains additional event emitters - either from the fuzzy package or custom ones.
-	// This is mainly used to add logic specifically for one type of demo (e.g. Matchmaking, FaceIt etc.).
-	// This is a beta feature and may be changed or replaced without notice.
-	// See also: package fuzzy for existing emitters with fuzzy-logic that depends on the demo-type.
-	AdditionalEventEmitters []EventEmitter
-}
-
-// EventEmitter is the interface to define additional event-emitters.
-// The emitters may fire additional events by calling the eventDispatcher function received during registration of the emitter.
-//
-// See also: package fuzzy for existing emitters with fuzzy-logic that depends on the demo-type.
-type EventEmitter interface {
-	Register(parser *Parser, eventDispatcher func(event interface{}))
 }
 
 // DefaultParserConfig is the default Parser configuration used by NewParser().
@@ -263,10 +249,6 @@ func NewParserWithConfig(demostream io.Reader, config ParserConfig) *Parser {
 
 	p.additionalNetMessageCreators = config.AdditionalNetMessageCreators
 
-	for _, emitter := range config.AdditionalEventEmitters {
-		emitter.Register(&p, p.eventDispatcher.Dispatch)
-	}
-
 	return &p
 }