I did not know that, thanks. I'll keep it in mind.

On Wed, May 1, 2013 at 6:49 PM, Kelly McLaughlin
<notifications@github.com>wrote:

> In src/riak_cs_manifest_utils.erl:
>
> > @@ -307,18 +337,85 @@ orddict_values(OrdDict) ->
> >  manifest_is_active(?MANIFEST{state=active}) -> true;
> >  manifest_is_active(_Manifest) -> false.
> >
> > -%% NOTE: This is a foldl function, initial acc = no_active_manifest
> > -most_recent_active_manifest(Manifest=?MANIFEST{state=active}, no_active_manifest) ->
> > -    Manifest;
> > -most_recent_active_manifest(_Manfest, no_active_manifest) ->
> > -    no_active_manifest;
> > -most_recent_active_manifest(Man1=?MANIFEST{state=active}, Man2=?MANIFEST{state=active}) ->
> > +-spec delete_time(lfs_manifest()) -> erlang:timestamp() | undefined.
> > +delete_time(Manifest) ->
> > +    case proplists:is_defined(deleted, Manifest?MANIFEST.props) of
>
> lists:keyfind is probably a much better choice generally. This is
> probably not a huge deal right now because our list of props is small, but
> that may not always be true.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/basho/riak_cs/pull/551/files#r4046494>
> .
>