refactor: use struct instead of pointer to avoid panic

Layr-Labs · Sep 5, 2024 · 11fb347 · 11fb347
1 parent eb15e20
commit 11fb347
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/services/bls_aggregation/blsagg_test.go b/services/bls_aggregation/blsagg_test.go
@@ -1150,7 +1150,7 @@ func TestIntegrationBlsAgg(t *testing.T) {
 			avsClients.AvsRegistryChainSubscriber,
 			avsClients.AvsRegistryChainReader,
 			nil,
-			&operatorsinfo.Opts{},
+			operatorsinfo.Opts{},
 			logger,
 		)
 		avsRegistryService := avsregistry.NewAvsRegistryServiceChainCaller(

diff --git a/services/operatorsinfo/operatorsinfo_inmemory.go b/services/operatorsinfo/operatorsinfo_inmemory.go
@@ -97,7 +97,7 @@ func NewOperatorsInfoServiceInMemory(
 	avsRegistrySubscriber avsRegistrySubscriber,
 	avsRegistryReader avsRegistryReader,
 	logFilterQueryBlockRange *big.Int,
-	opts *Opts,
+	opts Opts,
 	logger logging.Logger,
 ) *OperatorsInfoServiceInMemory {
 	queryC := make(chan query)
@@ -127,7 +127,7 @@ func (ops *OperatorsInfoServiceInMemory) startServiceInGoroutine(
 	ctx context.Context,
 	queryC <-chan query,
 	wg *sync.WaitGroup,
-	opts *Opts,
+	opts Opts,
 ) {
 	go func() {
 
@@ -284,7 +284,7 @@ func (ops *OperatorsInfoServiceInMemory) startServiceInGoroutine(
 
 func (ops *OperatorsInfoServiceInMemory) queryPastRegisteredOperatorEventsAndFillDb(
 	ctx context.Context,
-	opts *Opts,
+	opts Opts,
 ) error {
 	// Querying with nil startBlock and stopBlock will return all events. It doesn't matter if we query some events that
 	// we will receive again in the websocket,
@@ -338,7 +338,7 @@ func (ops *OperatorsInfoServiceInMemory) queryPastRegisteredOperatorEventsAndFil
 		// we print each socket info on a separate line because slog for some reason doesn't pass map keys via their
 		// LogValue() function, so operatorId (of custom type Bytes32) prints as a byte array instead of its hex
 		// representation from LogValue()
-		// passing the Bytes32 directly to an slog log statements does call LogValue() and prints the hex representation
+		// passing the Bytes32 directly to a slog log statements does call LogValue() and prints the hex representation
 		ops.logger.Debug(
 			"operator socket returned from registration events query",
 			"operatorId",

diff --git a/services/operatorsinfo/operatorsinfo_inmemory_test.go b/services/operatorsinfo/operatorsinfo_inmemory_test.go
@@ -153,7 +153,7 @@ func TestGetOperatorInfo(t *testing.T) {
 				mockAvsRegistrySubscriber,
 				mockAvsReader,
 				nil,
-				&Opts{},
+				Opts{},
 				logger,
 			)
 			time.Sleep(