CleverRaven / Cataclysm-DDA / 11553

33%

push

travis-ci

<a href="https://github.com/CleverRaven/Cataclysm-DDA/commit/<a class=hub.com/CleverRaven/Cataclysm-DDA/commit/2d18192b5de68fc66b99059f827b393b1569439c">2d18192b5<a href="https://github.com/CleverRaven/Cataclysm-DDA/commit/2d18192b5de68fc66b99059f827b393b1569439c">&quot;&gt;Fix vehicle part install CTD - Take2 (#26043)

* Revert &amp;quot;Use std::pair for carry_names instead (#26039)&quot;

This reverts commit </a><a class="double-link" href="https://github.com/CleverRaven/Cataclysm-DDA/commit/<a class="double-link" href="https://github.com/CleverRaven/Cataclysm-DDA/commit/02a7dce66ef1a766ee175a3fa45baa5fdb5d4754">02a7dce66</a>">02a7dce66</a><a href="https://github.com/CleverRaven/Cataclysm-DDA/commit/2d18192b5de68fc66b99059f827b393b1569439c">.

* Specify std::vector for carry_names std::stack

Using a deque causes horrendous memory issues since it uses at a
*minimum* 4k bytes and it also has the fun side-affect of causing
unexpected behavior when the deque is a inside an object referenced by
vector iterator when the vector gets resized.

Using a std::vector solves both of those issues.

28862 of 112623 relevant lines covered (25.63%)

78869.48 hits per line