bugGNU Octave - Bugs: bug #41204, Better pretty printing with 'echo...

 
 

bug #41204: Better pretty printing with 'echo on all'

Submitter:  Rik <rik5>
Submitted:  Sat 11 Jan 2014 04:47:35 PM UTC
   
 
Category:  Interpreter Severity:  2 - Minor
Priority:  3 - Low Item Group:  Other
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 13 Jan 2014 07:17:40 PM UTC, comment #3: 

I pushed the pretty print patch to the gui-release branch (http://hg.savannah.gnu.org/hgweb/octave/rev/7e297c293e4c).

Rik <rik5>
Group administrator
Sun 12 Jan 2014 01:44:22 AM UTC, comment #2: 

The pretty print patch works for me if you want to go ahead and commit it.

Rik <rik5>
Group administrator
Sat 11 Jan 2014 05:53:49 PM UTC, comment #1: 

I'm attaching a possible fix for this problem.


(file #30246)

John W. Eaton <jwe>
Group administrator
Sat 11 Jan 2014 04:47:35 PM UTC, original submission:  

With 'echo on all' the leading debug prompt of '+ ' is not printed when the source line is blank or looks like a blank comment.

Example:


echo on all
ls
+ ## Copyright (C) 2006-2013 John W. Eaton
##
+ ## This file is part of Octave.
##
+ ## Octave is free software; you can redistribute it and/or modify it
+ ## under the terms of the GNU General Public License as published by
+ ## the Free Software Foundation; either version 3 of the License, or (at
+ ## your option) any later version.
##
+ ## Octave is distributed in the hope that it will be useful, but
+ ## WITHOUT ANY WARRANTY; without even the implied warranty of
+ ## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ ## General Public License for more details.
##
+ ## You should have received a copy of the GNU General Public License
+ ## along with Octave; see the file COPYING.  If not, see
+ ## <http://www.gnu.org/licenses/>.

+ ## -*- texinfo -*-


Note the blank comment lines in the m-file ls.m do not have the debug prompt in front of them.  Similarly, the blank line between the GPL license and the start of the Texinfo documentation does not display the debug prompt.

This is a minor nit, but it might be easy to fix.


Rik <rik5>
Group administrator

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #30246:  echo-diffs.txt added by jwe (612B - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by rik5 (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-01-13 rik5 StatusPatch Submitted Fixed
        Open/ClosedOpen Closed
    2014-01-11 jwe Attached File- Added echo-diffs.txt, #30246
        StatusConfirmed Patch Submitted
    2014-01-11 rik5 Priority5 - Normal 3 - Low

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code