On Tue, May 20, 2014 at 11:25 AM, Tomaz Canabrava <[email protected]> wrote: > On Tue, May 20, 2014 at 2:52 AM, Dirk Hohndel <[email protected]> wrote: >> On Wed, May 14, 2014 at 10:51:10PM -0300, Tomaz Canabrava wrote: >>> So, I was looking at the profile today to port stuff the planner to >>> the new profile, but I couldn't, I was just fixing stuff that I looked >>> at there ( while I was thinking how to port the planner to the new >>> profile ), so no work was lost. :) >>> >>> The 0002 and 0011 patches disables / reenables the planner ( as I >>> tougth at the beginning that I was just going to port it and not fix >>> anything... silly me. ) >> >> Obviously I didn't take those two patches, nor did I take the ones that >> didn't apply to master (either now or before I pushed all the changes made >> on the boat). My guess is that you have some other planner code pulled >> into your tree that aren't in master. >> >> And I didn't take the patch that in all caps pointed out that you didn't >> know what you were doing :-) >> >>> I think tomorrow I'll start really porting the planner as those other >>> patches kept me busy. >>> >>> Dirk, please take a good look at the patches, as I don't program for >>> subsurface for quite a while, see if I didn't messed anything. ;) >> >> So five went in. And they went in under heavy cursing as you clearly lost >> all your settings for indentation and whitespace. I had to edit every >> single patch (I think). >> >> Tomaz, we went through this before. It's NOT HARD. But it makes my life as >> a maintainer a pain. PLEASE. Comply with our whitespace rules. > > "oops" > Editing my configurations, do not want to make your life miserable. :) > >> /D
Dirk, I didn't see any of those commited into master, do you want me to rework them? _______________________________________________ subsurface mailing list [email protected] http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
