lkml.org 
[lkml]   [2022]   [Dec]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v17 3/3] EDAC/npcm: Add NPCM memory controller driver
On Mon, Dec 26, 2022 at 11:50:54AM +0800, Kun-Fa Lin wrote:
> > > + syndrome = priv->location ? 1 << priv->bit :
> > > + data_synd[priv->bit];
> >
> > syndrome = priv->location ? 1 << priv->bit
> > : data_synd[priv->bit];
>
> Just to confirm the indentation, is it right as follows?
>
> syndrome = priv->location ? 1 << priv->bit
> : data_synd[priv->bit];
>
> And I was wondering if I should just remove the line break and let it stick out?

The idea is to have the '?' and the ':' under each other so that one
can visually immediately "parse" where each of the sides of the ternary
statement start.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette

\
 
 \ /
  Last update: 2023-03-26 23:20    [W:0.109 / U:1.328 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site