= Review (NS-22): BugFix 1336 =
|| Author || Colin ||
|| Moderator || Colin ||
|| State || Closed ||
== Objectives ==
{{{
}}}
== Summary ==
{{{
Reviewed. Cosmetic defects to be fixed by Colin.
}}}
'''Total Time Used (Coding,Documentation,Review)''':

{{{
Time use (Coding,Documentation,Review)
CSR: 1
SVC:  0.5
}}}
'''General comments''':
|| '''Description''' || '''Classification''' || '''Status''' ||


=== Comments on file 'trunk/src/dk/netarkivet/common/utils/JMXUtils.java', revision 713 ===
|| '''Lines''' || '''Description''' || '''Classification''' || '''Status''' ||
|| 125 || Add comment: "Use the default RMI port" (Whatever that is??) || Cosmetic || OK ||
|| 252-256 || Wrong identation || Cosmetic || OK ||
|| 247-256 || The exceptions are not logged here; And only the final exception after giving up are included in the IOFailure. Logging must be done at warning level || NA ||NA ||
|| 298-307 || The exceptions caught are not logged here, and only the final exception after giving up are included in the IOFailure. Please log at level warning || Cosmetic || OK ||