octave-bug-tracker
[Top][All Lists]
Advanced

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

[Octave-bug-tracker] [bug #63738] Changing iterators to range-based loop


From: John W. Eaton
Subject: [Octave-bug-tracker] [bug #63738] Changing iterators to range-based loops where possible
Date: Thu, 2 Feb 2023 08:35:41 -0500 (EST)

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

Yes, we want to use range-based for loops where possible.  At some point in
the past it was discussed and made a goal to change as we had time, though I
can't find the discussion at the moment.

If not using explicit declarations, I think we should prefer "const auto&"
where possible and "auto&" otherwise.  For example, in the last two hunks of
your patch, I think it should be possible to use "const auto&".

Also, we don't care much about changing this kind of thing in the qterminal
sources, so you can skip those files.


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/




reply via email to

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