Merge pull request #484 from tylitianrui/feature/code_optimize

code optimization

Author: binbin0325

core/hotspot/cache/lru.go

@@ -129,19 +129,20 @@ func (c *LRU) Contains(key interface{}) (ok bool) {
 func (c *LRU) Peek(key interface{}) (value interface{}, isFound bool) {
 	var ent *list.Element
 	if ent, isFound = c.items[key]; isFound {
-		return ent.Value.(*entry).value, true
+		return ent.Value.(*entry).value, isFound
 	}
 	return nil, isFound
 }
 
 // Remove removes the provided key from the cache, returning if the
 // key was contained.
 func (c *LRU) Remove(key interface{}) (isFound bool) {
-	if ent, ok := c.items[key]; ok {
+	var ent *list.Element
+	if ent, isFound = c.items[key]; isFound {
 		c.removeElement(ent)
-		return true
+		return
 	}
-	return false
+	return
 }
 
 // RemoveOldest removes the oldest item from the cache.

core/hotspot/traffic_shaping.go

@@ -115,9 +115,8 @@ func (c *baseTrafficShapingController) BoundMetric() *ParamsMetric {
 
 func (c *baseTrafficShapingController) performCheckingForConcurrencyMetric(arg interface{}) *base.TokenResult {
 	specificItem := c.specificItems
-	initConcurrency := new(int64)
-	*initConcurrency = 0
-	concurrencyPtr := c.metric.ConcurrencyCounter.AddIfAbsent(arg, initConcurrency)
+	initConcurrency := int64(0)
+	concurrencyPtr := c.metric.ConcurrencyCounter.AddIfAbsent(arg, &initConcurrency)
 	if concurrencyPtr == nil {
 		// First to access this arg
 		return nil