changeset:   5364:c7597d6d9369
user:        [email protected]
date:        Thu Jun 05 10:00:05 2014 +0530
summary:     amfd : use db_template in AVD_SVC_TYPE_CS_TYPE [#713]

changeset:   5365:801a840ae92c
user:        [email protected]
date:        Thu Jun 05 10:00:26 2014 +0530
summary:     amfd : use db_template in AVD_SVC_TYPE [#713]

changeset:   5366:ad6691bebb47
user:        [email protected]
date:        Thu Jun 05 10:00:42 2014 +0530
summary:     amfd : use db_template in avd_cstype_t [#713]

changeset:   5367:0dbd27a4b4f6
user:        [email protected]
date:        Thu Jun 05 10:00:58 2014 +0530
summary:     amfd : use db_template in AVD_CTCS_TYPE [#713]

changeset:   5368:212808eaf580
user:        [email protected]
date:        Thu Jun 05 10:01:14 2014 +0530
summary:     amfd : use db_template in AVD_CSI [#713]

changeset:   5369:e0134e43bf55
user:        [email protected]
date:        Thu Jun 05 10:01:29 2014 +0530
summary:     amfd : use db_template in AVD_COMP_TYPE [#713]

changeset:   5370:3fb70b791371
user:        [email protected]
date:        Thu Jun 05 10:01:44 2014 +0530
summary:     amfd : use db_template in AVD_COMPCS_TYPE [#713]

changeset:   5371:7a590b3d4c83
user:        [email protected]
date:        Thu Jun 05 10:01:58 2014 +0530
summary:     amfd : use db_template in AVD_SG [#713]

changeset:   5372:92daae1a8d05
user:        [email protected]
date:        Thu Jun 05 10:02:15 2014 +0530
summary:     amfd : use db_template in avd_sutype [#713]

changeset:   5373:cf0bbe1f73f7
user:        [email protected]
date:        Thu Jun 05 10:02:29 2014 +0530
summary:     amfd : use db_template in AVD_SUTCOMP_TYPE [#713]

changeset:   5374:e7f0a468e25a
user:        [email protected]
date:        Thu Jun 05 10:02:45 2014 +0530
summary:     amfd : use db_template in AVD_AMF_SG_TYPE [#713]

changeset:   5375:4ae423baafc2
tag:         tip
user:        [email protected]
date:        Thu Jun 05 10:03:01 2014 +0530
summary:     amfd : use db_template in AVD_COMP [#713]


[staging:c7597d]
[staging:801a84]
[staging:ad6691]
[staging:0dbd27]
[staging:212808]
[staging:e0134e]
[staging:3fb70b]
[staging:7a590b]
[staging:92daae]
[staging:cf0bbe]
[staging:e7f0a4]
[staging:4ae423]




---

** [tickets:#713] AMF refactoring for 4.5**

**Status:** review
**Milestone:** 4.5.FC
**Created:** Fri Jan 10, 2014 07:05 AM UTC by Hans Feldt
**Last Updated:** Tue Jun 03, 2014 04:54 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.

An ordered list of things to work on:
1. Use bool which is a native type in C++ (and remove SaBoolT)
2. Reduce number of casts in the code (introduced because of C++ files)
2. Use stl::maps (instead of patricia trees)
3. Use stl::vector/list (instead of legacy/home made lists)

Other yet non prioritized items:
- Remove use of EDU and do direct encode/decode
- Clean run of google's cpplint on the code base
- Convert model derived C structs to classes and change functions into methods
- Change macros to (inline) methods
- Change bit fields (flags) to boolean attributes (see 
https://sourceforge.net/p/opensaf/tickets/717/#d4b2)
- Use references instead of pointers
- ...

Non C++ related changes:
- split up long functions into smaller ones doing one things and not many
- use pmccabe complexity analysis to aid refactoring
- set a goal for pmccabe complexity
- use const as much as possible
- ...




---

Sent from sourceforge.net because [email protected] 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.
------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and their 
applications. Written by three acclaimed leaders in the field, 
this first edition is now available. Download your free book today!
http://p.sf.net/sfu/NeoTech
_______________________________________________
Opensaf-tickets mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

Reply via email to