unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Michael Mauger <mmaug@yahoo.com>
Cc: emacs-devel@gnu.org
Subject: Re: Patch for sql.el
Date: Fri, 7 May 2004 08:27:03 -0700 (PDT)	[thread overview]
Message-ID: <20040507152703.46444.qmail@web60306.mail.yahoo.com> (raw)
In-Reply-To: <jwv7jvpchc6.fsf-monnier+emacs@gnu.org>

--- Stefan Monnier <monnier@iro.umontreal.ca> wrote:
> > (defun font-lock-add-syntax (syntax-alist)
> > (defun font-lock-remove-syntax (syntax-alist)
> [..]
> > Or are we pushing the feature freeze...?
> 
> Not only that, but I feel it's just adding ad-hoc cruft above ad-hoc
> cruft.
> 

Understood.  I figured it was worth exploring.

> Is there any particular reason why you don't merge all those submodes
> and simply handle a language that's a superset of all?
> Basically accept @ # $ as symbol components, and accept all the
> keywords of all the known servers...
> That should simplify your code.
> 

The problem with SQL is that there is no real standard.  The core is
standard, but the functions and procedural extensions vary significantly
from one product to another.  

One of the reasons I originally started using sql-mode was that I was
using two database products at the same time (oracle and ms).  The
product specific features helped keep me sane.  For example, Oracle calls
its sub-string function `substr' whereas MS (and ANSI) call it
`substring'.  I go back 20+ years with Oracle so my instinct is the
abbreviated form.  When I used the abbreviated form in MS scripts I had
immediate feedback that I had used the wrong form.  

I have one more patch coming for sql.el that removes, at compile time,
keywords in product specific lists that are also in the ANSI list.  This
slows the compile time, but seems to speed up the fontification.  It also
allows me to keep the entire product specific keyword lists in the source
without manually filtering out the keywords duplicated in ANSI.  

I will keep the font-lock reset code as it is for now.  

Thanks for your assistance and recommendations.

Michael


=====

-- 
Michael Mauger 
http://www.geocities.com/mmaug/ 

As my friends know all too well, 
'Mike' is a four-letter word.

  reply	other threads:[~2004-05-07 15:27 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <m1vfjis64o.fsf-monnier+emacs@gnu.org>
2004-05-04  2:59 ` Patch for sql.el Michael Mauger
2004-05-04 21:59   ` Stefan Monnier
2004-05-06 22:30     ` Michael Mauger
2004-05-06 22:41       ` Stefan Monnier
2004-05-07 15:27         ` Michael Mauger [this message]
2004-05-07 16:48           ` Kevin Rodgers
2004-05-08 21:59             ` Richard Stallman
2004-05-08 21:59           ` Richard Stallman
2004-04-28  3:29 Michael Mauger
2004-04-29 23:28 ` Stefan Monnier

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=20040507152703.46444.qmail@web60306.mail.yahoo.com \
    --to=mmaug@yahoo.com \
    --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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).