[Octopus-devel] [Octopus-notify] svn commit: r3597 - in trunk: . liboct libxc libxc/testsuite m4 src/basic src/grid src/h_sys src/include src/main src/math src/opt_control src/species src/states src/sternheimer src/td src/xc by xavier

[Xavier Andrade]

I forgot to mention a change, in libxc I had to separate a printf statement
because the format string it was longer than 509 characters (the limit given
by the standard).

Xavier


On Fri, 23 Nov 2007, octopus-notify at tddft.org wrote:

>      Author: xavier
>        Date: Fri Nov 23 20:39:41 2007
> New Revision: 3597
>   Changeset: http://www.tddft.org/trac/octopus/changeset/3597
>
> Added:
>   trunk/src/basic/types.F90
> Modified:
>   trunk/configure.ac
>   trunk/liboct/oct_f.c
>   trunk/libxc/configure.ac
>   trunk/libxc/m4/acx.m4
>   trunk/libxc/testsuite/xc-get_data.c
>   trunk/m4/acx.m4
>   trunk/src/basic/Makefile.am
>   trunk/src/basic/datasets.F90
>   trunk/src/basic/liboct.F90
>   trunk/src/basic/liboct_parser.F90
>   trunk/src/basic/multicomm.F90
>   trunk/src/basic/string.F90
>   trunk/src/basic/varinfo.F90
>   trunk/src/grid/simul_box.F90
>   trunk/src/h_sys/epot.F90
>   trunk/src/h_sys/lasers.F90
>   trunk/src/h_sys/specie_pot.F90
>   trunk/src/include/global.h
>   trunk/src/main/main.F90
>   trunk/src/math/fft.F90
>   trunk/src/math/fftw3.F90
>   trunk/src/math/liboct_math.F90
>   trunk/src/math/spline.F90
>   trunk/src/opt_control/defstates.F90
>   trunk/src/opt_control/filters.F90
>   trunk/src/opt_control/parameters.F90
>   trunk/src/opt_control/target.F90
>   trunk/src/species/geometry.F90
>   trunk/src/species/specie.F90
>   trunk/src/species/xyz_adjust.F90
>   trunk/src/species/xyz_file.F90
>   trunk/src/states/states.F90
>   trunk/src/states/states_kpoints.F90
>   trunk/src/sternheimer/em_resp.F90
>   trunk/src/td/pes_rc.F90
>   trunk/src/td/spectrum.F90
>   trunk/src/td/td.F90
>   trunk/src/td/td_write.F90
>   trunk/src/xc/functionals.F90
>
> Log:
> * Now instead of using an integer of the size of a C pointer, there
> exists an opaque c_pointer_t type that has two methods associated,
> set_null and is_null. This is the same we have already for libxc.
>
> * For the parser blocks there is a different type, block_t.
>
> * The only exception is xc_functl_write_info where a pointer is used
> both as a pointer and an integer. The variable is defined as
> integer(SIZEOF_VOIDP), but we should fix it.
>
> * Removed the check for the non standard type 'long long' as a
> candidate for equivalence fortran integer, in theory this could be a
> problem, but is very likely that if a fortran integer is 8 bytes also
> a 'long' is 8 bytes. A better solution would be to check first if long
> long or int64_t are available, and then use them.
>
> * Removed the check for the size of a pointer in C, instead the size
> of void * is measured using the standard macro.
>
> With this changes octopus compiles with gcc -pedantic-errors.
>
>
> _______________________________________________
> Octopus-notify mailing list
> Octopus-notify at tddft.org
> http://www.tddft.org/mailman/listinfo/octopus-notify
>