octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: [bug #56425] Patch for the JIT


From: Gene Harvey
Subject: RE: [bug #56425] Patch for the JIT
Date: Sun, 2 Jun 2019 20:24:52 -0500

My fault, I left a lot of changes in that weren’t strictly necessary. I can definitely repackage it if people aren’t too far through yet.

 

With regards to the std::endl changes, it’s just premature optimization (std::endl makes flushes the buffer).

 

Gene Harvey

 

From: Jordi Gutiérrez Hermoso
Sent: Sunday, June 2, 2019 17:45
To: Gene Harvey; Jordi Gutiérrez Hermoso; address@hidden
Subject: [bug #56425] Patch for the JIT

 

Follow-up Comment #1, bug #56425 (project octave):

 

There seem to be a lot of unrelated changes here, what's the point of changing

`os << std::endl;` to `os << '\n'`?

 

This patch is also kind of mammoth which makes it difficult to review. Think

you could split it up?

 

    _______________________________________________________

 

Reply to this item at:

 

  <https://savannah.gnu.org/bugs/?56425>

 

_______________________________________________

  Message sent via Savannah

  https://savannah.gnu.org/

 

 


reply via email to

[Prev in Thread] Current Thread [Next in Thread]