Changes between Initial Version and Version 1 of Ticket #21139


Ignore:
Timestamp:
2021-07-19T16:18:04+02:00 (4 years ago)
Author:
taylor.smock
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #21139 – Description

    initial v1  
    33
    44As in comment:12:ticket:21064, the two patches attached to this ticket do the following:
    5 * attachment:21064.BasicPreferencesConfigReset.patch​ iterates through all loaded classes (I make an assumption that we only have one classloader), and sets all AbstractProperty preferences fields to the new preferences. This uses reflection to fiddle with final fields, which has some pitfalls.
    6 * attachment:21064.BasicPreferencesConfigReset.2.patch​ is the same as attachment:21064.BasicPreferencesConfigReset.patch​, except it additionally resets the preferences fields to null. This is better from a test isolation view, but I can almost guarantee that additional tests will fail.
     5* attachment:21139.patch​ iterates through all loaded classes (I make an assumption that we only have one classloader), and sets all AbstractProperty preferences fields to the new preferences. This uses reflection to fiddle with final fields, which has some pitfalls.
     6* attachment:21139.2.patch​ is the same as attachment:21139.patch​, except it additionally resets the preferences fields to null. This is better from a test isolation view, but I can almost guarantee that additional tests will fail.