On Wed, Jan 29, 2014 at 9:36 PM, Animesh Chaturvedi <animesh.chaturv...@citrix.com> wrote: > [Animesh] I know but at this time we need closer scrutiny and just the > specific change helps in that. There was a similar comment in private when > some of us were reviewing security patches.
point taken I am not cherry picking the serializable issues and the hiding overidden fields issues. also - result.replaceAll("<br>", "/n"); + result = result.replaceAll("<br>", "/n"); where result is returned without the replacement , because i think /n should be \n anyway I have removed excess formatting changes in only one case. I am going to run a test compile and will the push