Community
Participate
Working Groups
3.1 HEAD ColumnPixelData public boolean addTrim This read-write field was added as API in the 3.1 cycle. Exposing fields as API (other than public static final ones) is bad practice because a client can change the field and there's nothing the class can do to prevent or even track that change. All access to read-write field should be through accessor method.
This was added by Nick to fix Bug 90712. Moving to Nick
I'm aware of the limitations of using fields, but it's a fairly common practice in layouts to do this (e.g. see GridData and other SWT layout data classes). In this case, it's just a signal to the layout class to do some extra adjustment. I think we should just leave as-is for 3.1.