all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: MON KEY <monkey@sandpframing.com>
To: Lennart Borgman <lennart.borgman@gmail.com>
Cc: 6390@debbugs.gnu.org
Subject: bug#6390: Should not regexp-quote quote newline?
Date: Sat, 12 Jun 2010 02:18:43 -0400	[thread overview]
Message-ID: <AANLkTinTOIksS3tpFTnVto36n8MLFj59UIoTxHtdRT38@mail.gmail.com> (raw)
In-Reply-To: <AANLkTin1fblwRl02Iq-iryU5zpEdXDBB9kY3TTJEzXZO@mail.gmail.com>

On Fri, Jun 11, 2010 at 4:37 PM, Lennart Borgman
<lennart.borgman@gmail.com> wrote:
> All I proposed was that it should write tab as \t too.

 i) This is not the extent of what you proposed;
ii) It is none-the-less a bad idea;

Newline and formfeed are control characters which affect the display
and interpretation of vertical character motion and display. Tab OTOH
is horizontal whitespace and doesn't affect vertical display in the
same manner (either syntactically or visually).

Regardless, the function name `print-escape-newlines' and its
documentation SAY NOTHING ABOUT ESCAPING TAB CHARACTERS!!!!!

What you really want is `print-escape-arbitrarily-for-lennart' :P
You can implement such a feature by cobbling together various
bits of code from format-spec.el format.el descr-text.el pp.el and
`filter-buffer-substring'.

> On a brighter side I think you have misunderstood my question and
> suggestion.

I well understood your suggestion as presented:

,----
| Unfortunately print-escape-newlines does not quote tab, only newline
| and form-feed. Which in my opinion is a bug. To fix this just copy 5
| lines in print.c at line 1667 and replace form-feed with tab. (I have
| just done that in my patched copy, but it is easier to do this by hand
| then to use a patch.)
`----

If you want your suggestions to be understood by others with as little
ambiguity as possible you _must_ submit patches or example code.

As written I read your 'suggestion' to say:

 "... replace form-feed with tab ..."

In the absence of a patch/example how else could/should it have been
read? FWIW I _did_ take time to examine lines ~1667 of print.c and it
certainly wasn't clear to me what you intended.

Thanks for wasting my time twice. Once in attempting to understand
what you meant and now a second time to clarify a mutual
misunderstanding.

Multiply this type of waste by the number of people (now and in the
future) whom reference this bug report and you've potentially wasted
alot of people's time.

> I might not have proposed that if I knew it was that utterly
> disturbing to someone. ;-)

I doubt that.

You seem all too ready to file ambiguous bug reports without
backtraces and/or to suggest changes/fixes to vacuous `bugs' without
an accompanying patch or example source which is illustrative of both
a problem and of a proposed solution.

Indeed, I do find this utterly disturbing on your part (esp. in the
aggregate).

--
/s_P\





  reply	other threads:[~2010-06-12  6:18 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-10 14:42 bug#6390: Should not regexp-quote quote newline? Lennart Borgman
2010-06-10 15:01 ` Andreas Schwab
2010-06-10 15:02   ` Lennart Borgman
2010-06-10 15:34     ` Drew Adams
2010-06-10 16:08     ` Andreas Schwab
2010-06-10 16:11       ` Lennart Borgman
2010-06-10 16:22         ` Andreas Schwab
2010-06-10 16:46           ` Lennart Borgman
2010-06-10 17:03             ` Andreas Schwab
2010-06-10 18:07               ` Lennart Borgman
2010-06-10 23:11                 ` Lennart Borgman
2010-06-11  0:44                   ` Lennart Borgman
2010-06-10 15:22 ` Drew Adams
2010-06-10 15:34   ` Lennart Borgman
2010-06-10 16:28     ` Juanma Barranquero
2010-06-10 16:47       ` Lennart Borgman
2010-06-10 16:56     ` Andreas Schwab
2010-06-10 17:00       ` Lennart Borgman
2010-06-11  4:19 ` MON KEY
2010-06-11  4:43   ` Lennart Borgman
2010-06-11 20:09     ` MON KEY
2010-06-11 20:37       ` Lennart Borgman
2010-06-12  6:18         ` MON KEY [this message]
2010-06-12 13:28           ` Lennart Borgman
2010-06-14  3:00             ` MON KEY
2010-06-14  5:33               ` Lennart Borgman
2010-06-12  6:51         ` Kevin Rodgers
2010-06-12 13:37           ` Lennart Borgman
2011-07-09  5:30           ` Glenn Morris
2011-07-09 10:03             ` Lennart Borgman

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=AANLkTinTOIksS3tpFTnVto36n8MLFj59UIoTxHtdRT38@mail.gmail.com \
    --to=monkey@sandpframing.com \
    --cc=6390@debbugs.gnu.org \
    --cc=lennart.borgman@gmail.com \
    /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.