On 07/17/12 02:12 PM, Swati Sarraf wrote:
Hi,
Thanks for the review. please find the updated webrev:
https://cr.opensolaris.org/action/browse/caiman/ssarraf/7183018/
Regarding the indentation of line 655: I saw two different indentation method
in line 679 and line 665. which one is correct?
679 is better in this instance, since you have space on the line.
Sue
----- Original Message -----
From: [email protected]
To: [email protected]
Cc: [email protected]
Sent: Tuesday, July 17, 2012 1:45:06 PM GMT -08:00 US/Canada Pacific
Subject: Re: [caiman-discuss] Code Review request: 7183018
On 07/17/12 11:47 AM, Swati Sarraf wrote:
Hi All,
Can I have a code review for the following bug fix:
7183018 Gui livecd is not using size from image.info but using fallback image
size
http://monaco.us.oracle.com/detail.jsf?cr=7183018
Webrev link:
https://cr.opensolaris.org/action/browse/caiman/ssarraf/7183018/
Testing is done as follow:
1. Slim test done
Ran 1901 tests in 353.236s, FAILED (SKIP=1, failures=1)
failed test: Tests package with understood add option
Slim test pointer: file:///net/osol-bldx/datapool/ssarraf/7183018/
2. Pylint and pep8 test done , result OK
pylint result pointer: file:///net/osol-bldx/datapool/ssarraf/7183018/
3 Created DC image and tried installation with minimum size (4.7GB).
Installation finished successfully.
-Thanks
Swati Sarraf
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
Hi Swati,
I had all the same comments as Karen, plus a few more nits:
651 No need for backslash.
652 Should be indented so that "Size.mb_units is lined up with str on line above
654
if its fail
->
if it fails
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
