[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] add option --file to gtags
From: |
Hideki IWAMOTO |
Subject: |
Re: [PATCH] add option --file to gtags |
Date: |
Mon, 01 Aug 2005 00:17:20 +0900 |
On Tue, 26 Jul 2005 20:39:05 +0900, Shigio Yamaguchi wrote...
> > - When the specified file does not exist (realpath failed), exit as failure.
>
> I think it is enough to ignore such files with warning, because they are
> not harmful to us.
>
> > - Spaces are separator of path. Therefore, two or more files can be
> > specified in one line.
> >
> > echo a.c b.c c.c | gtags --file=-
>
> I think we should allow only newline as a separator, because we have
> a plan like follows:
>
> o Allow spaces in path name of source files.
>
> derived from [Plans for the future]
> <http://www.gnu.org/software/global/plans.html>
>
> (Currently GLOBAL cannot treat the path which includes spaces in it.)
>
> Though it is difficult to implement, a lot of people hope for it.
> If we allow space as a separator, above plan would become distant.
>
> > langmap is used only by parser.
>
> Doesn't the file list include the files which are not source file like
> README, Makefile or so on? I think the file list include such files,
> and gtags should decide whether a file is source file or not by referring
> langmap as done so now. Because, soon or later, the file list should be
> used by htags,
This is revised version.
gtags/gtags.c | 33 +++-
gtags/manual.in | 4
libutil/Makefile.am | 2
libutil/filelist.c | 418 ++++++++++++++++++++++++++++++++++++++++++++++++++++
libutil/filelist.h | 27 +++
libutil/global.h | 1
6 files changed, 480 insertions(+), 5 deletions(-)
----
Hideki IWAMOTO address@hidden
gtags-file3.patch
Description: Binary data