Don't store the result of std::min in a const &

gistfile1.cpp

#include <stdio.h>
#include <algorithm>

class B {
  public:
    int b_;

    B(int b) : b_(b) {}
    ~B() { b_ = 23; }

    bool operator<(const B & o) const {
      return b_ < o.b_;
    }
};

int main() {
  const B & m = std::min(B(3), B(5));
  printf("min %d\n", m.b_);
  return 0;
}

More like "don't pass temporaries into std::min and expect to use a reference to the result"

#include <stdio.h>
#include <algorithm>

class B {
  public:
    int b_;

    B(int b) : b_(b) {}
    ~B() { b_ = 23; }

    bool operator<(const B & o) const {
      return b_ < o.b_;
    }
};

int main() {
  B b3(3);
  B b5(5);
  const B & m = std::min(b3,b5);
  printf("min %d\n", m.b_);
  return 0;
}