On 08/18/10 05:01 PM, Dermot McCluskey wrote:
Sarah Jelinek wrote:
No, I changed them all to True|False.

That will require code changes in the AI client code.
There are several places where it does a case-sensitive
strcmp() of these values.

ok, I will revert back then. Thanks for letting me know.

sarah
*****
- Dermot




On 08/18/10 11:42 AM, Dermot McCluskey wrote:
Sarah,

Are you saying that you will change the property/val and facet/set
attributes to take capitalized True/False but leave all the other boolean
attributes in the schema to take lower case true/false?

I think that would be confusing for users.


- Dermot




On 08/18/10 17:40, Sarah Jelinek wrote:


On 08/18/10 10:34 AM, jean.mccormack wrote:
ai_manifest.xml:
line 235: There's a closing search_all but I can't find the opening one.

It's in the latest one. I fixed this after the webrev.
software.dtd:
lines 70-71: I discovered that it would be preferred to specify "True" rather than "true". I do have code to deal with this but since IPS doesn't see them as the same it would be nice to spec what IPS does understand just in case.

I can change this. And the one below.

sarah
line 85: Again, would prefer True/False but I can deal with true/false.


_______________________________________________
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


_______________________________________________
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

Reply via email to