Re: [hackers] Repeated usage of m->lt[m->sellt]->arrange deduplication

From: Hiltjo Posthuma <hiltjo_AT_codemadness.org>
Date: Mon, 24 Apr 2017 08:44:12 +0200

On Mon, Apr 24, 2017 at 10:14:36AM +1000, Daniel Cousens wrote:
> Indeed, apologies for the style issues.
> The argument could probably be const too.
>
> On 24 Apr. 2017 04:32, "Laslo Hunhold" <[1]dev_AT_frign.de> wrote:
>
> On Sun, 23 Apr 2017 20:14:01 +0200
> Hiltjo Posthuma <[2]hiltjo_AT_codemadness.org> wrote:
> Hey Hiltjo,
> wouldn't it also be better to remove the space between function name
> and list of arguments? The current patch is inconsistent with the
> dwm-style.
> > > +int canarrange (Monitor* m) {
> -->
> > > +int
> > > +canarrange(Monitor* m)
> With best regards
> Laslo
> --
> Laslo Hunhold <[3]dev_AT_frign.de>
>
> References
>
> 1. mailto:dev_AT_frign.de
> 2. mailto:hiltjo_AT_codemadness.org
> 3. mailto:dev_AT_frign.de

Don't make it const, but agree with the rest.

-- 
Kind regards,
Hiltjo
Received on Mon Apr 24 2017 - 08:44:12 CEST

This archive was generated by hypermail 2.3.0 : Mon Apr 24 2017 - 08:48:16 CEST