On 05/ 3/10 04:01 PM, Sue Sohn wrote:
On 05/03/10 14:43, jean.mccormack wrote:
On 05/ 3/10 03:20 PM, Sue Sohn wrote:
Could I please get a review of the changes to fix:
15840 - Entering a 0 for the month and day cause text installer
traceback
http://defect.opensolaris.org/bz/show_bug.cgi?id=15840
Webrev at:
http://cr.opensolaris.org/~sohn/15840/
Thanks,
Sue
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
Sue,
I'm curious as to why if the month is 0 you return 31 days and not
raise an error?
I can see that elsewhere in the code this check is done, just
wondering why not here.
Jean
If the user enters a 0 for the month, an error is indeed displayed for
that field. However, if the user then moves down to the day field
before resolving that error, this code (which calculates the number of
days in the month for the purposes of range checking) needs to be able
to handle the fact that the month is currently 0.
Sue
Thanks for the explanation.
Jean
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
