[PATCH 1/7] Refactor device handling threads
Anton Lundin
glance at acc.umu.se
Tue Jan 20 14:34:59 PST 2015
On 20 January, 2015 - Thiago Macieira wrote:
> On Tuesday 20 January 2015 22:40:49 Anton Lundin wrote:
> > This refactors the device handling threads into having one common
> > ancestor class.
>
> Why? What's the value of this? I'm assuming it's something further down the
> line, but it wasn't evident for me...
>
> Is it just to make things simpler?
It's part of the whole get-progress-events series, and I thought using a
common ancestor to all those classes made it easier to implement the
whole progress event thingie.
That way i can have them all behave the same way in that respect.
I also have some unfinished code for generating trace logs from
different actions for debugging purposes. I've used it my self to
diagnose some older bugs, but its kinda rough, and I haven't had the
time to clean that code up. (And it hasn't bin needed to get any debug
logs from users yet...)
Does that make more sense?
//Anton
--
Anton Lundin +46702-161604
More information about the subsurface
mailing list