#14 Faulty dirty tracking in XSolidBrush Color setter

v1.0 (example)
closed
nobody
None
5
2013-05-13
2012-09-24
Joshua Bell
No

The code in XSolidBrush's Color attribute reads;

this.color = value;
#if GDI
this.gdiDirty = this.gdiDirty || this.color != value;
#endif

Since the test follows the assignment it will never be true. In other words, once the brush has been realized, assignments to Color will not update it.

The assignment should be moved after the tests (i.e. to the end of the Color setter)

Discussion

  • PDFsharp Team
    PDFsharp Team
    2013-05-13

    To be fixed with next release.

     
  • PDFsharp Team
    PDFsharp Team
    2013-05-13

    • status: open --> closed
    • Group: --> v1.0 (example)