On Sat, Jul 23, 2011 at 02:12, Linux Kernel Mailing List
<[email protected]> wrote:
> Gitweb: http://git.kernel.org/linus/1d738e64f3d957d56c1b51e64ebdef986a8760e3
> Commit: 1d738e64f3d957d56c1b51e64ebdef986a8760e3
> Parent: 0cd20a278e1ef9da9f6a987942794c9d65af8c4d
> Author: Rafał Miłecki <[email protected]>
> AuthorDate: Thu Jul 7 15:25:27 2011 +0200
> Committer: John W. Linville <[email protected]>
> CommitDate: Fri Jul 8 11:11:25 2011 -0400
> diff --git a/drivers/net/wireless/b43/phy_lcn.h b/drivers/net/wireless/b43/phy_lcn.h
> new file mode 100644
> index 0000000..c046c2a
> --- /dev/null
> +++ b/drivers/net/wireless/b43/phy_lcn.h
> \ No newline at end of file
Woops...
gcc/cpp also complains:
drivers/net/wireless/b43/phy_lcn.h:14:28: warning: no newline at end of file
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
W dniu 8 sierpnia 2011 18:07 użytkownik Gábor Stefanik
<[email protected]> napisał:
> 2011/8/8 Rafał Miłecki <[email protected]>:
>> W dniu 8 sierpnia 2011 13:57 użytkownik Geert Uytterhoeven
>> <[email protected]> napisał:
>>> On Sat, Jul 23, 2011 at 02:12, Linux Kernel Mailing List
>>> <[email protected]> wrote:
>>>> Gitweb: http://git.kernel.org/linus/1d738e64f3d957d56c1b51e64ebdef986a8760e3
>>>> Commit: 1d738e64f3d957d56c1b51e64ebdef986a8760e3
>>>> Parent: 0cd20a278e1ef9da9f6a987942794c9d65af8c4d
>>>> Author: Rafał Miłecki <[email protected]>
>>>> AuthorDate: Thu Jul 7 15:25:27 2011 +0200
>>>> Committer: John W. Linville <[email protected]>
>>>> CommitDate: Fri Jul 8 11:11:25 2011 -0400
>>>
>>>> diff --git a/drivers/net/wireless/b43/phy_lcn.h b/drivers/net/wireless/b43/phy_lcn.h
>>>> new file mode 100644
>>>> index 0000000..c046c2a
>>>> --- /dev/null
>>>> +++ b/drivers/net/wireless/b43/phy_lcn.h
>>>
>>>> \ No newline at end of file
>>>
>>> Woops...
>>>
>>> gcc/cpp also complains:
>>>
>>> drivers/net/wireless/b43/phy_lcn.h:14:28: warning: no newline at end of file
>>
>> Sorry, I didn't know we want empty lines at the end. I also don't
>> think it's mentioned in kernel's CodingStyle.
>
> I'm pretty sure it is basic Unix knowledge that all text files SHOULD
> (RFC 2119) end with a newline character, unless otherwise noted
> explicitly. In fact, many Unix text editors automatically insert a
> newline at the end, and forbid you from deleting it.
OK, thanks, will avoid that in the future.
--
Rafał
2011/8/8 Rafa? Mi?ecki <[email protected]>:
> W dniu 8 sierpnia 2011 13:57 u?ytkownik Geert Uytterhoeven
> <[email protected]> napisa?:
>> On Sat, Jul 23, 2011 at 02:12, Linux Kernel Mailing List
>> <[email protected]> wrote:
>>> Gitweb: ? ? http://git.kernel.org/linus/1d738e64f3d957d56c1b51e64ebdef986a8760e3
>>> Commit: ? ? 1d738e64f3d957d56c1b51e64ebdef986a8760e3
>>> Parent: ? ? 0cd20a278e1ef9da9f6a987942794c9d65af8c4d
>>> Author: ? ? Rafa? Mi?ecki <[email protected]>
>>> AuthorDate: Thu Jul 7 15:25:27 2011 +0200
>>> Committer: ?John W. Linville <[email protected]>
>>> CommitDate: Fri Jul 8 11:11:25 2011 -0400
>>
>>> diff --git a/drivers/net/wireless/b43/phy_lcn.h b/drivers/net/wireless/b43/phy_lcn.h
>>> new file mode 100644
>>> index 0000000..c046c2a
>>> --- /dev/null
>>> +++ b/drivers/net/wireless/b43/phy_lcn.h
>>
>>> \ No newline at end of file
>>
>> Woops...
>>
>> gcc/cpp also complains:
>>
>> drivers/net/wireless/b43/phy_lcn.h:14:28: warning: no newline at end of file
>
> Sorry, I didn't know we want empty lines at the end. I also don't
> think it's mentioned in kernel's CodingStyle.
I'm pretty sure it is basic Unix knowledge that all text files SHOULD
(RFC 2119) end with a newline character, unless otherwise noted
explicitly. In fact, many Unix text editors automatically insert a
newline at the end, and forbid you from deleting it.
>
> I'll fix that when doing LCN stuff, if you agree separated patch is not needed.
>
> My gcc didn't complain :|
>
> --
> Rafa?
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to [email protected]
> More majordomo info at ?http://vger.kernel.org/majordomo-info.html
>
--
Vista: [V]iruses, [I]ntruders, [S]pyware, [T]rojans and [A]dware. :-)
W dniu 8 sierpnia 2011 13:57 użytkownik Geert Uytterhoeven
<[email protected]> napisał:
> On Sat, Jul 23, 2011 at 02:12, Linux Kernel Mailing List
> <[email protected]> wrote:
>> Gitweb: http://git.kernel.org/linus/1d738e64f3d957d56c1b51e64ebdef986a8760e3
>> Commit: 1d738e64f3d957d56c1b51e64ebdef986a8760e3
>> Parent: 0cd20a278e1ef9da9f6a987942794c9d65af8c4d
>> Author: Rafał Miłecki <[email protected]>
>> AuthorDate: Thu Jul 7 15:25:27 2011 +0200
>> Committer: John W. Linville <[email protected]>
>> CommitDate: Fri Jul 8 11:11:25 2011 -0400
>
>> diff --git a/drivers/net/wireless/b43/phy_lcn.h b/drivers/net/wireless/b43/phy_lcn.h
>> new file mode 100644
>> index 0000000..c046c2a
>> --- /dev/null
>> +++ b/drivers/net/wireless/b43/phy_lcn.h
>
>> \ No newline at end of file
>
> Woops...
>
> gcc/cpp also complains:
>
> drivers/net/wireless/b43/phy_lcn.h:14:28: warning: no newline at end of file
Sorry, I didn't know we want empty lines at the end. I also don't
think it's mentioned in kernel's CodingStyle.
I'll fix that when doing LCN stuff, if you agree separated patch is not needed.
My gcc didn't complain :|
--
Rafał
On Mon, Aug 08, 2011 at 06:07:07PM +0200, Gábor Stefanik wrote:
> 2011/8/8 Rafał Miłecki <[email protected]>:
> > W dniu 8 sierpnia 2011 13:57 użytkownik Geert Uytterhoeven
> > <[email protected]> napisał:
> >> On Sat, Jul 23, 2011 at 02:12, Linux Kernel Mailing List
> >> <[email protected]> wrote:
> >>> Gitweb: http://git.kernel.org/linus/1d738e64f3d957d56c1b51e64ebdef986a8760e3
> >>> Commit: 1d738e64f3d957d56c1b51e64ebdef986a8760e3
> >>> Parent: 0cd20a278e1ef9da9f6a987942794c9d65af8c4d
> >>> Author: Rafał Miłecki <[email protected]>
> >>> AuthorDate: Thu Jul 7 15:25:27 2011 +0200
> >>> Committer: John W. Linville <[email protected]>
> >>> CommitDate: Fri Jul 8 11:11:25 2011 -0400
> >>
> >>> diff --git a/drivers/net/wireless/b43/phy_lcn.h b/drivers/net/wireless/b43/phy_lcn.h
> >>> new file mode 100644
> >>> index 0000000..c046c2a
> >>> --- /dev/null
> >>> +++ b/drivers/net/wireless/b43/phy_lcn.h
> >>
> >>> \ No newline at end of file
> >>
> >> Woops...
> >>
> >> gcc/cpp also complains:
> >>
> >> drivers/net/wireless/b43/phy_lcn.h:14:28: warning: no newline at end of file
> >
> > Sorry, I didn't know we want empty lines at the end. I also don't
> > think it's mentioned in kernel's CodingStyle.
>
> I'm pretty sure it is basic Unix knowledge that all text files SHOULD
> (RFC 2119) end with a newline character, unless otherwise noted
> explicitly. In fact, many Unix text editors automatically insert a
> newline at the end, and forbid you from deleting it.
Please use an editor that does this appropriately, but do _not_
simply add an empty line at the end of a file. 'git am' complains
about empty lines at the end of a file during a commit.
I find that simple loading and saving such files with vi is sufficient
for me -- YMMV.
John
--
John W. Linville Someday the world will need a hero, and you
[email protected] might be all we have. Be ready.