[pacman-dev] [PATCH 3/3] Align titles automatically in information display

Pierre Neidhardt ambrevar at gmail.com
Sat Oct 17 15:44:20 UTC 2015


Thanks for all the good tips!

On 15-10-17 09:11:02, Dave Reisner wrote:
> Nit: it'd be more clear to use TITLE_ as a prefix instead of T_.

These constants are used to index the 'titles' variable. Having a TITLE prefix
would make it look like

	titles[TITLE_DESCRIPTION]

which I find a bit redundant. It also avoids potential future clashes with
TITLE_MAXLEN and TITLE_COUNT. (I removed the other TITLE_* variables as you
suggested.)

> > +	/* make aligned titles once only */
> > +	static int need_alignment = 1;
> > +	if(need_alignment) {
> > +		need_alignment = 0;
> > +		make_aligned_titles();
> > +	}
>
> Why not just have make_aligned_titles() return a dummy int value? Then,
> you can:
>
>   static const int make_aligned_once = make_aligned_titles();

I don't think this would work: static variables are initialized at compile time
if I'm not mistaken.

--
Pierre Neidhardt


More information about the pacman-dev mailing list