summaryrefslogtreecommitdiff
path: root/etc
diff options
context:
space:
mode:
authorlevine <levine@ae88bc3d-4319-0410-8dbf-d08b4c9d3795>1997-11-14 06:04:50 +0000
committerlevine <levine@ae88bc3d-4319-0410-8dbf-d08b4c9d3795>1997-11-14 06:04:50 +0000
commit4c3f56249fabd12150d77567e578965c1c5928b6 (patch)
tree325df46f0aa3a2a4c5c9d77afc9e07c7288fd3a9 /etc
parenta121093fd633ed6bea8f663e892a019de0065f59 (diff)
downloadATCD-4c3f56249fabd12150d77567e578965c1c5928b6.tar.gz
added mutator/accessor form
Diffstat (limited to 'etc')
-rw-r--r--etc/ACE-guidelines.html14
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 &lt;object_addr_&gt; cache from &lt;host&gt; and &lt;port&gt;.
+
+ ACE_INET_Addr &object_addr (void);
+ // Returns the &lt;ACE_INET_Addr&gt; 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