swarm/storage/mru: cosmetic changes as per comments in ethereum#784

swarm/storage/mru/lookup.go

@@ -26,7 +26,6 @@ import (
 // LookupParams is used to specify constraints when performing an update lookup
 // Limit defines whether or not the lookup should be limited
 // If Limit is set to true then Max defines the amount of hops that can be performed
-// \TODO this is redundant, just use uint32 with 0 for unlimited hops
 type LookupParams struct {
 	UpdateLookup
 	Limit uint32

swarm/storage/mru/metadata.go

@@ -29,7 +29,7 @@ import (
 type ResourceMetadata struct {
 	StartTime Timestamp      // time at which the resource starts to be valid
 	Frequency uint64         // expected update frequency for the resource
-	Name      string         // name of the resource, for the reference of the user
+	Name      string         // name of the resource, for the reference of the user or to disambiguate resources with same starttime, frequency, owneraddr
 	Owner     common.Address // public address of the resource owner
 }
 

swarm/storage/mru/request.go

@@ -62,10 +62,6 @@ func NewCreateUpdateRequest(metadata *ResourceMetadata) (*Request, error) {
 
 	request.version = 1
 	request.period, err = getNextPeriod(metadata.StartTime.Time, now, metadata.Frequency)
-	if err != nil {
-		return nil, err
-	}
-
 	if err != nil {
 		return nil, err
 	}

swarm/storage/mru/resource.go

@@ -31,8 +31,7 @@ const (
 	defaultRetrieveTimeout = 100 * time.Millisecond
 )
 
-// resource caches resource data. When synced it contains the most recent
-// version of the resource data and the metadata of its root chunk.
+// resource caches resource data and the metadata of its root chunk.
 type resource struct {
 	resourceUpdate
 	ResourceMetadata

swarm/swarm.go

@@ -192,10 +192,7 @@ func NewSwarm(config *api.Config, mockStore *mock.NodeStore) (self *Swarm, err e
 	self.fileStore = storage.NewFileStore(netStore, self.config.FileStoreParams)
 
 	var resourceHandler *mru.Handler
-	rhparams := &mru.HandlerParams{
-		// TODO: config parameter to set limits
-
-	}
+	rhparams := &mru.HandlerParams{}
 
 	resourceHandler, err = mru.NewHandler(rhparams)
 	if err != nil {