diff options
author | Pavel Labath <pavelo@centrum.sk> | 2011-07-22 15:52:58 (GMT) |
---|---|---|
committer | Pavel Labath <pavelo@centrum.sk> | 2011-09-14 17:39:11 (GMT) |
commit | d21ceb4a235d8f8994390d7bc7611b990e6c0c31 (patch) | |
tree | 909bd768f8b9893e0a425d01ef63e10185bfd3ac /src/FocusableList.cc | |
parent | 1f34dee9b3ce82a2b9f3578ecb14ad3c59080c46 (diff) | |
download | fluxbox-d21ceb4a235d8f8994390d7bc7611b990e6c0c31.zip fluxbox-d21ceb4a235d8f8994390d7bc7611b990e6c0c31.tar.bz2 |
Remove the assignment operator from a regular pointer to a RefCount
it is too easy too shoot yourself in the foot with it, other smart pointers also don't allow such
assignments. If you do want to assign to a RefCount pointer, use reset().
ps: assignment between two RefCounts remains possible, of course.
Diffstat (limited to 'src/FocusableList.cc')
-rw-r--r-- | src/FocusableList.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/FocusableList.cc b/src/FocusableList.cc index 77ff247..3373890 100644 --- a/src/FocusableList.cc +++ b/src/FocusableList.cc | |||
@@ -255,7 +255,7 @@ void FocusableList::attachSignals(Focusable &win) { | |||
255 | FbTk::RefCount<FbTk::SignalTracker> &tracker = m_signal_map[&win]; | 255 | FbTk::RefCount<FbTk::SignalTracker> &tracker = m_signal_map[&win]; |
256 | if (! tracker) { | 256 | if (! tracker) { |
257 | // we have not attached to this window yet | 257 | // we have not attached to this window yet |
258 | tracker = new SignalTracker; | 258 | tracker.reset(new SignalTracker); |
259 | tracker->join(win.titleSig(), MemFunSelectArg1(*this, &FocusableList::updateTitle)); | 259 | tracker->join(win.titleSig(), MemFunSelectArg1(*this, &FocusableList::updateTitle)); |
260 | tracker->join(win.dieSig(), MemFun(*this, &FocusableList::remove)); | 260 | tracker->join(win.dieSig(), MemFun(*this, &FocusableList::remove)); |
261 | if(fbwin) { | 261 | if(fbwin) { |