Ошибка в автоматическом преобразовании

Question

class Sample
{
public:
  Sample();
  Sample(int i);
  Sample(Sample& s);
  ~Sample();
};

Sample::Sample()
{
  cout<<"Default constructor called\n";
}

Sample::Sample(int i)
{
  cout<<"1-argument constructor called\n";
}

Sample::Sample(Sample& s)
{
  cout<<"Copy constructor called\n";
}

Sample::~Sample()
{
  cout<<"Destructor called\n";
}

void Fun(Sample s)
{

}

int main()
{
  Sample s1;
  Fun(5);

  return 0;
}

Я ожидал неявного преобразования 5. Но, когда я компилирую приведенный выше код, я получаю следующую ошибку:

main.cpp:7:8: error: no matching function for call to ‘Sample::Sample(Sample)’
main.cpp:7:8: note: candidates are:
Sample.h:10:3: note: Sample::Sample(Sample&)
Sample.h:10:3: note:   no known conversion for argument 1 from ‘Sample’ to ‘Sample&’
Sample.h:9:3: note: Sample::Sample(int)
Sample.h:9:3: note:   no known conversion for argument 1 from ‘Sample’ to ‘int’
Sample.h:8:3: note: Sample::Sample()
Sample.h:8:3: note:   candidate expects 0 arguments, 1 provided
Helper.h:6:13: error:   initializing argument 1 of ‘void Fun(Sample)’

В чем проблема? Когда я удаляю конструктор копирования, приведенный выше код успешно компилируется.

Заранее спасибо.

Answer 1

Временные ссылки не могут связываться с неконстантными ссылками. Ваш конструктор копирования должен быть:

Sample::Sample(const Sample&)

Удаление этого говорит компилятору генерировать тривиальный, который будет иметь вышеуказанную подпись.