On 04/20/2016 08:22 AM, tbsaunde+...@tbsaunde.org wrote:
From: Trevor Saunders <tbsaunde+...@tbsaunde.org>

+                 unsigned int len = cond_list.length ();
+                 for (unsigned int i = len - 1; i < len; i--)

This is a really icky way to write a loop, the i < len condition makes it look like a forward one. We have FOR_EACH_VEC_ELT{,_REVERSE}, any reason not to use these?


Bernd

Reply via email to