Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread Luis R. Rodriguez
On Mon, Oct 06, 2014 at 11:06:34PM +0200, SF Markus Elfring wrote:
> >> How do you think about to work with more data type definitions for such
> >> callback functions?
> > 
> > Sorry I don't understand what you mean.
> 
> Can a specific "typedef" help in corresponding software maintenance?
> 
> Do you find descriptions from an other software application useful for such an
> use case here?
> http://www.fltk.org/doc-1.3/group__callback__functions.html

Thanks I've reviewed this but I prefer to avoid typedefs.

  Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread SF Markus Elfring
>> How do you think about to work with more data type definitions for such
>> callback functions?
> 
> Sorry I don't understand what you mean.

Can a specific "typedef" help in corresponding software maintenance?

Do you find descriptions from an other software application useful for such an
use case here?
http://www.fltk.org/doc-1.3/group__callback__functions.html

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread Luis R. Rodriguez
On Sat, Oct 04, 2014 at 02:55:38PM +0200, SF Markus Elfring wrote:
> > This adds an extra argument onto parse_params() to be used
> > as a way to make the unused callback a bit more useful and
> > generic by allowing the caller to pass on a data structure
> > of its choice.
> 
> How do you think about to work with more data type definitions for such 
> callback
> functions?

Sorry I don't understand what you mean.

  Luis
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread Luis R. Rodriguez
On Sat, Oct 04, 2014 at 02:55:38PM +0200, SF Markus Elfring wrote:
  This adds an extra argument onto parse_params() to be used
  as a way to make the unused callback a bit more useful and
  generic by allowing the caller to pass on a data structure
  of its choice.
 
 How do you think about to work with more data type definitions for such 
 callback
 functions?

Sorry I don't understand what you mean.

  Luis
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread SF Markus Elfring
 How do you think about to work with more data type definitions for such
 callback functions?
 
 Sorry I don't understand what you mean.

Can a specific typedef help in corresponding software maintenance?

Do you find descriptions from an other software application useful for such an
use case here?
http://www.fltk.org/doc-1.3/group__callback__functions.html

Regards,
Markus
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-06 Thread Luis R. Rodriguez
On Mon, Oct 06, 2014 at 11:06:34PM +0200, SF Markus Elfring wrote:
  How do you think about to work with more data type definitions for such
  callback functions?
  
  Sorry I don't understand what you mean.
 
 Can a specific typedef help in corresponding software maintenance?
 
 Do you find descriptions from an other software application useful for such an
 use case here?
 http://www.fltk.org/doc-1.3/group__callback__functions.html

Thanks I've reviewed this but I prefer to avoid typedefs.

  Luis
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-04 Thread SF Markus Elfring
> This adds an extra argument onto parse_params() to be used
> as a way to make the unused callback a bit more useful and
> generic by allowing the caller to pass on a data structure
> of its choice.

How do you think about to work with more data type definitions for such callback
functions?

Regards,
Markus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Cocci] [PATCH v2 2/7] module: add extra argument for parse_params() callback

2014-10-04 Thread SF Markus Elfring
 This adds an extra argument onto parse_params() to be used
 as a way to make the unused callback a bit more useful and
 generic by allowing the caller to pass on a data structure
 of its choice.

How do you think about to work with more data type definitions for such callback
functions?

Regards,
Markus
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/