= Review (NS-46): Review of Bug 1694: LocalArcRepositoryClient is broken Fixes the ARCReader constructor call. =
|| Author || Søren ||
|| Moderator || Søren ||
|| State || Closed ||
== Objectives ==
{{{
/trunk/src/dk/netarkivet/common/distribute/arcrepository/LocalArcRepositoryClient.java, revision 838, lines 147-148
}}}
== Summary ==
{{{
Followup by Nicolas Giraud
}}}
'''Total Time Used (Coding,Documentation,Review)''':

{{{
Time use (Coding,Documentation,Review)
NGiraud: 0.25
SVC: 0
}}}
'''General comments''':
|| '''Description''' || '''Classification''' || '''Status''' ||
|| File does not end with a new line: Add new line after EOF || Cosmetic || OK ||
|| add a unittest for the get() method || Cosmetic || OK ||


=== Comments on file 'trunk/src/dk/netarkivet/common/distribute/arcrepository/LocalArcRepositoryClient.java', revision 838 ===
|| '''Lines''' || '''Description''' || '''Classification''' || '''Status''' ||
|| 148 || remove tab from line || Cosmetic || OK ||
|| 148 || remove unused line || Cosmetic || OK ||