From e3dc9b269441dfc7499e7b172fc28b14d682d2b7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Hoguin?= Date: Mon, 21 Mar 2011 17:51:21 +0100 Subject: Add specs to ensure_response and change the clauses order. --- src/cowboy_http_protocol.erl | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'src/cowboy_http_protocol.erl') diff --git a/src/cowboy_http_protocol.erl b/src/cowboy_http_protocol.erl index b02f471..a6cedbe 100644 --- a/src/cowboy_http_protocol.erl +++ b/src/cowboy_http_protocol.erl @@ -182,13 +182,14 @@ skip_body(Length, Req) -> _Any -> ok end. +-spec ensure_response(Req::#http_req{}, State::#state{}) -> ok. +%% The handler has already fully replied to the client. +ensure_response(#http_req{resp_state=done}, _State) -> + ok; %% No response has been sent but everything apparently went fine. %% Reply with 204 No Content to indicate this. ensure_response(#http_req{resp_state=waiting}, State) -> - error_response(204, State); -%% The handler has already fully replied to the client. -ensure_response(#http_req{resp_state=done}, _State) -> - ok. + error_response(204, State). -spec error_response(Code::http_status(), State::#state{}) -> ok. error_response(Code, #state{socket=Socket, -- cgit v1.2.3