> Needs unit test, and mangled whitespace on the change to CASImpl. > Nonetheless, I think the above commit is communicative.
Mangling whitespace is actually a substantial barrier to review. Can you please reformulate the patch with the current whitespace intact so we can more easily review the impact to existing components? Thanks, M -- You are currently subscribed to cas-dev@lists.jasig.org as: arch...@mail-archive.com To unsubscribe, change settings or access archives, see http://www.ja-sig.org/wiki/display/JSG/cas-dev