diff options
author | brunsch <brunsch@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2001-03-28 19:45:21 +0000 |
---|---|---|
committer | brunsch <brunsch@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2001-03-28 19:45:21 +0000 |
commit | 06f7ae1432e26b0cb3f59991adabb1caa09d4c19 (patch) | |
tree | 20b7b0fb6e6e6aa89667f4b91e6266fc3f8a5e00 /ChangeLogs | |
parent | 969b9b3e079cf2bbc5a4f18cf4eb60d525dea2f8 (diff) | |
download | ATCD-06f7ae1432e26b0cb3f59991adabb1caa09d4c19.tar.gz |
ChangeLogTag:Wed Mar 28 11:37:09 2001 Fuzz Master D <brunsch@uci.edu>
Diffstat (limited to 'ChangeLogs')
-rw-r--r-- | ChangeLogs/ChangeLog-02a | 34 | ||||
-rw-r--r-- | ChangeLogs/ChangeLog-03a | 34 |
2 files changed, 68 insertions, 0 deletions
diff --git a/ChangeLogs/ChangeLog-02a b/ChangeLogs/ChangeLog-02a index c30d7754ba2..f0729ef9dfa 100644 --- a/ChangeLogs/ChangeLog-02a +++ b/ChangeLogs/ChangeLog-02a @@ -1,3 +1,37 @@ +Wed Mar 28 11:37:09 2001 Fuzz Master D <brunsch@uci.edu> + + * bin/fuzz.pl: + + There are several instances where fuzz gives false + positives, and the code is impossible to fix without + creating compiler warnings. Example: + + void function (CORBA::Environment &ACE_TRY_ENV) + { + #if FOO + some_other_function (ACE_TRY_ENV); + ACE_CHECK; + #else + ACE_UNUSED_ARG (ACE_TRY_ENV); + #endif + } + + So normally ACE_UNUSED_ARG errors can be fixed by keeping + the CORBA::ENvironment unnamed and removing the ACE_UNUSED_ARG, + but in this case it isn't possible. So fuzz now accepts + an ignore command for single line use. The above can be fixed + by changing to: + + void function (CORBA::Environment &ACE_TRY_ENV) + { + #if FOO + some_other_function (ACE_TRY_ENV); + ACE_CHECK; + #else + ACE_UNUSED_ARG (ACE_TRY_ENV); // FUZZ: ignore check_for_ace_check + #endif + } + Tue Mar 28 13:03:07 2001 Chad Elliott <elliott_c@ociweb.com> * bin/make_pretty.pl: diff --git a/ChangeLogs/ChangeLog-03a b/ChangeLogs/ChangeLog-03a index c30d7754ba2..f0729ef9dfa 100644 --- a/ChangeLogs/ChangeLog-03a +++ b/ChangeLogs/ChangeLog-03a @@ -1,3 +1,37 @@ +Wed Mar 28 11:37:09 2001 Fuzz Master D <brunsch@uci.edu> + + * bin/fuzz.pl: + + There are several instances where fuzz gives false + positives, and the code is impossible to fix without + creating compiler warnings. Example: + + void function (CORBA::Environment &ACE_TRY_ENV) + { + #if FOO + some_other_function (ACE_TRY_ENV); + ACE_CHECK; + #else + ACE_UNUSED_ARG (ACE_TRY_ENV); + #endif + } + + So normally ACE_UNUSED_ARG errors can be fixed by keeping + the CORBA::ENvironment unnamed and removing the ACE_UNUSED_ARG, + but in this case it isn't possible. So fuzz now accepts + an ignore command for single line use. The above can be fixed + by changing to: + + void function (CORBA::Environment &ACE_TRY_ENV) + { + #if FOO + some_other_function (ACE_TRY_ENV); + ACE_CHECK; + #else + ACE_UNUSED_ARG (ACE_TRY_ENV); // FUZZ: ignore check_for_ace_check + #endif + } + Tue Mar 28 13:03:07 2001 Chad Elliott <elliott_c@ociweb.com> * bin/make_pretty.pl: |