[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/