Re: [9front] displayport thinkpad x230 external monitor black screen

[unobe@cpan.org]

Quoth qwx@sciops.net:
> Alright, since there are no reports for breakage, I suggest that we --
> you or me, as you prefer -- make a few minor edits as mentioned, make
> a new diff and just push it.  Apologies for the delays!

No apology needed: you're the one helping me!

Of the issues you brought up, I was certain about how to fix the 'i =
1;' -> 'break;', but less certain of how to match the surrounding code
in all cases.  The for loops spacing was easy enough, and the added
space at the end of a for loop that was unneeded (i.e., hunk @@
-2319,7, +2363,7 @@).

However, the shift operators I saw have both styles even within the
same block.  In some cases there were surrounding spaces, and in
others there weren't (e.g., hunk @@-2219,46 +2300,37 @@).  I don't
want to delay the push of the patch needlessly, but also don't want to
create disturbances in the Force by not catching the ones you noticed.
So I'd prefer you to fixup as you see fit.

Sincerely,
Romano

P.S. Thanks for also pushing the /lib/dict change!