instant team mailing list archive

[Johan Hake <hake@xxxxxxxxx>]

On Tuesday 13 January 2009 10:10:26 Martin Sandve Alnæs wrote:
> There is some small stuff in the TODO file.
> And there are some TODO and FIXMEs in the code.
> This is probably overlapping.
> In particular, there are some old arguments
> that may be removed or must be explained.
>

Also, what is the difference between additional_declarations and 
additional_definitions? It seems that both gets included before the 
main %include of the the source, in the constructoed interface file. Whould 
it be instructive to have a pre_declaration and a a post_declaration as we 
have both in pycc and dolfin?

Johan

> TODO:
> - Fix arguments not used in setup.py
>
> - Remove unused csrcs stuff.
>
> - Explain arrays argument.
>
> - Use object files argument.
>
> - Improve documentation
>
> - Add tests for all variants of cache mechanisms available now
>
> - Clean up imports, don't use import *
>
>
> martinal@martinal-xps:~/dev/fenics/instant/src/instant$ egrep FIXME *
> build.py:          FIXME: Describe this correctly. Tests pass arrays
> of arrays of strings.
> build.py:        instant_error("FIXME: setup.py doesn't use the C
> sources.") codegeneration.py:      - additional_definitions (FIXME:
> comment)
> codegeneration.py:      - additional_declarations (FIXME: comment)
> codegeneration.py:      - arrays (FIXME: comment)
> codegeneration.py:    #instant_warning("FIXME: Not using csrcs in
> write_setupfile().")
> codegeneration.py:    arrays = [] # FIXME: Example input here
> martinal@martinal-xps:~/dev/fenics/instant/src/instant$ egrep TODO *
> build.py:# TODO: Import only the official interface
> build.py:        # TODO: Error instead? Indicates race condition
> build.py:        - If you want to compile the files yourself. TODO:
> Not yet supported.
> cache.py:# TODO: We could make this an argument, but it's used
> indirectly several places so take care.
> __init__.py:# TODO: Import only the official interface
> inlining.py:    # TODO: Something more robust? Regexp?
> inlining.py:    # TODO: This isn't very general!
> inlining.py:    # TODO: This isn't very general!
> inlining.py:    # TODO: Is the second and third path here necessary?
> inlining.py:    # TODO: Is the second and third path here necessary?
>
>
> Martin
>
> On Tue, Jan 13, 2009 at 9:59 AM, Kent Andre <kent-and@xxxxxxxxx> wrote:
> > As always, improved documentation and better tests.
> > Anything else ?
> >
> > Kent
> >
> > _______________________________________________
> > Instant-dev mailing list
> > Instant-dev@xxxxxxxxxx
> > http://fenics.org/mailman/listinfo/instant-dev
>
> _______________________________________________
> Instant-dev mailing list
> Instant-dev@xxxxxxxxxx
> http://fenics.org/mailman/listinfo/instant-dev