Merge pull request #1844 from jrw972/monterey
[ACE_TAO.git] / TAO / examples / Advanced / ch_21 / CHANGES
blobec79d72c2d71adb4bdafad48670b557cf5b55861
1 Changelog for chapter 8 and 10  example in "Advanced CORBA
2 Programming with C++" by Michi Henning and Steve Vinoski,
3 Copyright 1999, Addison-Wesley, Reading, MA.  The following
4 changes have been made to the book's source code to make the
5 example work with TAO and with various platforms and compilers.
6 ______________________________________________________________________________
7 ESSENTIAL CHANGES:
8   1. Changed filename extensions from .hh and .cc to .h
9      and .cpp, and made all necessary changes in file content.
11   2. a) Commented-out "#include <corba/poaS.h>" in server.h
13      b) Added #include "icp.h" to server.h.
15      c) Unnecessary includes of <iostreams.h>, <assert.h>,
16         <fstream.h> and <stdlib.h> must be removed or moved to
17         follow all other includes in server.h, server.cpp,
18         client.cpp, and icp.cpp.
20      d) The following files may need their includes further rearranged
21         to avoid warnings and errors (g++). The following orderings
22         seem to work fine:
23           - server.cpp: "server.h", <algorithm>, "icp.h",
24                         <strstream.h>
25           - server.h:  "CCSS.h", "icp.h",<ace/Synch_T.h>, <map>,
26                         set, <list>
27           - icp.cpp:   <string>, <map>, <algorithm>, <stdlib.h>,
28                        "icp.h"
30   3. Added CORBA_Environment variable to
31      DeviceLocator::preinvoke() and ::postinvoke()
32      in server.h and server.cpp to match the signatures of parent
33      methods in TAO's ServantLocator class.
35   4. removed if 0 code surrounding
36      operator<<(ostream & os, const CCS::Controller::EChange & ec)
37      definition in server.cpp and client.cpp.
39   5. commented-out "delete this;" from method
40      Thermometer_impl::remove() in server.cpp.
41      (Change should soon be reflected in authors' code)
43   6. member initializers reordeder in Thermometer_impl::
44      Thermometer_impl() in server.cpp to match the declaration order
45      in server.h
47 ______________________________________________________________________________
48 ADDITIONAL CHANGES:
49   7. Added .in() to _var parameters wherever needed:
50      -client.cpp:
51          4 times in operator<<(ostream, CCS::Thermometer_ptr)
52          6 times in main().  5 are is_nil(), 1 is _narrow()
53          2 times added .inout() to _var parameters in main() for calls
54            to set_tmp(..)
55      -server.cpp,
56          2 changes in make_dref()
57          3 changes in Controller_impl::Controller_impl()
58          4 changes in Controller_impl::~Controller_impl()
59          1 change in Controller_impl::create_thermometer
60          2 changes in Controller_impl::create_thermostat()
61          1 change in Controller_impl::list()
62          3 changes in Controller_impl::find
63          7 changes in main()
65   8. created needed file at /tmp/CCS_DB. (location is hard-coded)
66      To run, move CCS_DB from this directory to /tmp on the
67      local machine.
69   9. fixed warnings by commented out unused parameters in
70        -DeviceLocator_impl::postinvoke(..) in server.h
71        -DeviceLocator_impl::preinvoke(..) in server.cpp
73   10. MAX_EQ_SIZE changed from int to unsigned int
74      in class DeviceLocator_impl in server.h.
76   11. added cast to unsigned long in ~ICP_Persist() in icp.cpp:
77       db << (unsigned long)(i->second.type) << endl;
79 ______________________________________________________________________________
80 To Do:
81   - fix error on 2 cases of fstream.close()
82     in server.cpp and icp.cpp
83 ____________________________________________________________