Sue,

I didn't know that we could have unbounded parameters.  Live and learn.  ;-)

Why not use copy to duplicate the entries in mancriteria to crit_dict?

 806     # create dict from sql
 807     for key in mancriteria.keys():
 808         crit_dict[key] = mancriteria[key]

This would look like:

    import copy
    ...
    crit_dict = copy.copy(mancriteria)

You could also get rid of line 803:

 803     crit_dict = {}

because this would be handled in the copy.copy() line.

Otherwise things look good.

Thanks,

John


On 11/19/10 10:38 AM, Sue Sohn wrote:
Could I please get a review of the changes for:

7001284 installadm list can have incorrect output if criteria has MIN or MAX unbounded

Webrev:
http://cr.opensolaris.org/~sohn/7001284

Thanks,
Sue
_______________________________________________
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