- changed status to wontfix
isolate structures to internal modules
Issue #24
wontfix
We pass miniat to everything, rather then just the minimum slice of the structure, so you get a lot of this:
void foo(miniat *m) {
m->foo.bar.baz = 7;
}
foo(m);
Where it should be more like this:
void foo(foo *f) {
f->bar.baz = 7;
}
foo(&m->foo);
We could start to move internal structures typedefs into these C files rather then the spew of headers (make whitebox testing harder).
Comments (1)
-
reporter - Log in to comment
This reason we wont change this is due to cyclic dependencies. We'll just leave this as is, it creates more of a mess trying to reorg it.