[ltt-dev] [PATCH 4/4] Define string arguments as const for trace control functions

[Mathieu Desnoyers]

* Mathieu Desnoyers (compudj at krystal.dyndns.org) wrote:
> * Zhaolei (zhaolei at cn.fujitsu.com) wrote:
[...]
> > @@ -714,7 +714,7 @@ static int ltt_trace_stop(char *trace_name)
> >   * @trace_type : Type of trace (normal, flight, hybrid)
> >   * @args : Arguments specific to the action
> >   */
> > -int ltt_control(enum ltt_control_msg msg, char *trace_name, char *trace_type,
> > +int ltt_control(enum ltt_control_msg msg, const char *trace_name, const char *trace_type,
> >  		union ltt_control_args args)
> >  {
> >  	int err = EPERM;


compudj at ok:~/git/morestable/linux-2.6-lttng$ scripts/checkpatch.pl patches/`quilt top`
WARNING: line over 80 characters
#97: FILE: ltt/ltt-tracer.c:698:
+int ltt_control(enum ltt_control_msg msg, const char *trace_name, const char *trace_type,

total: 0 errors, 1 warnings, 82 lines checked

I fixed the patch in my tree. I just email this as a reminder to use
checkpatch ;) (it looks like Andrew Morton eventually converted me to
checkpatch zealotness after all)

Mathieu

-- 
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68