Re: [LTP] [PATCH v4] lib: tst_module: add library functions for kernel modules

alexey.kodanev
2013-07-01T08:31:23Z

On 06/28/2013 09:35 AM, Mike Frysinger wrote:
> On Thursday 27 June 2013 11:09:22 Alexey Kodanev wrote:
> \> +void tst_module_load(void (cleanup_fn)(void),
>> + const char *mod_name, char *const argv[])
>> +{
>> + char *mod_path = NULL;
>> + tst_module_exists(cleanup_fn, mod_name,&mod_path);
>> +
>> + static int offset = 2; /* command name& module path */
> why static ? i would use const instead.
> -mike
Yes, const must be used here, and static is not needed for an int. I
will fix it.

Thanks,
Alexey