Skip to content

Conversation

luis-valverde
Copy link

This code review includes:

  1. Migration of libviface C++ to C code.
  • Created some examples of usage.
  • Created a test program to verify its functionality.
  1. Build system fix to compile C code and its bindings.
  • Some targets were added in the auto generated Makefiles to choose the library to compile/install.
    For example: make, make libviface, make install libviface, make doc, make libvifacecpp.
  • It creates Doxygen documentation for C and C++ library.
  • Uncrustify was modified to check C files (.c and .h).

valverlu added 3 commits January 14, 2016 23:26
Migration of libviface C++ library to C code. It includes
several examples and a test program to verify the functionality.
Fix build system to compile libviface C library and bindings
(currently C++). Some targets were added in the autogenerated makefiles
to choose the library to compile or install. For example:
make, make libviface, make libvifacecpp, make install libviface.
Incorporate code review comments suggested by Carlos Jenkins
to stick with the guidelines styles.
@carlos-jenkins carlos-jenkins self-assigned this Feb 5, 2016
IceMupppet added 3 commits February 10, 2016 11:37
Change data types of some input parameters and logic methods to make
it work with the GO wrapper and avoid APR dependencies from GO code.
Creation of a libviface wrapper in GO language. It includes the
wrapper GO code and some basic usage examples.
Fix build system to compile libviface GO wrapper and examples.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants