|
From: | Tom Rondeau |
Subject: | Re: [Discuss-gnuradio] Ben's docstring work |
Date: | Mon, 17 Oct 2011 13:46:41 -0400 |
Ahhh much better. I added generation for a few of the components in core
(general, gengen, filter, io)
http://gnuradio.org/cgit/jblum.git/commit/?h=swig_docs&id=d65572359a74e97ee6a01d89dcc44fd77ce54fef
Basically, its pretty nice. The xml only regenerates when header files
change, and the swig docs.i only regenerate when the xml is changed, and
so on. No checked-in generated files. XML generation is very quick. Your
python script actually takes a bit longer (but im not complaining). :-)
I would like to get the grc docs parser looking for __doc__ strings soon.
A few comments on your doc generator. Im not sure we agree on a standard
here for documenting headers... but useful doc blocks may be in several
places in the header:
1) docs for the factor/make function
2) docs for class definition
3) docs just in the header, think \file
It looks like the generated docstrings only gets one of these. I suggest
concatenating all forms of docs into the one class docstring so the full
amount of docs is available at runtime to python.
-Josh
>
> It was the position of the %include "swig_doc.i" in the digital_swig.i file.
> It needs to be at the top rather than at the bottom.
> Here's a diff.
>
> diff --git a/gr-digital/swig/digital_swig.i b/gr-digital/swig/digital_swig.i
> index c804b5c..f6372b1 100644
> --- a/gr-digital/swig/digital_swig.i
> +++ b/gr-digital/swig/digital_swig.i
> @@ -23,6 +23,8 @@
>
> %include <gri_control_loop.i>
>
> +%include "swig_doc.i"
> +
> %{
> #include "digital_binary_slicer_fb.h"
> #include "digital_clock_recovery_mm_cc.h"
> @@ -59,8 +61,6 @@
> %include "digital_cpmmod_bc.i"
> %include "digital_gmskmod_bc.i"
>
> -%include "swig_doc.i"
> -
> #if SWIGGUILE
> %scheme %{
> (load-extension-global "libguile-gnuradio-digital_swig"
> "scm_init_gnuradio_digital_swig_module")
_______________________________________________
Discuss-gnuradio mailing list
address@hidden
https://lists.gnu.org/mailman/listinfo/discuss-gnuradio
[Prev in Thread] | Current Thread | [Next in Thread] |