Re: [Doxygen-users] FILE_PATTERNS: one custom plus all defaults

[Albert <alb...@gm...>]

Bug report is not really needed.

I don't think the other places are in conflict.
There is no separate javascript parser but it is more or less integrated in
the c parser. The g_lang2extMap tells that javascript type of files have to
be parsed by the c parser and that the type of language is SrcLangExt_JS.
With the later it is possible to ask in the code which language a file has
and do special things for this language.

The initDefaultExtensionMapping maps the files extension to the language
type of files.

Albert




On Mon, Aug 17, 2015 at 10:09 AM, Clayton <cla...@gm...> wrote:

> Thanks for the quick work, Albert. Does this mean a bug report is now
> no longer needed? (That's quite the list of bugs in Bugzilla....)
>
> It looks to me like your patch leaves js in. And I am in fact seeing at
> least two different references in util.cpp, which also might be in
> conflict:
>
> g_lang2extMap[] =
> {
> //  language       parser           parser option
>   { "javascript",  "c",             SrcLangExt_JS       },
>
>
> void initDefaultExtensionMapping()
> {
>   g_extLookup.setAutoDelete(TRUE);
>   //                  extension      parser id
>   updateLanguageMapping(".as",       "javascript");
>   updateLanguageMapping(".js",       "javascript");
>
> The second reference above gives the impression that there is an
> internal doxygen parser for javascript? Really??
>
> Clayton
>
>
>
>
> On Sun, 16 Aug 2015 18:21:46 +0200
> Albert <alb...@gm...> wrote:
>
> > I've just pushed a proposed patch to github (pull request 383)
> >
> > Albert
> >
> > On Sun, Aug 16, 2015 at 4:06 PM, Albert <alb...@gm...>
> > wrote:
> >
> > > The following does not yet solve your problem, but points in the
> > > direction where we have to look to solve the problem:
> > > There is in util.cpp another list which does contain .js, looks
> > > like a small inconsistency between config.xml, util.cpp and
> > > config.l. Please file a bug report to signal this discrepancy.
> > >
> > > Albert
> > >
> > >
> > > On Sun, Aug 16, 2015 at 3:09 PM, Clayton <cla...@gm...> wrote:
> > >
> > >> On Sun, 16 Aug 2015 14:05:56 +0200
> > >> Stefan Pendl <ste...@gm...> wrote:
> > >>
> > >> > Am 16.08.2015 um 13:45 schrieb Clayton:
> > >> > > Hi doxygen,
> > >> > >
> > >> > > I am looking at the config file and writing to ask if I am
> > >> > > missing something.
> > >> > >
> > >> > > I am using
> > >> > >
> > >> > >     FILE_PATTERNS = *.js
> > >> > >     FILTER_PATTERNS = *.js=plugins/js2doxy/js2doxy.pl
> > >> >
> > >> > From the help file topic "Configuration => Configuration options
> > >> > related to the input files => FILE_PATTERNS" it seems that *.js
> > >> > is included in the default already.
> > >> >
> > >> > Is your doxygen version less than v1.8.10?
> > >>
> > >> Hi Stefan, the help is not the same as the code, from src/config.l
> > >> in a very recent clone of the source, I believe this to be the
> > >> ACTUAL default list of file patterns:
> > >>
> > >>   QStrList &filePatternList = Config_getList("FILE_PATTERNS");
> > >>   if (filePatternList.isEmpty())
> > >>   {
> > >>     filePatternList.append("*.c");
> > >>     filePatternList.append("*.cc");
> > >>     filePatternList.append("*.cxx");
> > >>     filePatternList.append("*.cpp");
> > >>     filePatternList.append("*.c++");
> > >>     filePatternList.append("*.java");
> > >>     filePatternList.append("*.ii");
> > >>     filePatternList.append("*.ixx");
> > >>     filePatternList.append("*.ipp");
> > >>     filePatternList.append("*.i++");
> > >>     filePatternList.append("*.inl");
> > >>     filePatternList.append("*.h");
> > >>     filePatternList.append("*.hh");
> > >>     filePatternList.append("*.hxx");
> > >>     filePatternList.append("*.hpp");
> > >>     filePatternList.append("*.h++");
> > >>     filePatternList.append("*.idl");
> > >>     filePatternList.append("*.odl");
> > >>     filePatternList.append("*.cs");
> > >>     filePatternList.append("*.php");
> > >>     filePatternList.append("*.php3");
> > >>     filePatternList.append("*.inc");
> > >>     filePatternList.append("*.m");
> > >>     filePatternList.append("*.mm");
> > >>     filePatternList.append("*.dox");
> > >>     filePatternList.append("*.py");
> > >>     filePatternList.append("*.f90");
> > >>     filePatternList.append("*.f");
> > >>     filePatternList.append("*.for");
> > >>     filePatternList.append("*.vhd");
> > >>     filePatternList.append("*.vhdl");
> > >>     filePatternList.append("*.tcl");
> > >>     filePatternList.append("*.md");
> > >>     filePatternList.append("*.markdown");
> > >>
> > >> *.js is not in there. The plugin is necessary, the point is, how
> > >> to get the plugin to integrate with the default list
> > >> automatically, without explicitly adding the default list to
> > >> FILE_PATTERNS.
> > >>
> > >> Thanks,
> > >> Clayton
> > >>
> > >>
> > >>
> ------------------------------------------------------------------------------
> > >> _______________________________________________
> > >> Doxygen-users mailing list
> > >> Dox...@li...
> > >> https://lists.sourceforge.net/lists/listinfo/doxygen-users
> > >>
> > >
> > >
>
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Doxygen-users mailing list
> Dox...@li...
> https://lists.sourceforge.net/lists/listinfo/doxygen-users
>