On Thu, Nov 16, 2017 at 3:00 PM, Robert Helling <[email protected]> wrote:
> Hi, > > I wish I wouldn’t have to write this message but could rather send a PR… > > I have started work to make the planner thread save so we could plan in > the background without holding up the UI. This of course means that it > should not keep stuff in global variables but that each instance should > hold its own copy of local variables. Therefore, I am going through the > code and try to turn global state variables in local ones (passed around as > a pointer to a struct planner_state). Unfortunately, the current state is a > mess (it compiles but the planner is broken, wrong plans and also the tests > fail). My hope is this can be fixed with not too much additional work (I > have already invested almost two days in this) but right now I don’t have > the time (and due to a family celebration it is very unlikely that is will > be resolved over the weekend). > > But what I already have touches a lot of code in the planner (planner.c > deco.c and also user interface .cpp related to it). So could I please ask > you to hold back working on the planner over the weekend so I don’t run > into a big merge problem when I hopefully have this ready early next week? > > If you are interested in what I am doing, look at my threadedplanner > branch on GitHub. > you deserve a kiss. > Best > Robert > > -- > .oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo. > oOo.oOo.oOo.oOo.oOo.oO > Robert C. Helling Elite Master Course Theoretical and Mathematical > Physics > Scientific Coordinator > Ludwig Maximilians Universitaet Muenchen, Dept. > Physik > Phone: +49 89 2180-4523 Theresienstr. 39, rm. B339 > http://www.atdotde.de > > Enhance your privacy, use cryptography! My PGP keys have fingerprints > A9D1 A01D 13A5 31FA 6515 BB44 0820 367C 36BC 0C1D and > DCED 37B6 251C 7861 270D 5613 95C7 9D32 9A8D 9B8F > > > > > > > _______________________________________________ > subsurface mailing list > [email protected] > http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface > >
_______________________________________________ subsurface mailing list [email protected] http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
