[Lxr-dev] [ lxr-Bugs-453387 ] variable 'v' range

[<no...@so...>]

Bugs item #453387, was opened at 2001-08-20 07:58
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350

Category: genxref
>Group: current cvs
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Pavel Hlavnicka (pavel_hlavnicka)
Assigned to: Peder O. Klingenberg (pok)
Summary: variable 'v' range

Initial Comment:
I downloaded devel version 0.8 (no group available :)

If you use the commented out example in the lxr.conf
how to set the range of the variable 'v' it doesn't work.

First, at the line 74 in genxref is this variable
accesed  directly, there should be
@versions->varrange('v') I guess.

Second, after hacking this line it still doesn't work,
because the $LXR::Common::pathname (used in lxr.conf)
is not set at the time of the execution. I've no clue,
what was the original author's intention :)
It seems, that the range sould be read once per
processed file, because the releases (probably) and
revisions (for sure) may differ for each file. But I
didn't read the code so carefully to be sure.

HTH

----------------------------------------------------------------------

Comment By: Joseph Wilhelm (tarken)
Date: 2001-09-26 08:13

Message:
Logged In: YES 
user_id=326177

Okay.. as far as I know, this is complete.  I took the list 
of reserved words from the list on php.net.

'php' => {
		  'reserved' => 
['and','$argv','$argc','break','case','class',
					
	 'continue','default','do','die','echo','else',
					
	 'elseif','empty','endfor','endforeach','endif',
					
	 'endswitch','endwhile','E_ALL','E_PARSE','E_ERROR',
					
	 'E_WARNING','exit','extends','FALSE','for','foreach
',
					
	 'function','HTTP_COOKIE_VARS','HTTP_GET_VARS',
					
	 'HTTP_POST_VARS','HTTP_POST_FILES','HTTP_ENV_VARS',
					
	 'HTTP_SERVER_VARS','if','global','list','new','not'
,
					
	 'NULL','or','parent','PHP_OS','PHP_SELF','PHP_VERSI
ON',
					
	 'print','return','static','switch','stdClass',
					
	 'this','TRUE','var','xor','virtual','while','__FILE
__',
					
	 '__LINE__','__sleep','__wakeup'
						],
		  'spec' => ['comment',  '/\*',       '\*/',
				
	 'comment',  '//',            "\$",
				
	 'comment',  '#',             "\$",
				
	 'string',   '"',             '"',
				
	 'string',   "'",             "'",
				
	 'include',  'require',      "\$",
				
	 'include',  'include',      "\$",
				
	 'include',  'require_once', "\$",
				
	 'include',  'include_once', "\$"
					],
		},


----------------------------------------------------------------------

Comment By: Malcolm Box (mbox)
Date: 2001-09-26 02:33

Message:
Logged In: YES 
user_id=215386

Is there any chance you could post the stuff you added to
generic.conf to get php working?  I'm trying to flesh the
file out, since it currently doesn't support a lot of the
languages that ctags can cope with.

----------------------------------------------------------------------

Comment By: Joseph Wilhelm (tarken)
Date: 2001-09-17 13:48

Message:
Logged In: YES 
user_id=326177

Hmm, on my last follow-up to this, perhaps it didn't work 
properly.  I added a 'php' entry to generic.conf so that 
everything should work great in my PHP files... but I'm not 
getting any cross-references on functions or anything else. 
All I'm getting them on is require()'ed files in the 
current directory.  Also, I looked at the lxr database in 
MySQL... and it's completely empty.  Perhaps this is 
evidence that genxref didn't do quite what it was supposed 
to? :)

----------------------------------------------------------------------

Comment By: Joseph Wilhelm (tarken)
Date: 2001-09-17 12:53

Message:
Logged In: YES 
user_id=326177

Actually I got this fixed on my copy by making the line 
look like this:
    @versions = eval($config->{variables}{v}{range});
Instead of:
    @versions = @{$config->{variables}{v}{range}};

And it's working perfectly now :)

----------------------------------------------------------------------

Comment By: Peder O. Klingenberg (pok)
Date: 2001-08-20 11:17

Message:
Logged In: YES 
user_id=222352

See patch 453450.

----------------------------------------------------------------------

Comment By: Peder O. Klingenberg (pok)
Date: 2001-08-20 10:59

Message:
Logged In: YES 
user_id=222352

No, it doesn't work at the moment, because genxref is
broken.  I have fixed it, but the patch has not been
committed yet.  Your hunch is correct, it's intended to be
called per file.

Stay tuned.

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=390117&aid=453387&group_id=27350