bugGNU Octave - Bugs: bug #48906, [octave forge] (lssa)...

 
 

bug #48906: [octave forge] (lssa) fastlscomplex.cc does not compile against g++

Submitter:  Rafael Laboissière <rlaboiss>
Submitted:  Thu 25 Aug 2016 08:07:53 PM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Build Failure
Status:  Duplicate Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * other Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 23 Jul 2018 05:15:37 PM UTC, comment #5: 

Closing as a duplicate of #48425 and will reopen if anything changes

John Donoghue <lostbard>
Group Member
Mon 23 Jul 2018 12:50:44 PM UTC, comment #4: 

comment #3 looks like it isn't the latest code from the repo, which should work

John Donoghue <lostbard>
Group Member
Sun 21 Jan 2018 07:35:34 AM UTC, comment #3: 

not against version 7.2


fastlscomplex.cc: In function ‘bool flscomplex(const RowVector&, const ComplexRowVector&, double, int, int, ComplexRowVector&)’:
fastlscomplex.cc:375:78: error: lvalue required as left operand of assignment
                             record_current->power_series[p].real() -= h.imag();
                                                                              ^


Juan Pablo Carbajal <juanpi>
Group Member
Fri 26 Aug 2016 05:23:58 PM UTC, comment #2: 

The code in Mercurial compiles correctly against g++ v6.  Please, close this bug report.

Rafael Laboissière <rlaboiss>
Thu 25 Aug 2016 08:23:54 PM UTC, comment #1: 

fastlscomplex.cc has been modified in the repository to fix Bug #48425. Can you please make a patch that can be applied to the current version?

Nir Krakauer <nir_krakauer>
Thu 25 Aug 2016 08:07:53 PM UTC, original submission:  

File fastlscomplex.cc in version 0.1.2 of the lssa package does not compile against g++ version 6, yielding error messages like this:


fastlscomplex.cc:375:78: error: lvalue required as left operand of assignment
record_current->power_series[p].real() -= h.imag();


This comes from the fact that assignments like:


x.real() = 3;


must actually be written as:


x.real(3);


The patch attached to this bug report fixes the problem.

Rafael Laboissière <rlaboiss>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #38358:  compile-with-gcc-6.patch added by rlaboiss (5KiB - 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 lostbard (Posted a comment)
  • -email is unavailable- added by juanpi (Posted a comment)
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by nir_krakauer (Posted a comment)
  • -email is unavailable- added by rlaboiss (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
    2018-07-23 lostbard StatusNone Duplicate
        Open/ClosedOpen Closed
    2018-01-21 juanpi Summary[octave forge] (lssa) fastlscomplex.cc does not compile against g++ v6 [octave forge] (lssa) fastlscomplex.cc does not compile against g++
    2017-08-13 jwe Summarylssa: fastlscomplex.cc does not compile against g++ v6 [octave forge] (lssa) fastlscomplex.cc does not compile against g++ v6
    2016-08-25 rlaboiss Attached File- Added compile-with-gcc-6.patch, #38358

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code