Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

This adds patch to the list of installed software for android.
Signed-off-by: George Nash <george.nash@intel.com>

When originally written the whitespace_commit_checker did not save
a list of the files that had bad formatting. The idea was that
the developer could run clang-format on any of the files and it
would not be a big deal.  However, if the change is large this
results in the tool saying you have x file(s) with bad formating
but then shows every file that was changed. Just showing the files
with bad formatting seems to make more since. It lets the developer
focuse only on the files that need fixing.

Also fixed typo in the output text.
Signed-off-by: George Nash <george.nash@intel.com>

In addition to adding the introspection data this makes
the OC_IDD_API a default option for the Linux build.

This change additionally adds a new introspection client that retrieves
the introspection data from servers and prints it out.
Signed-off-by: George Nash <george.nash@intel.com>

Add inline documentation for whitespace_commit_checker

Add the build_doc.sh and whitespace_commit_checker.sh
scripts to the README file.
Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: George Nash <george.nash@intel.com>

Fixes removing observation over TCP connection, when a peer close
the connection.

Signed-off-by: George Nash <george.nash@intel.com>

This uses the git --diff-filter option to limit the files
show in the diff to only deleted files. The diff order had
to be changed to bee HEAD~1 HEAD instead of HEAD HEAD~1 to
properly detect the files that were deleted in the last
commit.
Signed-off-by: George Nash <george.nash@intel.com>

This adds the clang-format tool to the CI build. This enables
catching formating errors on from contributers that have not
installed the pre-commit hook.

This change was integrated into the same build as the doxygen
build script.  This was done to reduce the number of CI build
pipelines needed to pass the build.

Copyright block added to the build_doc.sh script. The scripts
also print out some blocks of text to indicate there run status
making it easer to instantly see the results of the build
output.

The new script added to the .gitignore if it is found in the
top of the project  incase users want to use it localy on
their own computers. Similar to the reason the _clang-format
file is in the ignore file.
Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: George Nash <george.nash@intel.com>

The codan static analysis tool is built into eclipse CDT which
I used time to time.

Most of these warnings were suggestions to surround bitwise
operations with parenthesis.  Some of the suggetions were
about return values for pthread code in the network_event_thread
function.

In addition a white space error was fixed in the documentation
in oc_api.h

The most common warning is about not break statment at the end
of case statments. All but one of these was done purposly.
Signed-off-by: George Nash <george.nash@intel.com>

This add a shell script that can be run to build
documentation if the Doxygen.log file is empty the script
will cat the contents to the shell so it can be viewed.

Also add this script to the .gitlab-ci.yml.
Signed-off-by: George Nash <george.nash@intel.com>

This adds documentation for:
 - oc_new_resource
Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: Oleksandr Andrieiev <o.andrieiev@samsung.com>

Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

This adds doxygen documentation for:
 - oc_random_pin_cb_t - callback function
 - oc_set_random_pin_callback
 - oc_reset
 - oc_reset_device
Signed-off-by: George Nash <george.nash@intel.com>

Doxygen documentation added for
 - oc_set_factory_presets_cb
 - oc_factory_presets_cb_t
Signed-off-by: George Nash <george.nash@intel.com>

This enables getting just the uri portion of an endpoint from
an endpoint string.

This is useful for a few reasons.
- The introspection contains an endpoint string to
  the introspection data. Since we typically already have the
  endpoint there is no need to used the oc_string_to_endpoint
  function and allocate another copy of the endpoint when we are
  only interested in the path part of the string.

- Language bindings with Java. Functions that return multiple
  values through variables require unusual workarounds to get
  a clean Java API.  By making this availible in a separate
  function we can make two functions with only one
  responability. Making the Java APIs cleaner.

While testing the oc_endpoint_string_parse_path function
multiple failures showed up in the StringToEndpoint unit test.
Once I figured out the failures were due to the fact that my
build did not have IPV4=1 I updated the StringToEndpoint test so
it will not test IPV4 strings if OC_IPV4 is not defined.
Signed-off-by: George Nash <george.nash@intel.com>

Persistant storage is currently only enabled if OC_SECURITY
is defined.  This has lead implementation problems when
implementing features like introspection, cloud, or
software-update which also require persistant storage.

This commit replaces the `#ifdef OC_SECURITY` with
`#ifdef OC_STORAGE` in oc_storage.c.

Inside config.h OC_STORAGE is defined if OC_SECURITY, OC_IDD_API
OC_SOFTWARE_UPDATE, or OC_SECURITY is defined.

Also updated samples that call oc_storage_config so they call it
if OC_STORAGE is defined instead of OC_SECURITY.

Removed the dummy storage implmentation from the
oc_cloud_storage.c since OC_STORAGE is now defined by default
when OC_CLOUD is defined.

Some preprocessor check were added to check it one of the
features that require OC_STORAGE is set but OC_STORAGE is not
set. This will cause a build error and inform developers to
check oc_config.h to make sure OC_STORAGE is set there.

Change-Id: I9a9242349e939393f190cab373fe7d0252ae656c
Signed-off-by: George Nash <george.nash@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>

Signed-off-by: Kishen Maloor <kishen.maloor@intel.com>