steve

steve

finished reviewing CR-399

02 May
steve

added victor to review CR-385

21 Mar
steve

Depends; can 2.2 binaries work on 2.4? If not then I suggest we follow the normal practice and factor out into it's own directory.

steve

finished reviewing CR-384

21 Mar
steve

this looks great, how much testing have you done?

steve

why are we not sorting anymore?

steve

Inc

steve

ForgeRock Inc please

steve

finished reviewing CR-364

28 Feb
steve

Looks great,but for future generations perhaps some javadoc around the method to describe the valid version string format defines in the reg ex?

steve

finished reviewing CR-365

28 Feb
steve

Simple as that, wow!

steve

finished reviewing CR-367

28 Feb
steve

Should be updated to 2011-2012 and ForgeRock Inc

steve

finished reviewing CR-363

23 Feb
steve

want to wrap that with a if (debug.messageEnabled())

steve

created OPENAM-1138, OPENAM-1137

23 Feb
steve

started review CR-360

20 Feb
OPENAM-1049: Exceptions in IdRepo debug log after upgrade to OpenAM 10
steve

resolved OPENAM-1049

20 Feb
Fixed R1690
steve

could we put this in a try {} finally {} block so we can guarantee that the close will be called

steve

is there a reason why these have been left in by commented out?

steve

We should be updating the copyright notice at the top of the file to 2012