lkml.org 
[lkml]   [2009]   [Oct]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH tracing/kprobes v2 1/5] tracing/kprobes: Rename special variables syntax
On Mon, Oct 05, 2009 at 08:12:53PM -0400, Steven Rostedt wrote:
> On Mon, 2009-10-05 at 21:26 +0200, Frederic Weisbecker wrote:
> > On Mon, Oct 05, 2009 at 12:59:01PM -0400, Masami Hiramatsu wrote:
> > > As far as I can see in arch/*/include/asm/ptrace.h, all registers start with
> > > alphabets :-). So, I'd like to suggest renaming sp-vars to '_sp-vars'.
> > >
> > > Then, we will have;
> > > - $local-vars
> >
> >
> > There is a risk of bash collision.
>
> I actually prefer the "$" notation. As for bash collision, it is common
> for shell script writers to be able to distinguish a variable from bash.
> Yes we can backslash it, or quote it. But when I see a $var it sticks
> out to me that it is a variable. It's not hard to get around. For
> example, type:
>
> $ echo "hello $DISPLAY"' or $DISPLAY'
>
> and see what you get.
>
> Makefiles and Perl use '$' for variables those that need to handle it
> with bash can easily cope with it.
>
> So my vote is to keep the '$'. It is the most intuitive to what it
> means.


Hrrmm...

I fear about future complains, but I may be somehow biased in
that my usual usecases of bash don't involve '$' characters to
protect, so it's not something I'm used to.

Whatever choice we make, there are either downsides in the prefix
self meaning, the collisions or the shell intrpretation.

Now you are two who prefer that, let's pick this one :)




\
 
 \ /
  Last update: 2009-10-07 00:53    [W:0.350 / U:0.804 seconds]
©2003-2014 Jasper Spaans. Advertise on this site