Hi Noam and everyone,

thank you *very* much for your review. I appreciate the effort to review
such a series of patches. Please find attached the new patches. I took
all your comments into account except for what follows:

Noam Postavsky <npostavs@gmail.com> writes:
> Damien Cassou <damien@cassou.me> writes:
>> * lisp/auth-source-pass.el: Private functions were refactored to
>> reduce the number of decryption operations.
>
> Double spacing, and this ChangeLog entry is a little sparse.  It looks
> like the last two prose paragraphs could be easily made into ChangeLog
> entries, since they're already talking about specific functions.


I thought about doing that as well but didn't. If you insist, I will do
the requested changes but here are my reasons for not doing it right
away:

- The changes are on private methods (with the "--" naming convention)
  and I'm not sure how much of private changes should be in the
  ChangeLog.

- The commit message you mention modifies functions that are modified
  again by a later patch (named "Refactoring of auth-source-pass"). I
  usually only send the latest version of my code and not the whole
  history but, this time, I'm not the author of the original version and
  I believe the author deserves to have his name in Emacs' git history
  because of the massive work he did for the package.

- I have tried not to rewrite too much of contributor's code and text in
  their own commit so that the authorship makes sense. I have no problem
  changing contributions in a later commit though (as I've shown in the
  patch "Refactoring of auth-source-pass").


What do you think?

>> * lisp/auth-source-pass.el: Refactoring.
>
> This one's a little empty too.


Changes are again on private functions. What do you suggest?

Thank you again for your work!

-- 
Damien Cassou
http://damiencassou.seasidehosting.st

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill