Re: [PATCH] staging: sm750fb: convert nDirection to snake_case
From: Dan Carpenter
Date: Thu Feb 12 2026 - 00:36:05 EST
On Thu, Feb 12, 2026 at 12:24:44AM +0200, Rachel Ovrani wrote:
> Thank you for the review!
>
> The 'n_' prefix stands for "new" - the variable holds a newly calculated
> direction value based on the overlap between source and destination regions.
>
> That said, if you think 'direction' would be clearer, I'm happy to submit a
> v2 removing the prefix.
I would have never guessed it was new_. Normally n_ means number and I
couldn't think of any meaning in Hungarian notation. Naming it
"new_direction" is even better.
regards,
dan carpenter