Understood. As I mentioned, it's highly likely that it gets normalized
prior to this point, so I'm not overly concerned about it.
- Keith
On 02/ 3/11 02:53 PM, John Fischer wrote:
Keith,
One other thing .... As Ethan points out this part of the code was in
the product
prior to my push on Tuesday.
John
On 02/ 3/11 02:47 PM, John Fischer wrote:
On 02/ 3/11 02:20 PM, Keith Mitchell wrote:
Parsed and validated isn't quite the same as "normalized." i.e.,
<Criteria Name="abc" /> is valid XML and would parse
correctly, but would not get caught by the given regex. So unless
the extra spaces are normalized at some point prior as a result of
XML processing, there's potential for latent bugs. However, it's
very likely that the XML gets normalized before that point. If it's
how the code was before, and the bug to replace with XML parsing is
resolved quickly enough, I'm not as concerned.
I hear you.
I am dubious about how long 'quickly enough' will be. The referenced
CR is from 2009-11-05. I suppose that it just needs to get assigned to
someone and worked.
John
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
