doxygen-users

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

[Clayton <cla...@gm...>]

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

to successfully (more or less) process javascript files. However, I
would *also*, at least in principle, like FILE_PATTERNS to
automatically include all the default patterns that would normally be
processed if the FILE_PATTERNS parameter was left blank, without having
to explicitly list them (noting that the list in the code is not the
same as the list in the config file comment, for starters).

I am not seeing any combination of configuration parameters that will
achieve that. Is that true?

Thanks,
Clayton

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

[Stefan P. <ste...@gm...>]

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?


-- 
*Stefan P.*

Top-posting:
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

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

[Clayton <cla...@gm...>]

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

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

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

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
>

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

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

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
>>
>
>

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

[Clayton <cla...@gm...>]

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
> >>
> >
> >

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
>

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

[Clayton <cla...@gm...>]

Hi Albert, I just built your pull request

	https://github.com/doxygen/doxygen/pull/383

from source and tested it on a hundred+ github javascript repositories,
it works as expected: lots of javascript output. We would really like to
see this patch merged into master. 

Plus, the current master quite likely gives many people the impression
that doxygen processing of javascript is broken, which it is not.

Thanks for writing the fix,
Clayton

On Mon, 17 Aug 2015 18:44:48 +0200
Albert <alb...@gm...> wrote:

> 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
> >

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

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

Dear Clayton,

The pull request 383 has been integrated in the main branch.

Albert

On Mon, Aug 24, 2015 at 11:11 AM, Clayton <cla...@gm...> wrote:

> Hi Albert, I just built your pull request
>
>         https://github.com/doxygen/doxygen/pull/383
>
> from source and tested it on a hundred+ github javascript repositories,
> it works as expected: lots of javascript output. We would really like to
> see this patch merged into master.
>
> Plus, the current master quite likely gives many people the impression
> that doxygen processing of javascript is broken, which it is not.
>
> Thanks for writing the fix,
> Clayton
>
> On Mon, 17 Aug 2015 18:44:48 +0200
> Albert <alb...@gm...> wrote:
>
> > 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
> > >
>
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Doxygen-users mailing list
> Dox...@li...
> https://lists.sourceforge.net/lists/listinfo/doxygen-users
>

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

[Richard D. <Ri...@Da...>]

On 8/16/15 7:45 AM, Clayton wrote:
> 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
>
> to successfully (more or less) process javascript files. However, I
> would *also*, at least in principle, like FILE_PATTERNS to
> automatically include all the default patterns that would normally be
> processed if the FILE_PATTERNS parameter was left blank, without having
> to explicitly list them (noting that the list in the code is not the
> same as the list in the config file comment, for starters).
>
> I am not seeing any combination of configuration parameters that will
> achieve that. Is that true?
>
> Thanks,
> Clayton
>
Have you tried += to add the items to the lists? I haven't done this to 
keep defaults, but it might work.

-- 
Richard Damon

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

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

The += won't work as it will add the *.js to an empty list resulting in a
non empty list and as a consequence doxygen thinks non empty list so I
don't have to apply the defaults.

Albert

On Sun, Aug 16, 2015 at 8:07 PM, Richard Damon <Ri...@da...>
wrote:

> On 8/16/15 7:45 AM, Clayton wrote:
> > 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
> >
> > to successfully (more or less) process javascript files. However, I
> > would *also*, at least in principle, like FILE_PATTERNS to
> > automatically include all the default patterns that would normally be
> > processed if the FILE_PATTERNS parameter was left blank, without having
> > to explicitly list them (noting that the list in the code is not the
> > same as the list in the config file comment, for starters).
> >
> > I am not seeing any combination of configuration parameters that will
> > achieve that. Is that true?
> >
> > Thanks,
> > Clayton
> >
> Have you tried += to add the items to the lists? I haven't done this to
> keep defaults, but it might work.
>
> --
> Richard Damon
>
>
>
> ------------------------------------------------------------------------------
> _______________________________________________
> Doxygen-users mailing list
> Dox...@li...
> https://lists.sourceforge.net/lists/listinfo/doxygen-users
>

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

[Clayton <cla...@gm...>]

Thanks to both of you. I think to achieve this another configuration
parameter may be required, since no other suggestions are falling out.

Clayton

On Mon, 17 Aug 2015 18:33:04 +0200
Albert <alb...@gm...> wrote:

> The += won't work as it will add the *.js to an empty list resulting
> in a non empty list and as a consequence doxygen thinks non empty
> list so I don't have to apply the defaults.
> 
> Albert
> 
> On Sun, Aug 16, 2015 at 8:07 PM, Richard Damon
> <Ri...@da...> wrote:
> 
> > On 8/16/15 7:45 AM, Clayton wrote:
> > > 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
> > >
> > > to successfully (more or less) process javascript files. However,
> > > I would *also*, at least in principle, like FILE_PATTERNS to
> > > automatically include all the default patterns that would
> > > normally be processed if the FILE_PATTERNS parameter was left
> > > blank, without having to explicitly list them (noting that the
> > > list in the code is not the same as the list in the config file
> > > comment, for starters).
> > >
> > > I am not seeing any combination of configuration parameters that
> > > will achieve that. Is that true?
> > >
> > > Thanks,
> > > Clayton
> > >
> > Have you tried += to add the items to the lists? I haven't done
> > this to keep defaults, but it might work.
> >
> > --
> > Richard Damon
> >
> >
> >
> > ------------------------------------------------------------------------------
> > _______________________________________________
> > Doxygen-users mailing list
> > Dox...@li...
> > https://lists.sourceforge.net/lists/listinfo/doxygen-users
> >