Merge "OCStartPresence now can be called when the server comes up for the first time, when it comes up from offline, or when it reenters network."

OCStartPresence now can be called when the server comes up for the first time, when it comes up from offline, or when it reenters network.

Similarly, OCStopPresence can be called when the server is terminating, going offline or when it leaves network.

Change-Id: I610979c26c26c1a7c0fcd4fb95294c5dedaeb4e4

Gave typedefs to every use of std::function in the C++ stack.  Only existing use of std::function< without a typedef is a comment in OCClientTestApp.cpp
Patch 2: Sudarshan's review fixes
Change-Id: I8c62347c1091a2225d5db1b0bda5221a3346586e

Merge "Get/Put overload implementation in OCResource.  Allows for specifiying interface and type."

Merge "Patch 1 : Adding Arduino WiFi support. This requires updated Arduino WiFi Shield firmware Patch 2 : Removed ocserver_wifi app and incorporated wifi changes in the single ocserver application.    Updated location of Time library in arduinomega and arduinodue properties file.    Added some more comments in sample app.    Added license info in ocserver which was missing."

Merge "Fix for arduinomega.properties. It was pointing at an invalid location for avrdude with arduino 1.0.5."

Merge "Stack/Observer: Free OCEntityHandlerRequest object after sent notification"

Get/Put overload implementation in OCResource.  Allows for specifiying interface and type.

Patch 2: fixing the emplace for a crappy old compiler
Patch 3: Sashi's comments
Patch 4: Sudarshan's comments
Change-Id: Ic1e9c07424229325443c532bcb03c67c5bfce6f1

Patch 1 : Adding Arduino WiFi support. This requires updated Arduino WiFi Shield firmware
Patch 2 : Removed ocserver_wifi app and incorporated wifi changes in the single ocserver application.
  Updated location of Time library in arduinomega and arduinodue properties file.
  Added some more comments in sample app.
  Added license info in ocserver which was missing.

Change-Id: Ie852a61ff3b0b5707f79899427668698792d1fcc

Fix for arduinomega.properties. It was pointing at an invalid
location for avrdude with arduino 1.0.5.

Change-Id: I15878e29a2daa341886d0b8e0fa2010f21771f32

Updated the name of the API to unregisterResource. And this API returns
whatever result it got from the stack. NOTE: Only this API has been changed.

Added C++ API for unregistering a resource
Patch 3: Rebase
Patch 4: added test code, fixed ocicuc bug

Change-Id: Ida95886fcbbaceab349bc127ae4b4e44ff823828

Merge "OCLib is now called src. Now liboc.a is generated instead OCLib.a ."

Using the compiler that comes with Arduino 1.0.5, it
complains that the token in the async struct cannot
have a size of zero.

I fixed it, since it's the end of the struct, this is okay.

Patch 2: Updated with respect to Sachin's comment review.

Patch 3: When building in debug mode, the compiler that
comes with Arduino 1.0.5 complains about the tokens defined
in pdu.h as well. In this patch as well, also removed errors
found when building in debug. Rebased on Sachins changeset.

Change-Id: I568481562af2a614624608851def5bf5aa68287a

OCLib is now called src. Now liboc.a is generated instead OCLib.a .

Change-Id: Id9aa7652987a0915842fa6441efa118809a0b01b

Fixing broken TB stack due to compilation errors for Arduino platforms.

Change-Id: Id211b8f03ddb07dd8c50148ece513bd2548b447f

OCResourceRespnse: JSON value should be wrapped in double quotes

C++ stack packages the JSON payload and send it as payload within CoAP.
And the JSON format said, "A string is a sequence of zero or more
Unicode characters, wrapped in double quotes, using backslash escapes."

Change-Id: I9641df115945dead2dcd41a08576047dcdd01d98

Minor formatting fix for the release notes.

Change-Id: Ieee8839edbaef07ae5ab18efb81162c99d70d440

Stack/Observer: Free OCEntityHandlerRequest object after sent notification

Change-Id: I539778ed8ad65c45635ff64f0b71a5ade1743894

Fixed a minor bug in C stack and minor cleanup from Presence Notification.

Release notes from Sudarhan, with minor comments from Sashi.

OCPlatform.h with better comments from Sudarshan.

Change-Id: Ie1950b6d3486e3a2314ac7428350f61bba20351f

Includes C++ Presence Notification changes from Erich, Jesse, Sudarshan and Sashi.

Change-Id: I113cd2dc3051bace0d0111289184299e5fd21f87

Merge "Patch 1 (Yamin): This is where all work is merged from Sashi, Joey and myself"

Patch 1 (Yamin): This is where all work is merged from
Sashi, Joey and myself

Patch 2 (Sashi): Looks like some debugging printf work. Should
be removed. Sashi?

Patch 3 (Joey): Applying changes as per review by Chanchala in parent
changeset Change-Id: I2625063e4a6bdfa53e3954310a060ea3c0ee5d3c

Patch 4 (Joey & Yamin): Squash all of Active Discovery and rebase
on to master.

Patch 5 (Joey): Cleaned up warnings and refactored merge conflicts
that caused build server to fail.

Patch 6 (Yamin): More debugging and restructuring... more work needed specifically for the TTL logic

Patch 7 and 8 (Yamin): Fixed TTL logic and presence cancelation, more clean up to come...

Patch 9: Added the random TTL logic and cleaned up.

Change-Id: I4bbd72e066ed4b50f4d2be52367844918a29823a

Checkpoint: monoprocess bug fixed-- need to improve CLI handling.

Change-Id: I73ad330df3716cda2f5907a62dc09478bc03fe5e

Merge "Updated Makefile to support different build targets. Removed delegating ctors (work-around non-conforming compiler). Configuration requires explicit ctor. Restructured client/server program; review fixes; hopefully extricated from the dangerous world of b0rked commits."

Updated Makefile to support different build targets.
Removed delegating ctors (work-around non-conforming compiler).
Configuration requires explicit ctor.
Restructured client/server program; review fixes; hopefully
extricated from the dangerous world of b0rked commits.

Change-Id: I68462b3d0306d969ba92b5e7affb515aebe03be3

Partial fix for Jira issue IOT-24.
    Upon a successful observe registration/dereg new return values have been added.
    These values are for success cases and need to be considered along with OC_STACK_OK.

    Patch set #2:
    1) ObserveReq struct has a member holding the result of registering and observer
    2) OCStackResult has only one success code possible

    Change-Id: I3b6fc51d64e602b8dbbca85e0e669fd7d7d9f230

Patch Set #3:
1) responded to comments
2) rebased on master

Change-Id: I2ee4d6a641b9b17adf69efca5da75cf3a4e4d3b7

Merge "Patch 1 :Added Arduino Due support Patch 2 :Modified code for review comments.   Added more informative info in README files for Arduino Due builds.   Fixed unit test makefiles to accomodate the 'sole' local.properties file. Patch 3 :Fixed compilation errors in unused tests.(Those test   should be removed from the repo). This would make the Build Server   stop complaining. Patch 4 (Joey): Rebased on "

Patch 1 :Added Arduino Due support
Patch 2 :Modified code for review comments.
 Added more informative info in README files for Arduino Due builds.
 Fixed unit test makefiles to accomodate the 'sole' local.properties file.
Patch 3 :Fixed compilation errors in unused tests.(Those test
 should be removed from the repo). This would make the Build Server
 stop complaining.
Patch 4 (Joey): Rebased on master.

Change-Id: Ib4a0e838a5519712bf3b0138e01abf74893593fb

Finished doing TODOs in ValidateQuery for container resources with default entity handler.
Updated Sashi's review.

Change-Id: I732bc7bf63acd7522051ce39a9f80c6a41df9b52

Adding API changes needed for Active Discovery in the TB Stack layer.

Change-Id: I8c457cccb2bbbfefedc6aa27a89bdc47ad7aa061

Fix for TB Unit Tests.  This should make Jenkins build again.

Change-Id: I1dc0e13d9e2a56f5799f352d66b0916df869c738

Merge "Applying a quick fix for "prog_char is deprecated" warning in Arduino. As 'prog_char' is completely removed in later versions of avr-gcc/g++, people with Ubuntu 14.04 won't be able to build Arduino out of the box because of this. This commit clears up some warnings AND allows Ubuntu 14.04 to build our stack for Arduino (However, the question as to whether we should update our versions of Ubuntu and/or Arduino are still up in the air.)."

IOT-25 Jira Bug : RightShift on signed numbers have an undefined behavior. This caused send operations
    to fail randomly on Arduino Mega boards.

Change-Id: I6b0a7f620b6ef35f67161d5ecb7521cbef4108c8

Added helper function to escape a special character in a string.

Change-Id: Icdca0fe4fb1c959b374cd6afea57272576e86061

Applying a quick fix for "prog_char is deprecated" warning
in Arduino. As 'prog_char' is completely removed in later
versions of avr-gcc/g++, people with Ubuntu 14.04 won't be
able to build Arduino out of the box because of this. This
commit clears up some warnings AND allows Ubuntu 14.04 to
build our stack for Arduino (However, the question as to
whether we should update our versions of Ubuntu and/or
Arduino are still up in the air.).

Majorly, this enables Shamit to keep moving along. He is
currently making sure all of our documentation for building
the stack for Arduino is up to par. He will also be seeing
how far our current stack will take us (i.e. will it work on
other embedded devices without any further mods?).

Change-Id: I9a079b7211082e3d3fd418dd5162aaa606a8fc04

Merge "Folks using sample apps get confused with error-prone code in samples. Fixing few issues here: 1. EH should copy the payload and not modify the payload pointer location. 2. EH should check the length of response payload before copying data into it. 3. Fixed getResult. 4. Updated payload to reflect our modified JSON format."

Fixed simpleserver to match correct APIs to set and get Representation from the entity handler. Added code in roomclient to get resource type and interface. Cleared some dead code.

Change-Id: I189ccb86bd38d742866c7201c7b00efac12cc827

Folks using sample apps get confused with error-prone code in samples. Fixing few issues here:
1. EH should copy the payload and not modify the payload pointer location.
2. EH should check the length of response payload before copying data into it.
3. Fixed getResult.
4. Updated payload to reflect our modified JSON format.

Change-Id: I9847742c01c8db203b4fa74b5397136414f86994

Merge "Fix for missing data in the observe notification. JSON response was not built correctly."

Merge "1. Fixed double free in OCDoResource that caused segfault 2. Modified csdk/stack/test/stacktests.cpp because API change caused ocstack unit test build to fail."

Merge "TA1622 - Updated the copyright company from Intel Corporation to Intel Mobile Communications GmbH"

Fix for missing data in the observe notification. JSON response was not built correctly.

Change-Id: Idc7043228119854da497591cc468f95a6af38516

Fix build issue.

Change-Id: I4e587768e921d9654d0473fa702bf35a2685211d

Merge "Exposed APIs for getting resource types and resource interfaces in OCResource header. Updated samples accordingly."

1. Fixed double free in OCDoResource that caused segfault
2. Modified csdk/stack/test/stacktests.cpp because API change caused
ocstack unit test build to fail.

Change-Id: Ie06a5ad603e40ea67fbaadead66b4334d290c8ac

Merge "Makefile again builds OCProperties."

Merge "examples are created in just release/debug folder."

Exposed APIs for getting resource types and resource interfaces in OCResource header. Updated samples accordingly.

Change-Id: Iac6ac0dc064c62c419c1dee186f05d5aeb064623

examples are created in just release/debug folder.

Change-Id: I9da8aab784d98fca7273dc2e9f03ba8dd8bda35c

Merge "Incrementing the size of request/response packets for non-Arduino platforms. Also moved all the configurable items in a ocstackconfig.h include file"

Makefile again builds OCProperties.

AttributeMap -> OCRepresentation

Change-Id: I49effbd0a76845391a26aedcedacf783de8d0aa6

Fix buildng C++ examples. Some examples are removed from the build script until those are fixed.

Change-Id: I706a770e9760250edc092ff2d3c78898f73b444f

Incrementing the size of request/response packets for non-Arduino platforms. Also moved all the configurable items in a ocstackconfig.h include file

Change-Id: I46e4e4cc58d7b48890878faaa52a786cb8a459ab

TA1622 - Updated the copyright company from Intel Corporation to Intel Mobile Communications GmbH

Change-Id: Ia34c1254127b2c78f3a57aa89ae12b5805cb8be0

CPP Default entity handler functionality and misc fixes.

Patch 1:
Fixed simpleclient to work with Observe, along with JSON representation changes.

Patch 2:
Fixed some stuff in simpleclientserver, and other things.

Patch 3:
C++ change to discover multiple resources on a server

Patch 4:
C++ Default handler functionality.
Fixes in C Stack for Default entity hanlder for C++ stack to work.
Few other misc. fixes.

Patch 5: Changed entityHandler name to EntityHandler.

Added Try/Catch around read_json

Fix for simpleclientserver

Change-Id: I3e2006b73de22567ccb7d770d5843e5a5eebce47

Initial Push of Arduino Sample Apps

Right now, tokens are messed up. I cannot get Observe operation to
function properly on Linux or Arduino.

Right now, here is the Arduino server only.

Patch Set 2: Removed "local.properties." This file should NEVER be
committed.

Patch Set 4: Made changes to READMEs as proposed by Bill Dieter.

Patch Set 5-7: Rebases

Path Set 8: Updated stack files to build for Arduino as well.
Arduino compiler is a little bit more per-snickety than the linux compiler.

Change-Id: I21b9396d1d22b9189db65ef31d4528720b01c5b9

GET for CPP/Client

Change-Id: I1a6b9a087bec196387b4a8ab11f153451af54961

Merge "A commit with Confirmable support: Patch set 1 which supports: 1) Sending a delayed response when the request is received as multicast, tested only on linux. 2) Sending confirmable requests. 3) Sending confirmable observer notifications. 4) Sending nonconfirmable observer notification with support of detecting client leaving. 5) Added support for client to send RESET instead of regular GET. 6) Error handling in cases of failed retransmissions. 7) Code reorganizations and refactoring."

A commit with Confirmable support:
Patch set 1 which supports:
1) Sending a delayed response when the request is received as multicast, tested only on linux.
2) Sending confirmable requests.
3) Sending confirmable observer notifications.
4) Sending nonconfirmable observer notification with support of detecting client leaving.
5) Added support for client to send RESET instead of regular GET.
6) Error handling in cases of failed retransmissions.
7) Code reorganizations and refactoring.

Patch set 2:
1) Response to code reviews

Patch set 3:
1) Rebased on master

Patch set 4:
Removed tabs.

Change-Id: I502168bf71c7253f3d4b64b9c2edfe40b1cd8d7a

Merge "CPP Collection"

Merge "Added client/server multiple-resource program; fixed reflection exception namespaces."

Merge "Modified stack unit test makefile (root/csdk/stack/test/makefile) that was broken when root/csdk/makefile mod that created debug and release subdirectories was pushed."

CPP Collection

Patch 1:
Server side CPP collection interfaces implementation.
Includes new class OCRepresentation for collection resources and simple resource.
Includes Sudarshan's changes related to Query Params.

Patch 2:
Client side CPP Put on collection

Patch 3:
Addresses Jesse's comments

Change-Id: Ia1b8fa32a7d80d13519f456aaa6817466ffdfa60

Added client/server multiple-resource program; fixed reflection exception namespaces.

Change-Id: Ibdcf3b722c31f5f5040e8b594431b943274b743d

Merge "Check-in for container tests for C APIs. They were supposed to be in the previous commit, somehow they got missed during rebase."

Check-in for container tests for C APIs. They were supposed to be in the previous commit, somehow they got missed during rebase.

Removed tabs

Change-Id: Id5c6f6d64d42cf3ac25ad866ef926c209f7812ec

Fix build error. Now the entityHandler has its own return type

Change-Id: I1151d88db2c1cb3fada268504948d34723b25e09

Default entity handler outline for collection resources

Added GET for LL and def for default entity handler.

This commit has Sachin's and Chanchala'a changes.
This is GET (LL and B) and PUT (B) working from C apps. New JSON format is
also implemented.

Jesse's review incorporated and tabs removed.

Change-Id: I9d27612d545fdeda1798a5b8c01953a260cfdbef

Merge "integrated coapuriparser fixed the crash occuring when freeing memory in OCUtilities.cpp added a temporary patch in OCUtilities.cpp after discussion with Sudarshan made changes to makefile and OCUtilities.h as per Bill and Joey's comments."

integrated coapuriparser
fixed the crash occuring when freeing memory in OCUtilities.cpp
added a temporary patch in OCUtilities.cpp after discussion with Sudarshan
made changes to makefile and OCUtilities.h as per Bill and Joey's comments.

Change-Id: I53f838bf6cd2e78d5a9c9eadfeb982d29bde54cb

Merge "we don't need linux/if_link.h. Removing this allows building in other platforms"

we don't need linux/if_link.h. Removing this allows building in other platforms

Change-Id: Idef28aeb8f1aa8341439bdac8bfeb17326dd3344

Porting TBStack Functionality to Arduino. The following changes were made
for this:

Added PCF() wrappers around const char* strings to store pre-defined debug
statements in flash, rather than RAM. This only 'affects' Arduino. It has
no affect on linux.

Removed erroneous/un-wanted #include headers from the public TB Stack API

Removed #include ocrandom.h as it was/is unneeded by ISVs.

Updated makefiles to be more friendly between Linux Vs.
Arduino builds (i.e. made both build without a hitch).

Cleaned up warnings that AVR-GCC marked as warnings that regular 'ol GCC
 did not.  Most notably line 328 or 329 "OCStopCoAP()" is being called
with a parameter. OCStopCoAP does not take ANY parameters.

Fixed bug with Arduino devices not retrieving correct IP address structs
from Linux devices (i.e. OCDevAddr needed to be #pragma packed).

Change-Id: Ided8d61ba647889d7a52fa7a4477ca67944b0fa0

Modified stack unit test makefile (root/csdk/stack/test/makefile) that was
broken when root/csdk/makefile mod that created debug and release
subdirectories was pushed.

Note that this unit test MAY be used to verify the build on the build server.

Change-Id: If6075d0d06f233ff4902e480a66dda2cc7248366

Fixing the segfault reported by Doug. It is caused because we are attempting to double free pointer. Carrying this from previous repo.

Change-Id: Iac6ccdbc87a2761103487d0aadf25133be50c9c2

Merge Teamforge ccf-resource repo onto OIC oic-resource repo.

Merge remote-tracking branch 'oic/master' into oic-merge

Conflicts:
csdk/stack/test/linux/Makefile

Merge "* initial creation of API documentation Change-Id: I396fa690ce65a4ff88bcfe8c574759a425ec9a53"

* initial creation of API documentation
Change-Id: I396fa690ce65a4ff88bcfe8c574759a425ec9a53

Updating all of the makefiles to support separate directories for debug and release artifacts.

Change-Id: I61df1142255bc7abb4d0547c5edcc5e4567a9517

Merge "First implementation for Client and Server in same process.  Additionally changed OCResource's IClientWrapper reference to a weak_ptr so that it doesn't cause a deadlock on destruction."

Increasing the size of max server payload(MAX_REQUEST_LENGTH) allowed
Patch 2 : Missed fixing the 'real' culprit.

Change-Id: I4eff9363fa589d3b15b0f9f85c830d59c9f0f979

1. Get/Put callbacks now returning OC_STACK_DELETE_TRAN. This allows us to do memory profiling on stack using sample app. 2. Moved error translation to a common file which can be used by client/server app. 3. some other error cleaning identified in previous reviews

Change-Id: I6b54f127cf3297b9c0b1f6d6f453c2cc68211c01

First implementation for Client and Server in same process.  Additionally changed OCResource's IClientWrapper reference to a weak_ptr
so that it doesn't cause a deadlock on destruction.

SimpleClientServer Example that shows off a client and a server operating on the same process.  The client only does a discovery and a get
Change-Id: I5221a658e58c9c4f7d0c58cbbc657b6880d860bf

1. Modified OCCreateResource to allow NULL to be passed as entity handler parameter.  If NULL
   is passed, the stack's default entity handler is assigned to the resource
2. Modified OCCreateResource so that resource and other dynamic elements it references are
   freed upon creation error.
3. Modified OCStop so that all resources and other dynamically created elements they
   reference are freed.
4. Modified unit test makefile to build unit test with octbstack.a.
5. Moved unit test makefile from root/csdk/stack to root/csdk/stack/test
6. Fixed all broken API calls in root/csdk/stack/test/stacktests.cpp

Merge "Updated functions description to match with params listed in function signature"

Updated functions description to match with params listed in function signature

Patch1:
1. Renamed all container stuff to collection stuff both in C/C++.
2. Simplified function names in C stack.
3. Implemented OCUnbindResource in C stack.
4. Implemented unbindResource(s) in C++ stack.

Patch2: Incorporated Doug and Erich's review comments.

Change-Id: I32d8f7248b65e2fee9a05ca52d22f7d247748984

Makefile to build with c++0x, so we can support compilers old enough to attend 1st grade
Change-Id: Iff0367bf67df7e7fd701e42222fbc068797f3916

US1313, abillity to create an OCResource object without discovering it
Change-Id: Ie283435901a6438c4695496c79cca15e248273a2

Add ObserveAll case in SimpleClient C++
Change-Id: I8769f8bec28ddfa607819926384f2aee9143618f

Added Get and Put for collection. TODO: implementation

bindResource and bindResources implementation

Change-Id: I26747c1360ebc4f385474ee09e403af10b245253

1) Minor change in ocsocket and libcoap to identify if a packet was received on unicast or multicast address/socket
Patch 2 : Fixed identified review comments.
  Modified ocserver to listen on '5683' so that occlient
          can send unicast discovery requests to it.
Patch 3 : 'Really' taking care of tabs this time.

Change-Id: I50c0d47133a469d20ba17e26f2df540e97d099bc

Accidentally pushed and set makefile for TB Stack to build for ARDUINO. I reverted this to LINUX.  To build for ARDUINO issue this in terminal:

 make PLATFORM=arduino

Fix for building OIC code. Makefile was pointing to ccf-utilities which was incorrect. Fix is to point to oic-utilities.

Change-Id: Ic2944a890d9856bb32f940946d6fd5e3c42d3300

Merge "Changing TBStack makefiles to allow for DEBUG & RELEASE modes through & through."

Merge "- US1321 - Now, C++ API users will be able to register multiple types and interfaces   to a resource. The API names may change after review. - Removed unwanted trailing whitespaces at the end of lines. - Updated according to review comments."

Changing TBStack makefiles to allow for DEBUG & RELEASE modes through & through.

Renaming all instances of "Makefile" to "makefile". Justification:
From what I can read "makefile" is ran over "Makefile" when "make" is ran.
Also, more of our makefiles have a lower-case "m" as of now.
Also, this move/rename doesn't affect anyone..I used the git-mv command because I know using "mv" would be detrimental to anyone using Windows.

- US1321 - Now, C++ API users will be able to register multiple types and interfaces
  to a resource. The API names may change after review.
- Removed unwanted trailing whitespaces at the end of lines.
- Updated according to review comments.

Change-Id: I13f7fcd07386d4a13d42f00254f870cfbd8ca4b4

Updated C++ stack makefiles to build in DEBUG or RELEASE modes.