bugGNU Octave - Bugs: bug #38579, Nested multiline comments not...

 
 

bug #38579: Nested multiline comments not recognised correctly

Submitter:  None
Submitted:  Sun 24 Mar 2013 07:08:27 AM UTC
   
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  John Donoghue Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * Any Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Sun 24 Mar 2013 05:25:36 PM UTC, comment #2: 

Thanks, I made this change and checked it in:

http://hg.savannah.gnu.org/hgweb/octave/rev/1e064963c90a

John W. Eaton <jwe>
Group administrator
Sun 24 Mar 2013 02:48:47 PM UTC, comment #1: 

A slightly easier to follow example which illustrates the same thing.


%{
  not run
  %{
    not run
  %}
  disp ('end 1st comment');
%}
disp ('end 2nd comment');


Rik <rik5>
Group administrator
Sun 24 Mar 2013 07:08:27 AM UTC, original submission:  

When encountering commands in a nested multiline comment, the lines after the first closed multiline comment are still run.

Example:
%{
 not run
%{
 not run
%}
 will run line
%}
 will run line

Faileds in dev version as release version 3.6.4 works correctly.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #27669:  38579_nexted_comment.patch added by None (482B - text/x-patch - Patch to lex.ll for nested multiline comments)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by jordigh (Updated the item)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-03-25 rik5 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2013-03-24 jordigh StatusNone Patch Submitted
    2013-03-24 None Attached File- Added 38579_nexted_comment.patch, #27669

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code