On Thu, 2024-09-26 at 23:24 +0200, Jakub Jelinek wrote: > Hi! > > diagnostic.h already relies on vec.h, it uses auto_vec in one spot. > > The following patch converts m_classification_history and m_push_list > hand-managed arrays to vec templates. > The main advantage is exponential rather than linear reallocation, > e.g. with current libstdc++ headers if one includes all the standard > headers there could be ~ 300 reallocations of the > m_classification_history > array (sure, not all of them will result in actually copying the > data, but > still). > In addition to that it fixes some formatting issues in the code. > > Bootstrapped on i686-linux so far, bootstrap/regtest on x86_64-linux > and > i686-linux still pending, ok for trunk if it passes it?
Thanks, yes please. Dave