Hi, at a first look, the changes look fine to me; however it is too long ago that I touched the code... :)
Regards Carsten Ian Boston wrote > Jirad at https://issues.apache.org/jira/browse/SLING-1995 > Since I've had no comments, I assume this is Ok and wont break others work. > Ian > > On 24 Feb 2011, at 13:28, Ian Boston wrote: > >> Hi, >> >> The ContentLoader only updates content if the bundle is updated. If the >> bundle was installed while the container was not running, it doesnt get the >> "updated" state and so is ignored. >> >> [1] looks at the lastModified state of the bundle and compares it to the >> lastModfied time of the content loader registration node, so that if the >> bundle is newer, the content gets loaded, even if the bundle was installed >> when the container was not running. >> >> WDYT? >> Should I commit ? >> >> (doing review then commit since I haven't every patched this area) >> Ian >> >> 1 http://codereview.appspot.com/4220045 > > -- Carsten Ziegeler cziege...@apache.org
