bugGNU Octave - Bugs: bug #52015, warning name:...

 
 

bug #52015: warning name: Octave:possible-matlab-short-circuit-operator

Submitter:  Ernst Reissner <ernstreissner>
Submitted:  Thu 14 Sep 2017 09:21:33 AM UTC
   
 
Category:  Interpreter Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Documentation
Status:  Wont Fix Assigned to:  None
Originator Name:  ernstreissner Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 14 Sep 2017 03:02:41 PM UTC, comment #1: 

Changing warning IDs causes trouble for existing code that uses the IDs explicitly, so I'd rather not do that unless there is an overwhelming reason to do so.

The name might not be great, but it doesn't seem terrible to me.  I think the reason I chose the word "possible" is because you can't tell that the operator will short-circuit just from the syntax and this warning-id was originally used in a warning that was issued during parsing, not evaluation.

John W. Eaton <jwe>
Group administrator
Thu 14 Sep 2017 09:21:33 AM UTC, original submission:  

The id of the warning
Octave:possible-matlab-short-circuit-operator
is misleading, to be more precise the 'possible'
I think
Octave:conversion-into-short-circuit-operator
would be a better choice as the warnings are
warning: Matlab-style short-circuit operation performed for operator |

and

warning: Matlab-style short-circuit operation performed for operator &

Ernst Reissner <ernstreissner>

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

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 ernstreissner (Submitted the item)
  • -email is unavailable- added by ernstreissner
  •  

    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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-09-14 jwe StatusNone Wont Fix
        Open/ClosedOpen Closed
    2017-09-14 ernstreissner Carbon-Copy- Added -email is unavailable-

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code