Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-09 Thread Maxim Levitsky
On Wed, 2010-09-08 at 23:12 +0200, David Härdeman wrote: On Wed, Sep 08, 2010 at 11:10:40AM -0400, Andy Walls wrote: Tag files and a decent editor are all one needs for full code navigation. The kernel makefile already has a tags target to make the tags file. If you like to use

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread Mauro Carvalho Chehab
Em 07-09-2010 18:51, David Härdeman escreveu: This patch merges the files which makes up ir-core and renames the resulting module to rc-core. IMHO this makes it much easier to hack on the core module since all code is in one file. This also allows some simplification of ir-core-priv.h as

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread Jarod Wilson
On Wed, Sep 08, 2010 at 10:42:10AM -0300, Mauro Carvalho Chehab wrote: Em 07-09-2010 18:51, David Härdeman escreveu: This patch merges the files which makes up ir-core and renames the resulting module to rc-core. IMHO this makes it much easier to hack on the core module since all code is in

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread Andy Walls
Tag files and a decent editor are all one needs for full code navigation. The kernel makefile already has a tags target to make the tags file. Smaller files make for better logical isolation of functions, limiting visibilty/scope, and faster compilation of a file (but maybe at the expense of

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread David Härdeman
On Wed, Sep 08, 2010 at 11:10:40AM -0400, Andy Walls wrote: Tag files and a decent editor are all one needs for full code navigation. The kernel makefile already has a tags target to make the tags file. If you like to use tags, it won't be affected by many or few files so it's not an

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread David Härdeman
On Wed, Sep 08, 2010 at 10:42:10AM -0300, Mauro Carvalho Chehab wrote: Em 07-09-2010 18:51, David Härdeman escreveu: This patch merges the files which makes up ir-core and renames the resulting module to rc-core. IMHO this makes it much easier to hack on the core module since all code is in

Re: [PATCH 1/5] rc-code: merge and rename ir-core

2010-09-08 Thread Jarod Wilson
On Wed, Sep 8, 2010 at 5:42 PM, David Härdeman da...@hardeman.nu wrote: On Wed, Sep 08, 2010 at 10:42:10AM -0300, Mauro Carvalho Chehab wrote: Em 07-09-2010 18:51, David Härdeman escreveu: This patch merges the files which makes up ir-core and renames the resulting module to rc-core. IMHO