1. C++ / Говнокод #20314

    +5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    ostringstream s;

...

-    return s.str();
+    return std::move(s.str());

    соптимизировано

    Запостил: kurwa, 02 Июля 2016

    Комментарии (16) RSS

    • ava kurwa 02.07.2016 21:04 # +7

      Последние две строчки - это кусок дифа, если что.
      Ответить

    • ava Psionic 02.07.2016 22:24 # +5

      Где оптимизация?
      Ответить

    • ava 3_dar 02.07.2016 22:56 # 0

      Я правило понял, что если эта функция возвращает string, то эта оптимизация до жопы?
      Ответить

      • ava kurwa 02.07.2016 23:16 # +4

        Там std::move до жопы. s.str() и так возвращает временный объект.
        Ну и вообще std::move в return практически никогда ничего не оптимизирует - в лучшем случае это no-op, а в худшем еще и RVO ломает (т.е. делает медленнее).
        Ответить

    • ava Soul_re@ver 03.07.2016 17:49 # +8 

      return std::move(std::move(s.str());)

      Двойная оптимизация.

      Можно расширить на любой уровень оптимизации:
      template <size_t optimization_depth, typename T>
T&& optimized_return(T& value)
{
    return std::move(optimized_return<optimization_depth - 1>(value));
}

template <typename T>
T&& optimized_return<1>(T& value)
{
    return std::move(value);
}


      Говорят, что return optimized_return<numeric_limits<size_t>: :max()>(s.str()); выполняется за отрицательное время.
      Ответить

      • ava dxd 03.07.2016 21:59 # +2

        Но нарушение причинности в крестах - UB, поэтому так делать не надо.
        Ответить

        • ava Soul_re@ver 03.07.2016 22:00 # +2

          Стандарт ничего не говорит про нарушение причинности юзером, но прямо разрешает нарушать её компилятору в случае UB.
          Ответить

        • ava kurwa 03.07.2016 22:13 # 0

          Че за причинность?
          Ответить

    Добавить комментарий