diff options
author | levine <levine@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 1997-11-14 06:04:50 +0000 |
---|---|---|
committer | levine <levine@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 1997-11-14 06:04:50 +0000 |
commit | 4c3f56249fabd12150d77567e578965c1c5928b6 (patch) | |
tree | 325df46f0aa3a2a4c5c9d77afc9e07c7288fd3a9 /etc | |
parent | a121093fd633ed6bea8f663e892a019de0065f59 (diff) | |
download | ATCD-4c3f56249fabd12150d77567e578965c1c5928b6.tar.gz |
added mutator/accessor form
Diffstat (limited to 'etc')
-rw-r--r-- | etc/ACE-guidelines.html | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/etc/ACE-guidelines.html b/etc/ACE-guidelines.html index 5a8efe65ba1..f41ca66a479 100644 --- a/etc/ACE-guidelines.html +++ b/etc/ACE-guidelines.html @@ -142,7 +142,7 @@ to Graham for providing the OSE tools!<p> { ACE_ASSERT (spawn () == 0); } - </pre> + </pre><p> Similarly, <strong><code>if</code></strong> statements should have a space after the ``<strong>if</strong>'', and no spaces just after @@ -182,6 +182,18 @@ to Graham for providing the OSE tools!<p> properly. Some compilers, such as <code>g++</code> with the <code>-Wall</code> option, will issue warnings for violations.<p> + <li>Mutators and accessors should be of this form:<p> + + <pre> + void object_addr (const ACE_INET_Addr &); + // Sets <object_addr_> cache from <host> and <port>. + + ACE_INET_Addr &object_addr (void); + // Returns the <ACE_INET_Addr> for this profile. + </pre><p> + + instead of the ``set_'' and ``get_'' form.<p> + <li>Never use <strong><code>delete</code></strong> to deallocate memory that was allocated with <strong><code>malloc</code></strong>. Similarly, never associate <strong><code>free</code></strong> with |