[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Condor-devel] Clang compatibility fixes
- Date: Wed, 11 Jul 2012 10:21:13 -0500
- From: Greg Thain <gthain@xxxxxxxxxxx>
- Subject: Re: [Condor-devel] Clang compatibility fixes
The attached patches are necessary to compile Condor with Clang. Only
patch 2 is clearly correct (variable-length arrays are a GCC
extension). Patch 1 could be Clang bug, but I don't think so. Patch
3 is odd because the comment above the ThreadInfo class definition
strongly suggests that there should have been an operator== in the
first place.
Thanks for the patches! The first two look pretty safe, and we'll get
them into 7.9 right away. I think to think a bit about what we really
want operator== to mean for our thread wrappers.
-Greg