update some comments

Author: inhere

context_render.go

@@ -29,7 +29,8 @@ func (c *Context) Respond(status int, obj interface{}, renderer render.Renderer)
 
 	err := renderer.Render(c.Resp, obj)
 	if err != nil {
-		panic(err) // TODO or use AddError()
+		// panic(err) // TODO or use AddError()
+		c.AddError(err)
 	}
 }
 
@@ -96,7 +97,9 @@ func (c *Context) Stream(status int, contentType string, r io.Reader) {
 	_, err := io.Copy(c.Resp, r)
 
 	if err != nil {
-		panic(err)
+		// TODO use AddError()
+		// panic(err)
+		c.AddError(err)
 	}
 }
 

extends.go

@@ -26,7 +26,9 @@ type Validator interface {
  * Context function extends()
  *************************************************************/
 
-// Render context template
+// Render context template.
+//
+// please use ShouldRender() instead
 func (c *Context) Render(status int, name string, data interface{}) (err error) {
 	if c.router.Renderer == nil {
 		return errors.New("renderer not registered")
@@ -43,7 +45,7 @@ func (c *Context) Render(status int, name string, data interface{}) (err error)
 
 // Validate context validator
 // Deprecated
-// please use ShouldBind(), it will auto call validator.
+// please use ShouldBind() instead, it will auto call validator.
 func (c *Context) Validate(i interface{}) error {
 	if c.Router().Validator == nil {
 		return errors.New("validator not registered")

render/json.go

@@ -29,11 +29,7 @@ func (r JSONRenderer) Render(w http.ResponseWriter, obj interface{}) (err error)
 		enc.SetEscapeHTML(false)
 	}
 
-	if err = enc.Encode(obj); err != nil {
-		return err
-	}
-
-	return err
+	return enc.Encode(obj)
 }
 
 // JSON response rendering

router.go

@@ -96,8 +96,10 @@ type Router struct {
 	// Extends tools
 	//
 	// Renderer template(view) interface
+	// Deprecated
 	Renderer Renderer
 	// Validator validator interface
+	// Deprecated
 	Validator Validator
 }