- Description has changed:
Diff:
~~~~
--- old
+++ new
@@ -12,7 +12,8 @@
With this as the background, the following further cleanup and streamlining is
desired to enable leap code to be unit-test ready:
-- Remove any leftover dead code. I can see that for eg:- hj_stack.c, and some
functions in os_defs.c can be removed.
+- Remove any leftover dead code. I can see that for eg:- hj_stack.c, and some
functions in os_defs.c can be removed. The
[cppcheck](http://cppcheck.sourceforge.net/) tool can be helpful to identify
some of the dead code. The OpenSAF makefiles have been updated so that you can
type "make cppcheck" to run this tool.
- Move code related to one particular functionality into one file.
- Identify functions that could be equivalent to APIs around which unit tests
could be performed.
- Streamline .h files(same as above for .c files)
+- Remove preprocessor macros and instead call the underlying functions directly
~~~~
- **Milestone**: future --> 5.1.FC
---
** [tickets:#537] make base code unit test friendly and remove any dead code**
**Status:** accepted
**Milestone:** 5.1.FC
**Created:** Thu Aug 08, 2013 11:55 AM UTC by Mathi Naickan
**Last Updated:** Wed Mar 23, 2016 05:08 AM UTC
**Owner:** Ramesh
A great deal of legacy leap code cleanup was done through ticket
http://devel.opensaf.org/ticket/652.
It can be seen that LEAP is *no more* a portation layer that it used to be, but
is now providing functionality more of an *utility layer*.
Currently leap is acting as a bunch of wrapper/utility code providing the
following functionalities:
- encode/decode routines
- buffer management
- patricia db
- wrappers for tasks
- mailbox
- timers
- other utilties
With this as the background, the following further cleanup and streamlining is
desired to enable leap code to be unit-test ready:
- Remove any leftover dead code. I can see that for eg:- hj_stack.c, and some
functions in os_defs.c can be removed. The
[cppcheck](http://cppcheck.sourceforge.net/) tool can be helpful to identify
some of the dead code. The OpenSAF makefiles have been updated so that you can
type "make cppcheck" to run this tool.
- Move code related to one particular functionality into one file.
- Identify functions that could be equivalent to APIs around which unit tests
could be performed.
- Streamline .h files(same as above for .c files)
- Remove preprocessor macros and instead call the underlying functions directly
---
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.
------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are
consuming the most bandwidth. Provides multi-vendor support for NetFlow,
J-Flow, sFlow and other flows. Make informed decisions using capacity
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
_______________________________________________
Opensaf-tickets mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
