#1305 closed Patches (fixed)
Patch to improve support for the CodeGear (Borland) 5.9.2 compiler
| Reported by: | Nicola Musatti | Owned by: | Nicola Musatti |
|---|---|---|---|
| Milestone: | To Be Determined | Component: | parameter |
| Version: | Boost Development Trunk | Severity: | Problem |
| Keywords: | Cc: |
Description
The attached patches improve support for the latest CodeGear (Borland) compiler.
Attachments (2)
Change History (7)
by , 15 years ago
| Attachment: | parameter_boost071007.diff added |
|---|
by , 15 years ago
| Attachment: | parameter_libs071007.diff added |
|---|
comment:1 by , 15 years ago
| Owner: | changed from to |
|---|
comment:2 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:3 by , 15 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Nicola,
I'm not sure this is really the right patch. Shouldn't Borland have BOOST_NO_SFINAE set? Please feel free to close the ticket again if not.
comment:4 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
I'm aware that this patch is rather ugly. However bcc 5.9.2 does implement SFINAE, although it still has a few problems with it. I tried both alternatives with 1.34 and unsetting BOOST_NO_SFINAE while adding specific workarounds here and there was the combination that got best regression results.
I expect this is even more so with 1.35 as many libraries have since adopted SFINAE to tweak their interface. The availability of SFINAE is the single most common reason for tests that pass with 5.9.2, but fail with previous compilers.
comment:5 by , 15 years ago
OK, then if you can find the time, I would prefer then to have another configuration macro (parameter-library-specific if no other libs need it) for this purpose. Code repetition is evil, even PP code.
Applied