Re: ruby-mode.el updated

["Akinori MUSHA" <knu@iDaemons.org>]

[-- Attachment #1: Type: text/plain, Size: 1389 bytes --]

Hi,

At Mon, 14 Oct 2013 03:36:46 +0300,
Dmitry Gutov wrote:
> On 08.10.2013 13:55, Dmitry Gutov wrote:
> > Your approach with explicitly following how the encoder works (looking
> > up :prefer-utf-8 property, and defaulting to
> > default-buffer-file-coding-system) is workable, but I'd rather move
> > ruby-mode-set-encoding to after-save-hook and just use the exact
> > encoding that was used to save the file (after #15535 is fixed,
> > hopefully soon).
> >
> > And speaking of "reflect the change of the default source encoding in
> > Ruby 2.0", are you sure it's a good idea to do now?
> >
> > Some people are still using Ruby 1.9.3 in production (myself included),
> > so I'd rather the defaults were safe for this use case.
>
> I've installed your changes, aside from those mentioned above, in
> revision 114654. Please test and see if I've missed anything.

Thank you so much!

OK, I'll test it out on Emacs trunk and see how the amount of
compatibility code can be reduced.

> Calling `ruby-mode-set-encoding' in `after-save-hook' is a bit
> questionable, but this way we don't depend on low-level details of how
> coding systems in Emacs work.

OK.

> One drawback that I've seen so far is that `flymake-after-save-hook'
> usually ends up before `ruby-mode-set-encoding', so it will show an
> error, once.

I think it's a minor problem.

--
Akinori MUSHA / https://akinori.org/

[-- Attachment #2: OpenPGP Digital Signature --]
[-- Type: application/pgp-signature, Size: 196 bytes --]