[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [AUCTeX-devel] Spell checking of macros
From: |
Mosè Giordano |
Subject: |
Re: [AUCTeX-devel] Spell checking of macros |
Date: |
Sun, 17 Jul 2016 19:30:45 +0200 |
2016-07-17 19:28 GMT+02:00 David Kastrup <address@hidden>:
> Mosè Giordano <address@hidden> writes:
>
>> Hi Arash,
>>
>> 2016-07-12 10:25 GMT+02:00 Arash Esbati <address@hidden>:
>>> Arash Esbati <address@hidden> writes:
>>>
>>>> I'm thinking about writing a function like this to catch this kind of
>>>> arguments:
>>>>
>>>> (defun TeX-ispell-tex-arg-end (&optional arg1 arg2)
>>>> (condition-case nil
>>>> (progn
>>>> (while (looking-at "[ \t\n]*\\[") (forward-sexp))
>>>> (forward-sexp (or arg1 1))
>>>> (while (looking-at "[ \t\n]*\\[") (forward-sexp))
>>>> (forward-sexp (or arg2 1)))
>>>> (error
>>>> (message "Error skipping s-expressions at point %d." (point))
>>>> (beep)
>>>> (sit-for 2))))
>>>>
>>>> and do ("tabular[*xy]" TeX-ispell-tex-arg-end).
>>>
>>> Following up myself again, I implemented a function
>>> `TeX-ispell-tex-arg-end' to deal with constructs like:
>>>
>>> \raisebox{2em}[1em][1em]{text} or
>>> \begin{tabularx}{300pt}{|p{3cm}|>{\raggedright}X|c|S|} etc.
>>>
>>> I'd like to apply the attached patch. I would appreciate any comments
>>> before doing this.
>>
>> It looks overall good to me, but note that error messages *do not* end
>> with a period (see `error`'s docstring) and you should replace the
>> `message' function with `format' (and I don't think `beep' is needed).
>
> Uh, no?
>
> error is a compiled Lisp function in ‘subr.el’.
>
> (error STRING &rest ARGS)
>
> Signal an error, making a message by passing args to ‘format-message’.
> In Emacs, the convention is that error messages start with a capital
> letter but *do not* end with a period. Please follow this convention
> for the sake of consistency.
>
> Note: (error "%s" VALUE) makes the message VALUE without
> interpreting format characters like ‘%’, ‘`’, and ‘'’.
>
> You don't want to have format-message called twice. So just use
>
> (error "Error skipping s-expressions at point %d." (point))
Of course you're right, thanks.
Bye,
Mosè
- Re: [AUCTeX-devel] Spell checking of macros, (continued)
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/08
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/08
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/09
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/11
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/11
- Re: [AUCTeX-devel] Spell checking of macros, David Kastrup, 2016/07/11
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/11
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/12
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/17
- Re: [AUCTeX-devel] Spell checking of macros, David Kastrup, 2016/07/17
- Re: [AUCTeX-devel] Spell checking of macros,
Mosè Giordano <=
- Re: [AUCTeX-devel] Spell checking of macros, David Kastrup, 2016/07/17
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/18
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/20
- Re: [AUCTeX-devel] Spell checking of macros, Mosè Giordano, 2016/07/21
- Re: [AUCTeX-devel] Spell checking of macros, Arash Esbati, 2016/07/22