diff options
author | Jan Lehnardt <jan@apache.org> | 2009-09-05 20:37:45 +0000 |
---|---|---|
committer | Jan Lehnardt <jan@apache.org> | 2009-09-05 20:37:45 +0000 |
commit | aa5daeca5c54432e6b985508fadcd3d85cc5388d (patch) | |
tree | 97efa66d0b6def22a30f86df8265595b3c16cef9 /src | |
parent | a41b4818cc496f07a07c324a8a1f7c82fbf0317a (diff) |
code =~ s/imperative/declarative & faster/, thanks to Kostis Sagonas for pointing these out, more to come
git-svn-id: https://svn.apache.org/repos/asf/couchdb/trunk@811709 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'src')
-rw-r--r-- | src/couchdb/couch_config.erl | 4 | ||||
-rw-r--r-- | src/couchdb/couch_httpd_auth.erl | 2 | ||||
-rw-r--r-- | src/couchdb/couch_rep.erl | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/src/couchdb/couch_config.erl b/src/couchdb/couch_config.erl index 1e9ae41f..69d0ac93 100644 --- a/src/couchdb/couch_config.erl +++ b/src/couchdb/couch_config.erl @@ -94,8 +94,8 @@ init(IniFiles) -> {ok, ParsedIniValues} = parse_ini_file(IniFile), ets:insert(?MODULE, ParsedIniValues) end, IniFiles), - WriteFile = case length(IniFiles) > 0 of - true -> lists:last(IniFiles); + WriteFile = case IniFiles of + [_|_] -> lists:last(IniFiles); _ -> undefined end, {ok, #config{write_filename=WriteFile}}. diff --git a/src/couchdb/couch_httpd_auth.erl b/src/couchdb/couch_httpd_auth.erl index e7a2e3a1..522c0266 100644 --- a/src/couchdb/couch_httpd_auth.erl +++ b/src/couchdb/couch_httpd_auth.erl @@ -446,7 +446,7 @@ update_user_req(#httpd{method='PUT', mochi_req=MochiReq, user_ctx=UserCtx}=Req, Hash = case Password of <<>> -> CurrentPasswordHash; - _P when length(OldPassword) == 0 -> + _P when OldPassword = [] -> throw({forbidden, <<"Old password is incorrect.">>}); _Else -> OldPasswordHash = hash_password(OldPassword1, UserSalt), diff --git a/src/couchdb/couch_rep.erl b/src/couchdb/couch_rep.erl index 1411716f..b1c46db9 100644 --- a/src/couchdb/couch_rep.erl +++ b/src/couchdb/couch_rep.erl @@ -268,7 +268,7 @@ compare_replication_logs(SrcDoc, TgtDoc) -> compare_rep_history(SourceHistory, TargetHistory) end. -compare_rep_history(S, T) when length(S) =:= 0 orelse length(T) =:= 0 -> +compare_rep_history([], []) -> ?LOG_INFO("no common ancestry -- performing full replication", []), {0, []}; compare_rep_history([{S}|SourceRest], [{T}|TargetRest]=Target) -> |