Fix bug where multiprocessing.Queue hangs on large outputs.

Fix bug where multiprocessing.Queue hangs on large outputs.

Change-Id: Ia504a0518f7550f764be9ff05bdbd7749d065c65

BUG=chromium-os:14055
TEST=Ran through with ctest and a mario build I built today!

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=09254c2

[dgarrett, 2011-04-12 02:01:52]

LGTM with some nits. The style comments are just about how I would do it.

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
File au_test_harness/cros_au_test_harness.py (right):

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:83: return_codes):
off by one indention.

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:89: key_line_re =
re.compile('^PREGENERATED_UPDATE=([\w/.]+)')
Can do this outside the outer for loop and not regenerate for each file.

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:90: if return_code != 0:
Seems like this line should be the first line after the for loop on line 82, and
that it doesn't need an else clause at all. Just unindent the normal case code.

[sosa, 2011-04-12 02:09:50]

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
File au_test_harness/cros_au_test_harness.py (right):

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:83: return_codes):
On 2011/04/12 02:01:52, dgarrett wrote:
> off by one indention.

Done.

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:89: key_line_re =
re.compile('^PREGENERATED_UPDATE=([\w/.]+)')
On 2011/04/12 02:01:52, dgarrett wrote:
> Can do this outside the outer for loop and not regenerate for each file.

Done.

http://codereview.chromium.org/6825062/diff/4/au_test_harness/cros_au_test_ha...
au_test_harness/cros_au_test_harness.py:90: if return_code != 0:
I actually want to print out the output from a failed update (Warning(output))
so i need to get output first.

On 2011/04/12 02:01:52, dgarrett wrote:
> Seems like this line should be the first line after the for loop on line 82,
and
> that it doesn't need an else clause at all. Just unindent the normal case
code.