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

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 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 this makes

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

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 a

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 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

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