changeset: 4810:7938a6a570f4
parent: 4806:d8a9d45dfbdf
user: Hans Feldt <osafde...@gmail.com>
date: Fri Jan 10 15:43:33 2014 +0100
summary: amfnd: remove unneeded lock [#713]
changeset: 4811:eb57695a171b
tag: tip
user: Hans Feldt <osafde...@gmail.com>
date: Fri Jan 10 15:43:34 2014 +0100
summary: amf: remove amf_ipc.h [#713]
---
** [tickets:#713] AMF refactoring for 4.5**
**Status:** accepted
**Created:** Fri Jan 10, 2014 07:05 AM UTC by Hans Feldt
**Last Updated:** Fri Jan 10, 2014 07:05 AM UTC
**Owner:** Hans Feldt
This is a 4.5 ticket for continued code re-factoring of the AMF service for
4.5. The work started with #94 in 4.4.
So far no really no C++ features has been used. Files has been renamed so that
the C++ compiler is used and new/delete is used instead of malloc/free (where
appropriate).
In this ticket it is time to make use of C++ features to get a more
maintainable and understandable code base. We should aim for many small changes
instead of few big ones.
Some things to consider for the C++ conversion:
- Clean run of google's cpplint on the code base
- Use STL containers
- patricia tree converted to maps
- linked links converted to vectors (or lists when needed)
- Convert model derived C structs to classes and change functions into methods
- Change macros to (inline) methods
- Change bit fields (flags) to boolean attributes
- Use references instead of pointers
- ...
Non C++ related changes:
- split up long functions into smaller ones doing one things and not many
- use const as much as possible
- ...
This is just a starting point, please comment
---
Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/opensaf/tickets/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a
mailing list, you can unsubscribe from the mailing list.------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today.
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-tickets mailing list
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
