> 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

Reply via email to