Fix beep: flush the output buffer

[blueyed]

It works when using it through --onfail 'python -c "print(\"\a\", end=\"\")";', but not with the default options (without --nobeep).

I am not sure why that is required?!

I am using Python 3.5.0.

[blueyed]

Oh, it does not work: I was still using the bell char in --onfail.

So, don't merge this, but it should be an issue instead.

[joeyespo]

Ha! I just ran into this same issue in http://github.com/joeyespo/py-getch. Nice find.

The print(BEEP_CHARACTER, end='') line can actually stay as-is; this works across Python versions. Python 3.x added a flush argument to flush the output, but Python 2.7 doesn't have this, so the workaround is follow it by sys.stdout.flush() like you have.

# Works on both Python 2.7 and 3.x
print(BEEP_CHARACTER, end='')
sys.stdout.flush()

Is this ready?

[blueyed]

Yes, should be ready.
I've found out about the (too new) flush argument also, and when using sys.stdout.flush I've replaced the print also - seems to be more consistent.
Do you think it should be kept as print? Then feel free to amend when merging.

[joeyespo]

Gotcha. Yeah, will probably keep print to follow "There should be one-- and preferably only one --obvious way to do it" -- in this case, print out text. Someday the flush call can be replaced with a flush argument. (Probably years though haha.)

Thanks for the fix 😃