Browse Source

Removed redundant returns

gbbr 11 years ago
parent
commit
0b3b574035
1 changed files with 4 additions and 8 deletions
  1. 4 8
      server.go

+ 4 - 8
server.go

@@ -955,25 +955,21 @@ func (sc *serverConn) processSetting(s Setting) error {
 	case SettingHeaderTableSize:
 	case SettingHeaderTableSize:
 		sc.headerTableSize = s.Val
 		sc.headerTableSize = s.Val
 		sc.hpackEncoder.SetMaxDynamicTableSize(s.Val)
 		sc.hpackEncoder.SetMaxDynamicTableSize(s.Val)
-		return nil
 	case SettingEnablePush:
 	case SettingEnablePush:
 		sc.pushEnabled = s.Val != 0
 		sc.pushEnabled = s.Val != 0
-		return nil
 	case SettingMaxConcurrentStreams:
 	case SettingMaxConcurrentStreams:
 		sc.maxConcurrentStreams = int64(s.Val)
 		sc.maxConcurrentStreams = int64(s.Val)
-		return nil
 	case SettingInitialWindowSize:
 	case SettingInitialWindowSize:
 		return sc.processSettingInitialWindowSize(s.Val)
 		return sc.processSettingInitialWindowSize(s.Val)
 	case SettingMaxFrameSize:
 	case SettingMaxFrameSize:
 		sc.maxWriteFrameSize = s.Val
 		sc.maxWriteFrameSize = s.Val
-		return nil
 	case SettingMaxHeaderListSize:
 	case SettingMaxHeaderListSize:
 		sc.maxHeaderListSize = s.Val
 		sc.maxHeaderListSize = s.Val
-		return nil
+	default:
+		// Unknown setting: "An endpoint that receives a SETTINGS
+		// frame with any unknown or unsupported identifier MUST
+		// ignore that setting."
 	}
 	}
-	// Unknown setting: "An endpoint that receives a SETTINGS
-	// frame with any unknown or unsupported identifier MUST
-	// ignore that setting."
 	return nil
 	return nil
 }
 }