all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Bozhidar Batsov <bozhidar@batsov.com>
To: Dmitry Gutov <dgutov@yandex.ru>
Cc: emacs-devel <emacs-devel@gnu.org>
Subject: Re: Subject: [PATCH] * lisp/progmodes/ruby-mode.el: Set `ruby-insert-encoding-magic-comment' to nil by default.
Date: Sat, 2 Nov 2013 12:38:48 +0200	[thread overview]
Message-ID: <CAM9Zgm1vgK5i0h62Eh5jdjkHMNR3nXL0_r9GozVmcx2d+Bzt4w@mail.gmail.com> (raw)
In-Reply-To: <87txfvad0a.fsf@yandex.ru>

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

On 2 November 2013 02:00, Dmitry Gutov <dgutov@yandex.ru> wrote:

> Bozhidar Batsov <bozhidar@batsov.com> writes:
>
> > Since Ruby 1.8 reached end of life in the summer and most projects these
> days use Ruby 2.0 (which
> > assumes source files use utf8 by default) having
> `ruby-insert-encoding-magic-comment’ doesn’t make
> > much sense. Most people just disable it anyways, so why not have it
> disabled by default?
>
> AFAICT, even in 2.0 it's still important for Japanese users who may use
> an encoding other than UTF-8.
>
> Disabling the addition of magic comment when UTF-8 is used should be
> done via adding '(utf-8 . nil) to `ruby-encoding-map', and even that I
> can't support personally since our team, at work, is still on 1.9.3.
>
> Also see
> http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00236.html


Disabling this by default is not directly related to improving it. While I
think your suggestions for running this in an after-save-hook are sound, I
feel that the majority of Ruby programmers out there no longer need this,
so it doesn't make sense to have it on by default just to make people
search how to disable it (unless it stops messing with files that are
already using utf-8, which as you note is problematic for users of Ruby
1.9).

P.S. I also think we should insert the comment in Ruby style (e.g. "#
encoding: utf-8") instead of the current Emacs style encoding comment (or
make this customizable).

[-- Attachment #2: Type: text/html, Size: 2057 bytes --]

  reply	other threads:[~2013-11-02 10:38 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-01 12:59 Subject: [PATCH] * lisp/progmodes/ruby-mode.el: Set `ruby-insert-encoding-magic-comment' to nil by default Bozhidar Batsov
2013-11-01 17:46 ` Stefan Monnier
2013-11-01 18:51   ` Bozhidar Batsov
2013-11-02  0:00 ` Dmitry Gutov
2013-11-02 10:38   ` Bozhidar Batsov [this message]
2013-11-02 20:15     ` Dmitry Gutov
2013-11-03  7:35       ` Bozhidar Batsov
2013-11-03  9:22         ` Dmitry Gutov
2013-11-04 13:40           ` Bozhidar Batsov
2013-11-06 17:10             ` Bozhidar Batsov
2013-11-06 21:45               ` Dmitry Gutov
2013-11-07 10:23                 ` Bozhidar Batsov
2013-11-07 11:06                   ` Dmitry Gutov
2013-11-13 14:36                     ` Bozhidar Batsov
2013-11-13 19:19                       ` Dmitry Gutov
2013-11-14  9:53                         ` Bozhidar Batsov
2013-11-14 13:23                           ` Dmitry Gutov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAM9Zgm1vgK5i0h62Eh5jdjkHMNR3nXL0_r9GozVmcx2d+Bzt4w@mail.gmail.com \
    --to=bozhidar@batsov.com \
    --cc=dgutov@yandex.ru \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.