MacArthur, Ian (SELEX GALILEO, UK) wrote:
> Point test: typo at line 125, should be;
>  OLD:   fl_color(FL_GREEN); a = x(); b = y()+70;
>  NEW:   fl_color(FL_GREEN); a = x()+10; b = y()+70;

        Ah, OK -- fixed.

> Line test: description text seems to also include the description from
> the Point test, confusingly.

        Seems like my pasting error -- fixed.

> Also, should "blured" be spelled as "blurred"? I do not know... OK, this
> spell-checker favours "blurred", for what that is worth. (Maybe that's
> an en-gb thing though?)

        Spelling fixed. Might be en/gb ;)

> Also (also) I think the descriptive wording of the circle test might be
> tweaked a bit:
> 
> "The green rectangles should not be touched by circle drawings."
> 
> It seems to me the green rectangles *are* "touched" by the circle
> drawings, in that they fill adjacent pixels, but they are not
> "over-written" or "broken" (or some other word) by them?

        I'll let Matthias weigh in on that.

        Looks like you're right though; should probably say 'overwritten'.
        (I compared the old unittest image to the new app, and they draw
        this test the same in both, so it's not something I broke)

        I've applied the above fixes to the website copy at:
        http://seriss.com/people/erco/fltk/tmp/unittests-revised.cxx

        ..I added a 'v0.2' comment at the top:

// Fltk unit tests
// v0.1 - Greg combines Matthias + Ian's tests
// v0.2 - Ian's 02/12/09 fixes applied          <--

        So if you don't see that, hit reload.
_______________________________________________
fltk-dev mailing list
[email protected]
http://lists.easysw.com/mailman/listinfo/fltk-dev

Reply via email to