ccan
7 years agorfc822: Rename bad-header test
David Gibson [Thu, 4 Oct 2012 14:12:27 +0000 (00:12 +1000)]
rfc822: Rename bad-header test

The run-bad-header.c test in the rfc822 module is actually testing handling
of one specific sort of bad header - a header field which contains no
colon character.  Rename the test and its internal variables to better
reflect this.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agorfc822: Move check_header into helper.c
David Gibson [Wed, 3 Oct 2012 13:20:46 +0000 (23:20 +1000)]
rfc822: Move check_header into helper.c

The check_header() test function will be useful to more than one testcase
so this patch moves it into the common helper.c file.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agorfc822: Add an example program
David Gibson [Wed, 3 Oct 2012 13:02:39 +0000 (23:02 +1000)]
rfc822: Add an example program

Add a simple full-blown example program for the rfc822 module.
'headernames' simply dumps the header field name for every header field in
each message file given on the command line to stdout.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agostr: add STR_MAX_CHARS().
Rusty Russell [Mon, 24 Sep 2012 01:25:13 +0000 (10:55 +0930)]
str: add STR_MAX_CHARS().

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years agolist: change license to BSD-MIT.
Rusty Russell [Wed, 19 Sep 2012 01:08:40 +0000 (10:38 +0930)]
list: change license to BSD-MIT.

Approved by all contributors:
https://lists.ozlabs.org/pipermail/ccan/2012-September/000490.html
https://lists.ozlabs.org/pipermail/ccan/2012-September/000488.html
https://lists.ozlabs.org/pipermail/ccan/2012-September/000489.html

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years agotime: make trivial functions inline
Rusty Russell [Mon, 10 Sep 2012 06:43:15 +0000 (16:13 +0930)]
time: make trivial functions inline

And enhance the documentation a little.

7 years agotime: add time_check() call and test.
Rusty Russell [Mon, 10 Sep 2012 06:43:11 +0000 (16:13 +0930)]
time: add time_check() call and test.

Times can be invalid: non-normalized, or negative.

7 years agofailtest, tools: update for new timespec-using ccan/time module.
Rusty Russell [Mon, 10 Sep 2012 06:14:54 +0000 (15:44 +0930)]
failtest, tools: update for new timespec-using ccan/time module.

7 years agotime: use timespec instead of timeval.
Rusty Russell [Mon, 10 Sep 2012 06:14:52 +0000 (15:44 +0930)]
time: use timespec instead of timeval.

It's a superset of struct timeval.  It's also a bit trickier, since we
need -lrt on Linux to use clock_gettime(), and also we can't just cram the
time into a u64 for multiply and divide.

7 years agoccanlint: don't remove HAVE_STRUCT_TIMESPEC when testing without features.
Rusty Russell [Mon, 10 Sep 2012 06:14:45 +0000 (15:44 +0930)]
ccanlint: don't remove HAVE_STRUCT_TIMESPEC when testing without features.

If we do have it, and we redefine it, compilation breaks.

7 years agoconfigurator: add library testing
Rusty Russell [Mon, 10 Sep 2012 06:14:43 +0000 (15:44 +0930)]
configurator: add library testing

In particular, HAVE_CLOCK_GETTIME_IN_LIBRT.

7 years agoccanlint: make sure examples get correct -l options.
Rusty Russell [Mon, 10 Sep 2012 06:14:11 +0000 (15:44 +0930)]
ccanlint: make sure examples get correct -l options.

We intuit the dependencies of embedded examples by looking for #include.
But these extra dependencies may require extra libraries, so we need to
add those, too.

7 years agoccanlint: recurse to get -l options.
Rusty Russell [Mon, 10 Sep 2012 06:14:00 +0000 (15:44 +0930)]
ccanlint: recurse to get -l options.

If a dependent module needs -l options, we don't notice.  Fix that, now
we have one.

7 years agotools: more intelligent caching for compile _info.
Rusty Russell [Mon, 10 Sep 2012 05:03:01 +0000 (14:33 +0930)]
tools: more intelligent caching for compile _info.

We let the get_deps() caller hand us the compiled _info filename, but what
about recursive dependencies?  These we re-generate every time.

So fix this: hand a generator callback to get_deps(), expose the one
which simply compiles it, and add a ccanlint one which looks up the
manifest to see if we have one already.

Before:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
31

After:
$ ccanlint -vvvv ccan/failtest | grep -c 'Creating.*_info'
17

7 years agoMerge branch 'master' of ozlabs.org:ccan
Rusty Russell [Mon, 10 Sep 2012 05:02:01 +0000 (14:32 +0930)]
Merge branch 'master' of ozlabs.org:ccan

7 years agorfc822: Fix test bugs on 64-bit platforms
David Gibson [Tue, 14 Aug 2012 03:55:53 +0000 (13:55 +1000)]
rfc822: Fix test bugs on 64-bit platforms

This fixes some format string related bugs that show up on 64-bit builds.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agorfc822: Include RFC5322 text for reference
David Gibson [Sun, 12 Aug 2012 10:21:44 +0000 (20:21 +1000)]
rfc822: Include RFC5322 text for reference

For convenience, check in a copy of the full text of RFC5322 (the latest
revision of RFC822).

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agoccanlint: make depends_accurate test more accurate.
Rusty Russell [Wed, 18 Jul 2012 03:38:08 +0000 (13:08 +0930)]
ccanlint: make depends_accurate test more accurate.

In particular, don't complain on conditional lines.  That's less thorough,
but removes false negatives we have now.

7 years agorfc822: Fix bug parsing headers when body begins with linear whitespace
David Gibson [Sat, 30 Jun 2012 05:34:23 +0000 (15:34 +1000)]
rfc822: Fix bug parsing headers when body begins with linear whitespace

next_header_parse() had a bug which would mis-parse an extra header if the
body of the message began with a linear whitespace character (space or
tab).  This patch fixes it and adds a suitable testcase.

Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
7 years agorfc822: suppress license warning about failtest.
Rusty Russell [Tue, 26 Jun 2012 07:06:37 +0000 (16:36 +0930)]
rfc822: suppress license warning about failtest.

It's only used for testing, but it's LGPLv3 (vs LGPLv2).  But unless we
suppress it, ccanlint exits with status 1 and my sanity checks stop me from
uploading.

7 years agorfc822: new module.
Rusty Russell [Tue, 26 Jun 2012 06:59:31 +0000 (16:29 +0930)]
rfc822: new module.

7 years agoweb: fix umask on uploader so it's group readable.
Rusty Russell [Tue, 26 Jun 2012 06:58:08 +0000 (16:28 +0930)]
web: fix umask on uploader so it's group readable.

7 years agobytestring: new module.
Rusty Russell [Tue, 26 Jun 2012 06:46:18 +0000 (16:16 +0930)]
bytestring: new module.

7 years agoAvoid compiler errors if u32 is defined as a type
Alessandro Guido [Wed, 20 Jun 2012 08:46:19 +0000 (10:46 +0200)]
Avoid compiler errors if u32 is defined as a type

Fixes this error:
/hash.h:304:13: error: two or more data types in
declaration specifiers

7 years agoantithread, failtest: use ccan/err instead of err.h.
Rusty Russell [Sat, 9 Jun 2012 06:06:45 +0000 (15:36 +0930)]
antithread, failtest: use ccan/err instead of err.h.

7 years agoopt: avoid using err.h.
Rusty Russell [Sat, 9 Jun 2012 06:04:56 +0000 (15:34 +0930)]
opt: avoid using err.h.

Seems it's not available on Solaris.

7 years agoerr: actually implement replacement versions!
Rusty Russell [Fri, 8 Jun 2012 11:19:32 +0000 (20:49 +0930)]
err: actually implement replacement versions!

This is where you see the genius of the originals: you can't implement them
without horrible hacks to get the program name :(

7 years agoMerge branch 'master' of ozlabs.org:ccan
Rusty Russell [Fri, 8 Jun 2012 10:58:01 +0000 (20:28 +0930)]
Merge branch 'master' of ozlabs.org:ccan

7 years agoconfigurator: add HAVE_ERR_H test.
Rusty Russell [Fri, 8 Jun 2012 07:51:04 +0000 (17:21 +0930)]
configurator: add HAVE_ERR_H test.

7 years agoccan/err: new err.h-replacing module.
Rusty Russell [Fri, 8 Jun 2012 07:37:03 +0000 (17:07 +0930)]
ccan/err: new err.h-replacing module.

Seems like Solaris doesn't have err.h, as discovered by Samba.

7 years agotools/create-ccan-tree: Add basic waf support
Michael Ellerman [Fri, 25 May 2012 06:38:08 +0000 (16:38 +1000)]
tools/create-ccan-tree: Add basic waf support

This adds a "waf" buildtype to create-ccan-tree. When passed it
generates a basic wscript for building everything into a libccan.a

It doesn't support running the generator, so you need to provide
config.h

Signed-off-by: Michael Ellerman <michael@ellerman.id.au>
Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
7 years agotools/create-ccan-tree: Add --build-type=automake
Jeremy Kerr [Thu, 24 May 2012 05:48:19 +0000 (13:48 +0800)]
tools/create-ccan-tree: Add --build-type=automake

Add a new build type to create an appropriate Makefile.am for the
newly-created ccan tree.

Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
7 years agotools/create-ccan-tree: Add --build-type=TYPE
Jeremy Kerr [Thu, 24 May 2012 01:48:00 +0000 (09:48 +0800)]
tools/create-ccan-tree: Add --build-type=TYPE

Replace the --exclude-configurator argument with a
single --build-type=TYPE argument, allowing future expansion of build
system types (and allowing source-tree-only creation). We currently
support:

 * 'make' (just the Makefile, equivalent to --exclude-configurator)
 * 'make+config' (current default)

Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
7 years agotools/create-ccan-tree: replace --exclude-tests with --copy-all option
Jeremy Kerr [Thu, 24 May 2012 03:49:10 +0000 (11:49 +0800)]
tools/create-ccan-tree: replace --exclude-tests with --copy-all option

Rather than explicitly excluding the tests, just copy the source files
by default. Add an option (--copy-all) to include all of the non-source
stuff in each module.

Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
7 years agotools: ccan_depends requires a config.h
Jeremy Kerr [Tue, 22 May 2012 09:19:24 +0000 (17:19 +0800)]
tools: ccan_depends requires a config.h

Commit a77cc282 causes tools/create-ccan-tree to fail, as it can't build
tools/ccan_depends (via `make tools/ccan_depends`) without a
configuration header present.

This change adds a dependency on config.h, so that the configurator
kicks-in prior to building ccan_depends.o.

Other tools don't need this dependency, as tools/ccan_depends is built
during the Makefile 'include' process.

Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
7 years agoconfig.h: don't include in git repository.
Rusty Russell [Thu, 17 May 2012 03:44:18 +0000 (13:14 +0930)]
config.h: don't include in git repository.

We create and upload an example for the web anyway, if people really need it.

7 years agotools: Add create-ccan-tree utility
Jeremy Kerr [Wed, 16 May 2012 03:02:56 +0000 (11:02 +0800)]
tools: Add create-ccan-tree utility

Add a script to create a ccan source tree for inclusion into projects'
distributed code.

create-ccan-tree takes a list of ccan modules and copies them, plus any
dependent modules, into a target directory. Enough build infrastructure
is added to build libccan.a from the ccan modules.

Typical usage for my project that uses talloc and read-write-all:

  lib/ccan.git/tools/create-ccan-tree --exclude-tests \
lib/ccan/ talloc read-write-all

- this uses a git submodule in $PWDlib/ccan.git to create a source tree
in $PWD/lib/ccan, containing code for talloc and read-write-all.

Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com>
8 years agoasort: fix test when typesafe_cb not supported. origin/suncc
Rusty Russell [Thu, 29 Mar 2012 04:28:33 +0000 (14:58 +1030)]
asort: fix test when typesafe_cb not supported.

Solaris CC seems to support typeof, but not __builtin_types_compatible_p.

8 years agocast: make sure suncc sees a constant.
Rusty Russell [Thu, 29 Mar 2012 04:28:33 +0000 (14:58 +1030)]
cast: make sure suncc sees a constant.

cast_const() et. al. are supposed to be a constant expression, so you can do things like:
static char *p = cast_const(char *, (const char *)"hello");

Unfortunately, a cast to intptr_t and arithmetic makes suncc reject it as
a constant expression.  We need the cast, because (1) the expression could be
a void *, so we can't just add to it, and (2) gcc complains with -Wcast-qual
without it.

So instead of adding BUILD_BUG_OR_ZERO, we use a ? :, which keeps everyone happy.

8 years agoccan_tokenizer: don't assume __VA_ARGS__ can be empty.
Rusty Russell [Thu, 29 Mar 2012 04:28:23 +0000 (14:58 +1030)]
ccan_tokenizer: don't assume __VA_ARGS__ can be empty.

That's a gcc extension.

8 years agocast: test/compile_fail-cast_static.c should fail without COMPOUND_LITERALS.
Rusty Russell [Tue, 27 Mar 2012 05:10:45 +0000 (15:40 +1030)]
cast: test/compile_fail-cast_static.c should fail without COMPOUND_LITERALS.

It still gave a warning on gcc, because casting a char to a char* gives a warning.  Not so on sun CC.

8 years agoccanlint: module_links test must link with module.
Rusty Russell [Tue, 27 Mar 2012 05:01:27 +0000 (15:31 +1030)]
ccanlint: module_links test must link with module.

Gcc doesn't include unused inline functions, even without -O, but Solaris CC
does, so fails to link block_pool where one inline function refers to a
non-inline function.

8 years agoasprintf: #include <stdarg.h> in test/run.c
Rusty Russell [Tue, 27 Mar 2012 04:37:50 +0000 (15:07 +1030)]
asprintf: #include <stdarg.h> in test/run.c

We get away with it on Linux, not on Solaris.

8 years agoccanlint: tests_compile_coverage needs GCC.
Rusty Russell [Tue, 27 Mar 2012 04:36:46 +0000 (15:06 +1030)]
ccanlint: tests_compile_coverage needs GCC.

We only support gcc/gcov so far.

8 years agoccanlint: add ilog dependency.
Rusty Russell [Tue, 27 Mar 2012 04:22:13 +0000 (14:52 +1030)]
ccanlint: add ilog dependency.

This only matters on non-gcc, where ilog functions are not just mapped to builtins.

8 years agotap: assume variadic macros.
Rusty Russell [Tue, 27 Mar 2012 04:08:13 +0000 (14:38 +1030)]
tap: assume variadic macros.

The Sun C compiler has them, but fails this overzealous check.

8 years agoopt: fix Solaris compile.
Rusty Russell [Tue, 27 Mar 2012 04:00:29 +0000 (14:30 +1030)]
opt: fix Solaris compile.

Need sys/termios.h for struct winsize.

8 years agoiscsi: use <sys/filio.h> on Solaris.
Rusty Russell [Tue, 27 Mar 2012 03:54:00 +0000 (14:24 +1030)]
iscsi: use <sys/filio.h> on Solaris.

Needed for FIONREAD.

8 years agoalloc: fix return of void.
Rusty Russell [Tue, 27 Mar 2012 03:45:19 +0000 (14:15 +1030)]
alloc: fix return of void.

Sun CC complains.

8 years agotools: don't use __attribute__
Rusty Russell [Tue, 27 Mar 2012 03:45:18 +0000 (14:15 +1030)]
tools: don't use __attribute__

We have compiler.h for that.

8 years agotools: fix bogus unreachable return.
Rusty Russell [Tue, 27 Mar 2012 03:45:18 +0000 (14:15 +1030)]
tools: fix bogus unreachable return.

8 years agotools/configurator: fix test for /proc/self/maps.
Rusty Russell [Tue, 27 Mar 2012 03:45:18 +0000 (14:15 +1030)]
tools/configurator: fix test for /proc/self/maps.

Header file was wrong, suncc gives warning about missing open prototype, causing configurator to fail.

8 years agoMakefile: generate config.h in two stages
Rusty Russell [Tue, 27 Mar 2012 03:45:16 +0000 (14:15 +1030)]
Makefile: generate config.h in two stages

This means we don't create an empty file if configurator fails.

8 years agofailtest: compile fix for OpenBSD
Rusty Russell [Wed, 21 Mar 2012 00:48:18 +0000 (11:18 +1030)]
failtest: compile fix for OpenBSD

OpenBSD doesn't idempotent-wrap sys/mman.h, so when we #define mmap to
an alternative, it fails to compile when sys/mman.h is included again.

Workaround is not to #define mmap to add arguments on Open BSD.

8 years agoMerge branch 'ccantool'
Rusty Russell [Mon, 19 Mar 2012 05:51:00 +0000 (16:21 +1030)]
Merge branch 'ccantool'

8 years agotools: move config.h reading to its own file
Rusty Russell [Mon, 19 Mar 2012 05:36:27 +0000 (16:06 +1030)]
tools: move config.h reading to its own file

Move ccan_dir determination out to its own function, rather than implying it
by the first time we get the manifest of a module.

8 years agotools: explicit find_ccan_dir()
Rusty Russell [Mon, 19 Mar 2012 05:15:45 +0000 (15:45 +1030)]
tools: explicit find_ccan_dir()

Move ccan_dir determination out to its own function, rather than implying it
by the first time we get the manifest of a module.

8 years agoccanlint: handle -x correctly with multiple modules.
Rusty Russell [Mon, 19 Mar 2012 05:14:59 +0000 (15:44 +1030)]
ccanlint: handle -x correctly with multiple modules.

After the first module we were resetting even the excluded tests, which caused
a segfault as they weren't initialized.

8 years agotools: determing ccan directory differently
Rusty Russell [Mon, 19 Mar 2012 05:14:13 +0000 (15:44 +1030)]
tools: determing ccan directory differently

Don't assume it's two levels above the module dir, instead look for
last "/ccan/" in path.  This is an incremental step towards nested
module directories.

8 years agotools: split out manifest from ccanlint
Rusty Russell [Mon, 19 Mar 2012 05:13:13 +0000 (15:43 +1030)]
tools: split out manifest from ccanlint

Other tools will soon want to use this, so make it independent.
(eg. ccantool).

8 years agotdb2: remove: it's now in SAMBA where it belongs.
Rusty Russell [Fri, 9 Mar 2012 02:55:03 +0000 (13:25 +1030)]
tdb2: remove: it's now in SAMBA where it belongs.

8 years agohtable: fix bug where first entry has hash of 0 or 1.
Rusty Russell [Fri, 9 Mar 2012 02:50:35 +0000 (13:20 +1030)]
htable: fix bug where first entry has hash of 0 or 1.

Thanks to Zolt├ín Lajos Kis for the bug report and test case!

8 years agofailtest: don't assume FD_SETSIZE is maximum runtime fd.
Rusty Russell [Thu, 8 Mar 2012 03:44:22 +0000 (14:14 +1030)]
failtest: don't assume FD_SETSIZE is maximum runtime fd.

This breaks when rlimit is less.  Unfortunately, valgrind (32 bit x86,
3.7.0.SVN, Ubuntu) fails to set the file limit properly on the test:
reducing it to the obvious getrlimit/setrlimit/getrlimit works fine,
so leaving diagnostics for another day.

8 years agottxml: made alloc failure consistent - return NULL
Daniel Burke [Sun, 26 Feb 2012 17:03:59 +0000 (03:33 +1030)]
ttxml: made alloc failure consistent - return NULL

8 years agottxml: removed cruft from tests
Daniel Burke [Sun, 26 Feb 2012 17:03:05 +0000 (03:33 +1030)]
ttxml: removed cruft from tests

8 years agottxml: fixed license test
Daniel Burke [Sun, 26 Feb 2012 17:02:39 +0000 (03:32 +1030)]
ttxml: fixed license test

8 years agotdb: delete from CCAN.
Rusty Russell [Tue, 14 Feb 2012 09:39:18 +0000 (20:09 +1030)]
tdb: delete from CCAN.

This was an early experiment in putting code into CCAN, but TDB is a public
library distributed as part of the Samba sources; there is no reason to
publish it here (especially now the unit tests are also in Samba).

8 years agoconfigurator: define HAVE_CCAN.
Rusty Russell [Tue, 14 Feb 2012 09:34:00 +0000 (20:04 +1030)]
configurator: define HAVE_CCAN.

An interesting case came up with the tdb2 code in SAMBA recently.  It's a
public library, and it doesn't want to be dependent on CCAN, but the header
uses cast and typesafe_cb for extra type safety.

A good solution to this is to put dummy versions under !HAVE_CCAN.  Of course,
any CCAN config.h should define HAVE_CCAN.

8 years agotdb2: copy tdb1's changed expansion logic.
Rusty Russell [Sat, 28 Jan 2012 04:14:43 +0000 (14:44 +1030)]
tdb2: copy tdb1's changed expansion logic.

TDB2 uses the same expansion logic as TDB1, which got factored out
recently.  So update TDB2 to match.

8 years agotdb2: tdb1: use same expansion factor logic when expanding for new recovery area.
Rusty Russell [Sat, 28 Jan 2012 04:14:43 +0000 (14:44 +1030)]
tdb2: tdb1: use same expansion factor logic when expanding for new recovery area.

If we're expanding because the current recovery area is too small, we
expand only the amount we need.  This can quickly lead to exponential
growth when we have a slowly-expanding record (hence a
slowly-expanding transaction size).

(Syncs with tdb1 SAMBA patch 3a2a755e3380a8f81374009d463cd06161352507)

8 years agotdb2: Avoid a malloc/memcpy in _tdb1_store
Volker Lendecke [Sat, 28 Jan 2012 04:14:36 +0000 (14:44 +1030)]
tdb2: Avoid a malloc/memcpy in _tdb1_store
(Syncs tdb2 with SAMBA tdb1's 664add17757836c5ee98618aef11371f412b6e44)

8 years agotdb2: careful on wrap.
Rusty Russell [Wed, 21 Dec 2011 05:54:41 +0000 (16:24 +1030)]
tdb2: careful on wrap.

It's much harder to wrap a 64-bit tdb2 than a 32-bit tdb1, but we should still
take care against bugs.

Also, we should *not* cast the length to a size_t when comparing it to
the stat result, in case size_t is 32 bit.

8 years agotdb2: be more careful on 4G files (tdb1).
Rusty Russell [Wed, 21 Dec 2011 05:44:51 +0000 (16:14 +1030)]
tdb2: be more careful on 4G files (tdb1).

I came across a tdb which had wrapped to 4G + 4K, and the contents had been
destroyed by processes which thought it only 4k long.  Fix this by checking
on open, and making tdb_oob() check for wrap itself.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
(Ported from tdb1 SAMBA commit b64494535dc62f4073fc6302847593ed6e6ec38b)

8 years agotdb: don't free old recovery area when expanding if already at EOF.
Rusty Russell [Tue, 20 Dec 2011 07:10:57 +0000 (17:40 +1030)]
tdb: don't free old recovery area when expanding if already at EOF.

We allocate a new recovery area by expanding the file.  But if the
recovery area is already at the end of file (as shown in at least one
client case), we can simply expand the record, rather than freeing it
and creating a new one.

8 years agotdb: use same expansion factor logic when expanding for new recovery area.
Rusty Russell [Tue, 20 Dec 2011 07:09:28 +0000 (17:39 +1030)]
tdb: use same expansion factor logic when expanding for new recovery area.

If we're expanding because the current recovery area is too small, we
expand only the amount we need.  This can quickly lead to exponential
growth when we have a slowly-expanding record (hence a
slowly-expanding transaction size).

8 years agotdb: test exponential recovery expansion problem.
Rusty Russell [Tue, 20 Dec 2011 07:07:45 +0000 (17:37 +1030)]
tdb: test exponential recovery expansion problem.

8 years agoopt: much prettier usage (using terminal size)
Rusty Russell [Mon, 19 Dec 2011 01:21:45 +0000 (11:51 +1030)]
opt: much prettier usage (using terminal size)

8 years agoopt: handle -Wcast-qual warnings.
Rusty Russell [Fri, 16 Dec 2011 00:09:47 +0000 (10:39 +1030)]
opt: handle -Wcast-qual warnings.

8 years agoopt: don't leak on exit (valgrind complains).
Rusty Russell [Thu, 15 Dec 2011 03:45:47 +0000 (14:15 +1030)]
opt: don't leak on exit (valgrind complains).

Recent real usage case showed this leak when we call opt_usage_and_exit:
we don't bother freeing before exit.  With valgrind, it matters.

8 years agolist.h: opaque list iteration functionality
Andrey Smirnov [Mon, 12 Dec 2011 03:22:44 +0000 (13:52 +1030)]
list.h: opaque list iteration functionality

See https://github.com/rustyrussell/ccan/pull/1#issuecomment-2856561
for details.
(Reworked with minor cleanups -- Rusty)

8 years agocontainer_of: add container_off_var macro
Rusty Russell [Mon, 12 Dec 2011 03:20:05 +0000 (13:50 +1030)]
container_of: add container_off_var macro

Based on patch by Andrey Smirnov <andrew.smirnov@gmail.com>:

    There is are certain use-cases when it is necessary to know the offset
    of the member in a structure's memory layout. One such use-case can be
    seen in `ccan/list/list.h' in macros `list_for_each' and
    `list_for_each_safe'. This commit implements said functionality with
    `container_of_var_off' macro.

Renamed container_of_var_off -> container_off_var now we have container_off.

8 years agocheck_type: fix incorrect documentation.
Rusty Russell [Mon, 12 Dec 2011 03:20:05 +0000 (13:50 +1030)]
check_type: fix incorrect documentation.

8 years agocontainer_of: don't put member_ptr in container_off.
Rusty Russell [Mon, 12 Dec 2011 03:20:05 +0000 (13:50 +1030)]
container_of: don't put member_ptr in container_off.

It's convenient to check that the member is the given type, but we can leave
that to the callers.

8 years agoMakefile, configurator: Add additional debug flags to CFLAGS
Andrey Smirnov [Mon, 12 Dec 2011 03:19:57 +0000 (13:49 +1030)]
Makefile, configurator: Add additional debug flags to CFLAGS
Just adding `-g' to list of CFLAGS doesn't make gcc to generate debug
information required for macro expansion during debugging. Replacing
it with `-g3 -ggdb' rectifies this.

(configurator.c commit folded by Rusty)

8 years agoopt: suggest they look at ccan/autodata.
Rusty Russell [Wed, 7 Dec 2011 02:57:27 +0000 (13:27 +1030)]
opt: suggest they look at ccan/autodata.

Nice for distributed option declaration.

8 years agoautodata: add example, clean up description a little.
Rusty Russell [Wed, 7 Dec 2011 02:57:14 +0000 (13:27 +1030)]
autodata: add example, clean up description a little.

Also add #include to AUTODATA_TYPE example, so ccanlint doesn't try to
put it inside a function (which works, for this, but gives a warning
about unused variables).

8 years agotools/doc_extract: trim trailing blank lines on sections.
Rusty Russell [Wed, 7 Dec 2011 02:55:41 +0000 (13:25 +1030)]
tools/doc_extract: trim trailing blank lines on sections.

Makes things a bit neater.

8 years agoweb: publish See Also: for modules which supply it.
Rusty Russell [Wed, 7 Dec 2011 02:55:41 +0000 (13:25 +1030)]
web: publish See Also: for modules which supply it.

8 years agotools/doc_extract: simplify handling fields with spaces in them.
Rusty Russell [Wed, 7 Dec 2011 02:55:41 +0000 (13:25 +1030)]
tools/doc_extract: simplify handling fields with spaces in them.

eg. "See also" can match "see-also".

8 years agoccanlint: use ccan/autodata
Rusty Russell [Mon, 5 Dec 2011 12:48:50 +0000 (23:18 +1030)]
ccanlint: use ccan/autodata

Gets rid of generated file.

8 years agoautodata: stash pointers in a binary.
Rusty Russell [Mon, 5 Dec 2011 12:48:21 +0000 (23:18 +1030)]
autodata: stash pointers in a binary.

This is a more portable variant of the ELF section trick.

8 years agoconfigurator: HAVE_SECTION_START_STOP
Rusty Russell [Mon, 5 Dec 2011 12:17:46 +0000 (22:47 +1030)]
configurator: HAVE_SECTION_START_STOP

8 years agoptr_valid: test whether a ptr is valid.
Rusty Russell [Mon, 5 Dec 2011 12:16:46 +0000 (22:46 +1030)]
ptr_valid: test whether a ptr is valid.

Very slow, but sometimes you need to know without crashing.

8 years agoconfigurator: HAVE_PROC_SELF_MAPS
Rusty Russell [Mon, 5 Dec 2011 03:46:33 +0000 (14:16 +1030)]
configurator: HAVE_PROC_SELF_MAPS

8 years agoobjset: new module.
Rusty Russell [Sun, 4 Dec 2011 22:50:34 +0000 (09:20 +1030)]
objset: new module.

8 years agoccanlint.1: fix description of per-test options for tests_pass_valgrind.
Rusty Russell [Sun, 4 Dec 2011 22:49:34 +0000 (09:19 +1030)]
ccanlint.1: fix description of per-test options for tests_pass_valgrind.

8 years agodoc: man page for ccanlint.
Rusty Russell [Fri, 2 Dec 2011 03:15:08 +0000 (13:45 +1030)]
doc: man page for ccanlint.

8 years agoccanlint: remove argument to -k/--keep
Rusty Russell [Fri, 2 Dec 2011 03:37:24 +0000 (14:07 +1030)]
ccanlint: remove argument to -k/--keep

It's much easier to simply say "keep all", and it simplifies the code
quite a bit.

8 years agoccanlint: use -t for --target not --timeout
Rusty Russell [Fri, 2 Dec 2011 03:37:23 +0000 (14:07 +1030)]
ccanlint: use -t for --target not --timeout

I use --target far more than --timeout.

8 years agoccanlint: use traversal to trim tests we want to skip.
Rusty Russell [Fri, 2 Dec 2011 03:37:22 +0000 (14:07 +1030)]
ccanlint: use traversal to trim tests we want to skip.

Rather than using a separate index of tetst to skip, make sure tests
we don't want to run are removed from the graph of tests to start
with.

This means that the "skip" field is only set when we fail a test (and
thus must be reset when we test the next module).