search

lefticus / cpp_weekly

The official C++ Weekly Repository. Code samples and notes of future / past episodes will land here at various times. PR's will be accepted in some cases.
The Unlicense
698 stars 26 forks source link

-Wnrvo #388

Closed lefticus closed 3 months ago

lefticus commented 4 months ago

https://compiler-explorer.com/z/Mb58zeab6

fcolecumberri commented 4 months ago

Are you going to update https://github.com/cpp-best-practices/cmake_template/blob/main/cmake/CompilerWarnings.cmake as a result of this? or this is more like a situational flag you only want sometimes?

lefticus commented 4 months ago

I haven't decided yet, I'm going to have to work with it for a bit.

At the very least, it's an education tool.

On Sat, Jun 1, 2024 at 6:41 AM fcolecumberri @.***> wrote:

Are you going to update https://github.com/cpp-best-practices/cmake_template/blob/main/cmake/CompilerWarnings.cmake as a result of this? or this is more like a situational flag you only want sometimes?

— Reply to this email directly, view it on GitHub https://github.com/lefticus/cpp_weekly/issues/388#issuecomment-2143435184, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABZGJ7JWAEHUYRHCRNK3F3ZFG6QBAVCNFSM6AAAAABIP6EYJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBTGQZTKMJYGQ . You are receiving this because you authored the thread.Message ID: @.***>

-- C++ Trainer, Developer and Speaker Check out my training options: https://emptycrate.com/training.html Subscribe to learn about new training events: https://www.subscribepage.com/emptycrate

lefticus commented 3 months ago

https://compiler-explorer.com/z/s7Kqq4xhj

lefticus commented 3 months ago

Coming in Ep434