| T.R | Title | User | Personal
Name | Date | Lines |
|---|
| 3526.1 | Looks like progress... | DECC::SULLIVAN | Jeff Sullivan | Thu Apr 03 1997 21:10 | 26 |
| I'm not sure what the percieved problem is, but it looks like the newer version
of the header file has a more portable conditional.
I tested this on UNIX using DEC C++ V5.5-004
% cat t.cxx
#if defined (__DECCXX) && (defined(__CFRONT) || defined(__MS))
Passes test 1
#endif
#if defined (__DECCXX) && (__CFRONT || __MS)
Passes test 2
#endif
% cxx -E t.cxx | grep Pass
% cxx -E t.cxx -cfront | grep Pass
Passes test 1
Passes test 2
% cxx -E t.cxx -ms | grep Pass
Passes test 1
Passes test 2
Is the user concerned that the new conditional is not doing th right thing?
-Jeff
|
| 3526.2 | | DECCXL::OUELLETTE | crunch | Thu Apr 03 1997 22:22 | 4 |
| I think the user may be conserned that "#define __MS 0" is used somewhere.
For that there is a difference between the two #if's.
R.
|
| 3526.3 | verify please | HNDYMN::MCCARTHY | A Quinn Martin Production | Fri Apr 04 1997 13:17 | 11 |
| Could you please confirm the line:
>> #if defined (__DECCXX) && (defined(__CFRONT) || defined(__MS))
in the customers's version? Also double check the the OS version for me.
The reason I ask is that when I checked all the version of that header file
that we have ever created in our CMS library, the above line never
appears.
Brian J.
|
| 3526.4 | customer modification | CSC32::I_WALDO | | Fri Apr 04 1997 16:50 | 7 |
| re .3
>> #if defined (__DECCXX) && (defined(__CFRONT) || defined(__MS))
When I questioned the customer again I discovered that the above was
a modification which he put in stl_macros to make it behave that way he
considered to be correct.
|
| 3526.5 | what's wrong with it? | HNDYMN::MCCARTHY | A Quinn Martin Production | Fri Apr 04 1997 17:08 | 7 |
| I have to ask then, what behavior where they seeing that they considered
incorrect?
Were they defining __MS to some value? Does a compile with /list/show=all
show __MS defined (listing files show what macros are in effect)?
Brian
|
| 3526.6 | customer response to .5 | CSC32::I_WALDO | | Mon Apr 07 1997 20:44 | 34 |
|
>What problem were you resolving when you changed the line from
>
>#if defined (__DECCXX) && (__CFRONT || __MS) to
>
>#if defined (__DECCXX) && (defined(__CFRONT) || defined(__MS))
>
$ create t.c
#if defined (__DECCXX) && (__CFRONT || __MS)
static int i;
#endif
$ cxx t.c/warn=enable=all
#if defined (__DECCXX) && (__CFRONT || __MS)
...........................^
%CXX-I-UNKNOWNMACRO, "__CFRONT" is not currently defined as a macro. It has
been replaced by the constant zero.
at line number 1 in file DKA0:[TUCKER.PSTORE]T.C;4
#if defined (__DECCXX) && (__CFRONT || __MS)
.......................................^
%CXX-I-UNKNOWNMACRO, "__MS" is not currently defined as a macro. It has been
replaced by the constant zero.
at line number 1 in file DKA0:[TUCKER.PSTORE]T.C;4
Sorry about this. You have to use /WARN=ENABLE=ALL.
Thanks,
Roger Tucker (tucker@spike.cs.mci.com, or roger.tucker@mci.com)
|
| 3526.7 | Looks reasonable | HNDYMN::MCCARTHY | A Quinn Martin Production | Mon Apr 07 1997 22:06 | 3 |
| Thanks for the details.
bjm
|
| 3526.8 | FN | HNDYMN::MCCARTHY | A Quinn Martin Production | Tue Apr 08 1997 15:26 | 5 |
| We have made the change in our sources and it will be part of the V5.6 release.
Thanks for pointing it out.
Brian J.
|