Add the _ as prefix for private variables. (Actually it would be betteer use a suffixed _, as defined by our coding standards, but in that case you need to update all the members).
For testing, I think you can also emulate something without autopilot by making your LauncherSpecial request to be caused by a fake dbus server you write on tests.
64 + sigc::signal<void, std::string const&, AbstractLaunche rIcon:: Ptr, std::string const&, std::string const&, addrequest_ special;
65 + gint32, gint32, gint32> launcher_
You can just use int at this level, so we are more C++ conformant.
139 + ((SoftwareCente rLauncherIcon* )result. GetPointer( ))->Animate( launcher_ , icon_x, icon_y, icon_size);
Don't use C-style casts.
184 + finished = true;
185 + finished_just_now = true;
Set them using the constructor initializer list.
201 +void SoftwareCenterL auncherIcon: :AddSelfToLaunc her() >icon_animation _complete. emit(self_ abstract) ;
202 +{
203 + _launcher-
204 +}
306 + void AddSelfToLaunch er();
322 + AbstractLaunche rIcon:: Ptr self_abstract;
I don't really think you need those, please don't do that. You already handling an AbstractLaunche rIcon.
244 + target_x = (int)current_ bamf_icon- >GetCenter( launcher- >monitor) .x;
Again, use static_cast
256 + g_timeout_add(30, &SoftwareCenter LauncherIcon: :OnDragWindowAn imComplete, this);
Maybe it can be a little more lazy.
Anyway I'd focus on fixing the problem at the source, instead of using this workaround.
308 + static gboolean OnDragWindowAni mComplete( gpointer data);
Put in private fields.
310 + void Animate( nux::ObjectPtr< Launcher> launcher, gint32 icon_x, gint32 icon_y, gint32 icon_size);
You can also just use Animate( nux::ObjectPtr< Launcher> const& launcher ... And move to pure int's.
312 + void ActivateLaunche rIcon(ActionArg arg);
Put it in protected field.
323 + bool finished;
324 + bool finished_just_now;
Add the _ as prefix for private variables. (Actually it would be betteer use a suffixed _, as defined by our coding standards, but in that case you need to update all the members).
For testing, I think you can also emulate something without autopilot by making your LauncherSpecial request to be caused by a fake dbus server you write on tests.