Re: [dev] [PATCH st] Make DECSCUSR thickness configurable

Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]

From: David Phillips <dbphillipsnz_AT_gmail.com>
Date: Fri, 10 Apr 2015 15:57:57 +1200

>> - switch (xw.cursor) {
>> + switch(xw.cursor) {
>
> I don't think feature patches should include whitespace style changes.

Especially if the change goes against the style guide :)

-- 
Four word witty remark

Received on Fri Apr 10 2015 - 05:57:57 CEST

This message: [ Message body ]
Next message: David Phillips: "[dev] [dwm] Style changes"
Previous message: Eric Pruitt: "Re: [dev] [PATCH st] Make DECSCUSR thickness configurable"
In reply to: Eric Pruitt: "Re: [dev] [PATCH st] Make DECSCUSR thickness configurable"
Next in thread: Roberto E. Vargas Caballero: "Re: [dev] [PATCH st] Make DECSCUSR thickness configurable"

Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]

This archive was generated by hypermail 2.3.0 : Fri Apr 10 2015 - 06:00:11 CEST