Changes between Initial Version and Version 1 of Ticket #4523, comment 11


Ignore:
Timestamp:
May 18, 2015, 6:32:05 AM (7 years ago)
Author:
Edward Diener

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #4523, comment 11

    initial v1  
    1 I have fixed this on the latest 'develop' branch of Boost PP. I have used a different techniques than the patches submitted as I did not want to make as drastic a change as was suggested to auto_rec. I did use the technique suggested in the seq_for_each patch of caching the seq size rather than having to recalculate it each time. The fix further consists of accepting the 257th inner-loop in BOOST_PP_FOR without forcing an error as long the op(r,state) returned 0, meaning the BOOST_PP_FOR processing is finished.
     1I have fixed this on the latest 'develop' branch of Boost PP. I have used a different technique than the patches submitted as I did not want to make as drastic a change as was suggested to auto_rec. I did use the technique suggested in the seq_for_each patch of caching the seq size rather than having to recalculate it each time. The fix further consists of accepting the 257th inner-loop in BOOST_PP_FOR without forcing an error as long the op(r,state) returned 0, meaning the BOOST_PP_FOR processing is finished.
    22
    33I tested this successfully with gcc and clang with the example given in #4501. The VC++ compiler issues a compiler error: