Re: Obsolete code in savehist

[Karl Fogel <kfogel@red-bean.com>]

I finally got around to this:

  | revno: 116979
  | revision-id: kfogel@red-bean.com-20140416064636-l1a8ff3q21x5l2q5
  | committer: Karl Fogel <kfogel@red-bean.com>
  | timestamp: Wed 2014-04-16 14:46:36 +0800
  | message:
  |   * savehist.el (savehist-save): Remove workaround for a read-passwd
  |   bug that was fixed before 24.3.  Thanks to Juanma Barranquero for
  |   noticing that the shim was still present.

Hilariously, my original reply agreeing to do it was based on my
misreading Juanma's original Subject line.  He wrote "savehist" but
somehow I read "saveplace" (which I feel much more responsibility to
maintain), and I only realized my mistake after I got into the code and
nothing looked familiar.  But anyway it was an easy shim removal.

Best,
-K

Juanma Barranquero <lekktu@gmail.com> writes:
>On Fri, Feb 21, 2014 at 10:20 PM, Karl Fogel <kfogel@red-bean.com> wrote:
>
>> Have you by any chance verified that
>> the bug is no longer present, though?
>
>No, I haven't verified it. In theory, it is fixed though.
>
>2012-06-22  Stefan Monnier  <monnier@iro.umontreal.ca>
>
>        * subr.el (read-passwd): Don't use a history at all.
>        * savehist.el (savehist-save): Remove password saved accidentally
>        because of the above bug.