aboutsummaryrefslogtreecommitdiff
path: root/src/FocusableList.cc
diff options
context:
space:
mode:
authorPavel Labath <pavelo@centrum.sk>2011-07-22 15:52:58 (GMT)
committerPavel Labath <pavelo@centrum.sk>2011-09-14 17:39:11 (GMT)
commitd21ceb4a235d8f8994390d7bc7611b990e6c0c31 (patch)
tree909bd768f8b9893e0a425d01ef63e10185bfd3ac /src/FocusableList.cc
parent1f34dee9b3ce82a2b9f3578ecb14ad3c59080c46 (diff)
downloadfluxbox-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.cc2
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) {