Re: [PATCH 2/3] ASoC: wm9713: Use empty struct initializer

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 14/02/18 17:01, Fabio Estevam wrote:
On Wed, Feb 14, 2018 at 2:56 PM, Richard Fitzgerald
<rf@xxxxxxxxxxxxxxxxxxxxx> wrote:

I agree but your commit message didn't say that. It's irrelevant now
Mark has merged the patch, but for anyone looking at the commit message
later, the structure of your commit message implies that the problem
is that {0} doesn't work correctly (probably untrue and certainly not

Yes, and that is the message I want to give:

{0} does not work correctly if the first member of the struct is a
pointer, as we should not explicitly write 0 to pointers.


I gather that, so I'd have liked to your commit message to have said that. But what you actually said was

"{ 0 } only clears the first member of the structure."

which is a very different message.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux