> > You have mixed up core changes like inlining
> ReadTwoBytes
> > with the bestgames feature change. These should be in
> separate patches, not together. Large patches like this
> should have documentation about wtf they do. Ditto the
> Storedline changes.
> > You've removed the button bar. Congratulations...
> > but this should be totally separate otherwise you're
> wasting the time of everyone who reviews this patch.
> > Sorry for being so blunt, and perhaps i am also out of
> line.
> >
> At some point during the discussion on tree & filter in
> the mailing list i had the impression that there was
> some semantic misunderstandings.
> So I sent a screenshot and a cvs diff to my current code to
> avoid any language problem.
> I know it's difficult to look at a patch that do so many
> different things, but my intention was only to give the
> ability to try the new behavior of the tree window.
Yes... thanks :>
It was great to get the patch and see what you're doing,
and can't wait for the weekend to use some of your ideas.
S.
------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires
February 28th, so secure your free ArcSight Logger TODAY!
http://p.sf.net/sfu/arcsight-sfd2d
_______________________________________________
Scid-users mailing list
Scid-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/scid-users
