Re: [PATCH] Add 2-pass mode to gtags

[Hideki IWAMOTO]

Hi.
On Fri, 22 Jan 2010 11:21:10 +0900, Shigio YAMAGUCHI wrote...
> How about saying what the user should do?  For example:
> 
> gtags: The format of GRTAGS and GSYMS is different.
>       |
>       v
> gtags: The format of GRTAGS and GSYMS is different.\n  Please invoke gtags 
> again without the -i option.

The error message was changed as shown in your example. 

$ gtags
$ cp GTAGS GSYMS
$ gtags -i
gtags: The format of GRTAGS and GSYMS is different.
  Please invoke gtags again without the -i option.
$ rm GSYMS
$ gtags -i
gtags: GSYMS doesn't exist though GRTAGS exists.
  Please invoke gtags again without the -i option.
$ mv GRTAGS GSYMS
$ gtags -i
gtags: GRTAGS doesn't exist though GSYMS exists.
  Please invoke gtags again without the -i option.
$ rm GSYMS
$ gtags -i

Other changes:
  - Stopped use of defined() in gtags and global. 
  - The output of gtags --stastics was changed as follows. 
$ foreach cachesize ( 8388608 33554432 134217728 )
foreach? echo ==== cachesize:$cachesize ====
foreach? rm -fr linux-2.6.32; tar xfj ~/download/linux/linux-2.6.32.tar.bz2; 
sync
foreach? (cd linux-2.6.32; env GTAGSCACHE=$cachesize gtags --statistics)
foreach? end
==== cachesize:8388608 ====
period                                    user[sec] system[sec] elapsed[sec] 
%CPU
----------------------------------------- --------- ----------- ------------ 
----
Time of creating GTAGS and temporary file    25.066       3.352       39.789 
71.4
Time of creating GRTAGS and GSYMS            15.309      10.157      140.749 
18.1
Time of flushing B-tree cache                 0.000       0.216        7.199  
3.0
----------------------------------------- --------- ----------- ------------ 
----
The entire time                              40.375      13.725      187.737 
28.8
==== cachesize:33554432 ====
period                                    user[sec] system[sec] elapsed[sec] 
%CPU
----------------------------------------- --------- ----------- ------------ 
----
Time of creating GTAGS and temporary file    25.262       2.432       29.818 
92.9
Time of creating GRTAGS and GSYMS            17.169       2.892       62.196 
32.3
Time of flushing B-tree cache                 0.012       0.316        9.892  
3.3
----------------------------------------- --------- ----------- ------------ 
----
The entire time                              42.443       5.640      101.906 
47.2
==== cachesize:134217728 ====
period                                    user[sec] system[sec] elapsed[sec]  
%CPU
----------------------------------------- --------- ----------- ------------ 
-----
Time of creating GTAGS and temporary file    25.202       1.940       27.679  
98.1
Time of creating GRTAGS and GSYMS            17.497       0.464       17.966 
100.0
Time of flushing B-tree cache                 0.032       1.324       17.857   
7.6
----------------------------------------- --------- ----------- ------------ 
-----
The entire time                              42.731       3.728       63.502  
73.2


Diffstat:
 configure.ac              |   11
 global/global.c           |  151 +++++++++
 gtags.conf.in             |    5
 gtags/gtags.c             |  400 +++++++++++++++++++++++---
 gtags/manual.in           |    4
 libutil/.cvsignore        |    4
 libutil/C.c               |  664 +++++++++++++++++++++++++++++++++++++++++++
 libutil/Cpp.c             |  706 ++++++++++++++++++++++++++++++++++++++++++++++
 libutil/Makefile.am       |    8
 libutil/asm_parse.y       |  166 ++++++++++
 libutil/asm_scan.l        |  210 +++++++++++++
 libutil/global.h          |    1
 libutil/gtagsop.c         |  200 ++++++++++++-
 libutil/gtagsop.h         |   11
 libutil/java.c            |  163 ++++++++++
 libutil/parser.c          |  277 ++++++++++++++++++
 libutil/parser.h          |   53 +++
 libutil/parser_internal.h |   52 +++
 libutil/php.l             |  357 +++++++++++++++++++++++
 reconf.sh                 |   13
 20 files changed, 3394 insertions(+), 62 deletions(-)

----
Hideki IWAMOTO  address@hidden

20100125-builtin-parser+1pass-parsing.patch
Description: Binary data