patchGNU Octave - Patches: patch #7708, use news as command

 
 

patch #7708: use news as command

Submitter:  Carnë Draug <carandraug>
Submitted:  Thu 26 Jan 2012 11:48:59 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Wont Do Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Mon 21 Oct 2013 09:58:28 PM UTC, comment #6: 

You're right. Seems I did it with cset 3a93af21d206 when I was changing news() to share code with citation().

I have just pushed the other part than changes the syntax after installing a package (cset 8dd280b64de1).

Carnë Draug <carandraug>
Group Member
Mon 21 Oct 2013 06:06:28 PM UTC, comment #5: 

I think that syntax change has already been done.  If I do 'help news' it lists the command form already.

Rik <rik5>
Group administrator
Mon 21 Oct 2013 06:02:06 PM UTC, comment #4: 

ok. But aside changing "Function File" to "Command" in deftypefn, can I push a change that makes the rest of the changes? I think it's much more natural the command syntax in this case.

Carnë Draug <carandraug>
Group Member
Mon 21 Oct 2013 04:21:42 PM UTC, comment #3: 

We will be getting rid of "Function File" and "Command" categories at the beginning of the line in documentation strings, so I'm going to close this patch.

Rik <rik5>
Group administrator
Mon 30 Jan 2012 01:47:26 PM UTC, comment #2: 

I know the part about all functions behaving that way. But because it looked to me that it has a similar behavior or use as the 'help' command, I thought I would change it for consistency. I just don't know what defines a function versus a command.

If this change is ever made, the new patch also changes deftypefn from "Function File" to "Command".

(file #24923)

Carnë Draug <carandraug>
Group Member
Thu 26 Jan 2012 06:13:45 PM UTC, comment #1: 

ALL functions that take character string arguments may be called as "commands".  Listing them that way in the docs is OK, but then people don't understand that it is possible to call them with the function-style syntax.

I'm not sure of the best way to handle this, but I think it should be done consistently for all functions that are commonly called with command-style syntax.

John W. Eaton <jwe>
Group administrator
Thu 26 Jan 2012 11:48:59 AM UTC, original submission:  

I thought that "news" can be used without parentheses, similarly to the help command. As such, I just have a very simple changeset to remove them from the help text (and the displayed message from pkg).

Carnë Draug <carandraug>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #24923:  news-usage-v2.patch added by carandraug (1KiB - text/x-patch)
file #24896:  news-usage.patch added by carandraug (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by jwe (Posted a comment)
  • -email is unavailable- added by carandraug (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 logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-10-21 rik5 StatusNone Wont Do
        Open/ClosedOpen Closed
    2012-01-30 carandraug Attached File- Added news-usage-v2.patch, #24923
    2012-01-26 carandraug Attached File- Added news-usage.patch, #24896

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code