Subject: bug#14332: closed (Re: bug#14332: 24.3; Octave indentation bug)

From: [email-masked] (GNU bug Tracking System)
Date: Tue, 07 May 2013 00:00:02 +0000

Your bug report

#14332: 24.3; Octave indentation bug

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
14332: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=14332
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#14332: 24.3; Octave indentation bug
From: Leo Liu <[email-masked]>
To: Stefan Monnier <[email-masked]>
Cc: [email-masked]
Date: Tue, 07 May 2013 07:58:10 +0800
List-Id: <debbugs-submit.debbugs.gnu.org>

Fixed in trunk.

On 2013-05-07 05:37 +0800, Stefan Monnier wrote:
> No, octave-syntax-propertize-sqs just skips over the contents of
> single-quoted strings and from what you say you only want to handle
> backslash in double-quoted strings.

Thank you, Stefan, for the help. Much appreciated.

Leo

Subject: 24.3; Octave indentation bug
From: Leo Liu <[email-masked]>
To: [email-masked]
Date: Thu, 02 May 2013 12:04:44 +0800
List-Id: <debbugs-submit.debbugs.gnu.org>

Indent the following lines in octave mode

if (true)
  a = (1 \(2-3));
else
  a = 0;
endif

gives me

if (true)
  a = (1 \(2-3));
                else
                  a = 0;
                endif

It seems treating \ as escape outside strings is incorrect. Is the
following fix is correct?

diff --git a/lisp/progmodes/octave.el b/lisp/progmodes/octave.el
index a58fdefb..1e2c2425 100644
--- a/lisp/progmodes/octave.el
+++ b/lisp/progmodes/octave.el
@@ -233,7 +233,7 @@ (defvar octave-mode-syntax-table
     (modify-syntax-entry ?& "."   table)
     (modify-syntax-entry ?| "."   table)
     (modify-syntax-entry ?! "."   table)
-    (modify-syntax-entry ?\\ "\\" table)
+    (modify-syntax-entry ?\\ "." table)
     (modify-syntax-entry ?\' "."  table)
     ;; Was "w" for abbrevs, but now that it's not necessary any more,
     (modify-syntax-entry ?\` "."  table)