lkml.org 
[lkml]   [2014]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH] scripts: Have make TAGS not include structure members
On Tue, Nov 19, 2013 at 07:34:14PM -0500, Steven Rostedt wrote:
> I wasn't going to use this change, but instead add this change:
[...]
> --- linux-trace.git.orig/scripts/tags.sh 2013-11-19 19:29:34.352371615 -0500
> +++ linux-trace.git/scripts/tags.sh 2013-11-19 19:32:37.821152204 -0500
> @@ -218,7 +218,7 @@ exuberant()
>
> emacs()
> {
> - all_target_sources | xargs $1 -a \
> + all_target_sources | xargs $@ -a \
> --regex='/^\(ENTRY\|_GLOBAL\)(\([^)]*\)).*/\2/' \
> --regex='/^SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/sys_\1/' \
> --regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1/' \
> @@ -263,7 +263,7 @@ xtags()
> if $1 --version 2>&1 | grep -iq exuberant; then
> exuberant $1
> elif $1 --version 2>&1 | grep -iq emacs; then
> - emacs $1
> + emacs $@
> else
> all_target_sources | xargs $1 -a
> fi
> @@ -313,7 +313,7 @@ case "$1" in
>
> "TAGS")
> rm -f TAGS
> - xtags etags
> + xtags etags --no-members
> remove_structs=y


The logic looks OK, but it is not completely obvious that $@ means
'etags --no-members, iff called as etags'. How about using a more
descriptive variable like this?

diff --git a/scripts/tags.sh b/scripts/tags.sh
index 58c4559..3afc479 100755
--- a/scripts/tags.sh
+++ b/scripts/tags.sh
@@ -219,7 +219,7 @@ exuberant()

emacs()
{
- all_target_sources | xargs $1 -a \
+ all_target_sources | xargs $1 $no_members -a \
--regex='/^\(ENTRY\|_GLOBAL\)(\([^)]*\)).*/\2/' \
--regex='/^SYSCALL_DEFINE[0-9]?(\([^,)]*\).*/sys_\1/' \
--regex='/^TRACE_EVENT(\([^,)]*\).*/trace_\1/' \
@@ -308,12 +308,15 @@ case "$1" in

"tags")
rm -f tags
+ no_members=
xtags ctags
remove_structs=y
;;

"TAGS")
rm -f TAGS
+ # Do not index struct members when generating Emacs TAGS
+ no_members=--no-members
xtags etags
remove_structs=y
;;
Michal


\
 
 \ /
  Last update: 2014-01-03 15:21    [W:0.078 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site