Bugs item #2800841, was opened at 2009-06-04 04:01
Message generated for change (Comment added) made by chjaeg
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=390117&aid=2800841&group_id=27350
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: genxref
Group: current cvs
Status: Closed
Resolution: None
Priority: 5
Private: No
Submitted By: Christian Jaeger (chjaeg)
Assigned to: Nobody/Anonymous (nobody)
Summary: Not an ARRAY reference (from langinfo('reserved'))
Initial Comment:
In my case $self->langinfo('reserved') is returning some 14000 times an empty hash instead of an array. I don't know why, but I've made the attached workaround for me, which doesn't seem to have adverse side effects.
----------------------------------------------------------------------
Comment By: Christian Jaeger (chjaeg)
Date: 2009-06-05 22:24
Message:
To nitpick perl didn't say it was a hash reference, only "Not an ARRAY
reference". But I agree with your conclusion.
----------------------------------------------------------------------
Comment By: AdrianIssott (adrianissott)
Date: 2009-06-05 22:11
Message:
It'd be nice to add a better error message but
(i) the current error was OK since it was complaining that it was given a
hash instead of an array which is exactly what was happening.
(ii) there are a number of similar places that would need updating in a
similar way. There's little point just doing just this one and doing them
all would be a fair amount of effort for the return.
----------------------------------------------------------------------
Comment By: Christian Jaeger (chjaeg)
Date: 2009-06-05 00:23
Message:
Uh, sorry, in the lxr main config file I have still been referring to the
generic.conf file from the Debian lxr-cvs package (which I still had
installed).
Changing that to the config from CVS make the problem go away.
(Probably the culprit was with the following change for the 'Make'
language:
'reserved' => {},
to
'reserved' => [],
)
So, no change needed, unless you want to give a better error message for
such cases than Perl provides.
----------------------------------------------------------------------
Comment By: AdrianIssott (adrianissott)
Date: 2009-06-04 21:31
Message:
Christian, would you mind attaching lib\LXR\Lang\generic.conf file that
you're using?
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=390117&aid=2800841&group_id=27350
|