patchGNU Octave - Patches: patch #8464, Make lssa clang compliant (that...


patch #8464: Make lssa clang compliant (that is, make it correct C++).

Submitted by:  None
Submitted on:  Sun May 18 18:32:52 2014  
Category: NonePriority: 5 - Normal
Status: In ProgressPrivacy: Public
Assigned to: NoneOriginator Email: -unavailable-
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Mon May 19 17:40:48 2014, comment #1:

Your patch might be technically correct, but the commit message is a bit too pithy.

How about instead something like

Your username in the patch is also rather nonstandard. Do you prefer that username instead of your full name followed by an email address? Is this username what you set in your ~/.hgrc?

You can further amend your patch by editing files and doing

One further point: have you profiled this to ensure this doesn't result in a slowdown? It calls a ctor and dtor on each iteration of a loop, doesn't it? There might be a better approach.

I hope my review of your patch does not cause you any undue burden.

Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Sun May 18 18:32:52 2014, original submission:

If compiling the package lssa with clang, it quits with an error. The problem is that in strict C++, the functions "real" and "image" in std::complex do not return lvalues. gcc relaxes the rules and allows them to be lvalues, but clang doesn't.



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

Attach File(s):

Attached Files
file #31401:  stephens-patch-for-fixing-octave.diff added by None (3kB - text/x-patch)


Depends on the following items: None found

   patch dependencies.


Carbon-Copy List
  • -unavailable- added by jordigh (Posted a comment)
  • -unavailable- added by None (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed Jul 6 12:50:53 2016lachlanDependencies-=>patch #7925 is dependent
    Mon May 19 17:40:48 2014jordighStatusNone=>In Progress
    Sun May 18 18:32:52 2014NoneAttached File-=>Added stephens-patch-for-fixing-octave.diff, #31401

    Back to the top

    Powered by Savane 3.1-cleanup1