Как правильно написать MutexLocker?

Question

[polar_winter]

Создал самописную обёртку MutexLocker, согласно RAII.

class pthread_mutex_locker
{
public:
    pthread_mutex_locker(pthread_mutex_wrapper * mu);
    ~pthread_mutex_locker();

private:
    pthread_mutex_wrapper * mutex;
};

Применил свой MutexLocker c анонимным объектом.

// Тело какой-то функции.
{
 pthread_mutex_locker(&mutex);
//Какой-то код

}

Есть подзрение, что анонимный объект уничтожается раньше, чем нужно и не защищает код. Как сделать, что бы он уничтожался во время выхода из функции. Должен ли MutexLocker быть именнованным объетом? Гарантируется ли уничтожение именнованных объектов по выходу из функции,а не вдругом месте?

gcc version 4.8.1 20130909

Comments

[Sergey Lagner]

Армянское Радио правильно указал, что переменная не может быть анонимной. Чисто для информации в std уже есть lock_guard. Посмотрите пример его использования может станет понятней
en.cppreference.com/w/cpp/thread/lock_guard

Answer 1

[Армянское Радио]

Ваш вариант не пройдет - мьютекс будет уничтожен сразу же после создания. Строго говоря, в C++ нет такого понятия "анонимный экземпляр", есть "временный экземпляр".

Для именованного экземпляра гарантируется уничтожение при выходе за из области видимости - составного оператора {}.

Answer 2

[Mercury13]

pthread_mutex_locker locker(&mutex);

Назови его как-нибудь, и всё будет окей.

Answer 3

[Дмитрий]

А чем не угодил std::lock_guard?

Comments

[polar_winter]

Отлаживал очень давнешний код, до массовой поддержки, с++11

Answer 4

[AxisPod]

Если нет C++11, берите boost::thread.