Hi Kinkie,

Thanks for the structs.h removal. It does seem to have got past the old FreeBSD clang error as I suspected, but is now finding at least one other identical bug still in trunk.

Can you please add the FreeBSD clang build to your sourceformat for the next round of polishing please?

I'm procrastinating on the 3.3 branching event but would still like to see it happen by the end of this weekend.

Amos


On 04.10.2012 12:48, [email protected] wrote:
See

<http://build.squid-cache.org/job/3.HEAD-amd64-FreeBSD-9.0-clang/13/changes>

Changes:

[Francesco Chemolli] sourceformat: split protos.h into more specific
headers, change many functions' likage to c++.

[Amos Jeffries] Bug 3661: Segmentation fault when using more than 1 worker

Previous change to Ipc::Mem::PageStack making variable arrays into
dynamically-allocated arrays added an allocate after use bug.

------------------------------------------
[...truncated 3298 lines...]

In file included from ../../../src/comm/AcceptLimiter.cc:5:
../../../src/fde.h:115:5: error: struct '_fde_disk' was previously
declared as a class [-Werror,-Wmismatched-tags]
    struct _fde_disk disk;
    ^
../../../src/fde.h:47:7: note: previous use is here
class _fde_disk {
      ^
../../../src/fde.h:115:5: note: did you mean class here?
    struct _fde_disk disk;
    ^~~~~~
    class
1 error generated.
*** Error code 1



Reply via email to