Thiago Macieira [Fri, 5 Sep 2014 01:20:13 +0000 (18:20 -0700)]
Fix mistaken use of && where & was intended
Found by Clang:
occoaphelper.c:93:59: warning: use of logical '&&' with constant operand [-Wconstant-logical-operand]
It's clearly meant to be a &, as seen by the right shift by 5 and the
constant of 31 (5 bits)
Change-Id: Ie35ad6c313ad53d766fc6fe88ffd03b462d49519
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Erich Keane [Thu, 11 Sep 2014 17:57:44 +0000 (10:57 -0700)]
Fix use of uninitialized variable
If ctx == NULL, then VERIFY_NON_NULL would goto exit, which would then
OCFree the uninitialized variables.
Found by Clang:
occoap.c:274:5: warning: variable 'rcvObserveOption' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
occoap.c:44:36: note: expanded from macro 'VERIFY_NON_NULL'
#define VERIFY_NON_NULL(arg) { if (!arg) {OC_LOG_V(FATAL, TAG, "%s is NULL", #arg); goto exit;} }
^~~~
occoap.c:459:16: note: uninitialized use occurs here
--Erich: rebase on master
Change-Id: Ie829c6ac284dfd988eeeffad446d9872424e604a
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Fri, 5 Sep 2014 01:00:53 +0000 (18:00 -0700)]
Fix warnings about diff and base possibly not initialized
This is a bogus warning by GCC, but it's easy to fix by having an "else"
case with no extra conditions.
Change-Id: I0f99527065138ff96c0613a57b0462613139900c
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Fri, 5 Sep 2014 00:52:18 +0000 (17:52 -0700)]
Rewrite examples/ocicuc/Makefile
This Makefile was almost completely broken, as the comment at the top
indicated. This rewrite makes it work and also respect the top-level set
CXX variable.
Change-Id: I462b5caedfc090e6ebff72c869e82e3d57dc16c2
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Fri, 5 Sep 2014 00:27:00 +0000 (17:27 -0700)]
Fix parallel build
All the .o files must be compiled before we can create the .a library
from them. When compiling with a -j option (as low as -j4), often one or
two .o files would be missing from the resulting .a because they hadn't
finished compiling yet.
Change-Id: Ie7bc0ad489b8b0afecab3ec318f2784ee706ae12
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Fri, 5 Sep 2014 00:09:53 +0000 (17:09 -0700)]
Use CXX for the variable containing the name of the C++ compiler
CXX is standard, CCPLUS is not used anywhere else.
Change-Id: I2b98d609ade694306dd20062b8149a242ff7e91b
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Fri, 5 Sep 2014 00:05:52 +0000 (17:05 -0700)]
Building examples requires OCLib.a to be built first
Change-Id: I2864d8c2adc7373f6a7c6ed5619f671af26c5727
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Thu, 4 Sep 2014 23:58:47 +0000 (16:58 -0700)]
Update the #pragma GCC diagnostic usage
Don't force pedantic back on. Instead, push and pop the current state.
Don't pass these options to Clang and ICC, since they don't have the
-pedantic option anyway.
Change-Id: I302f7fcfe974f23423ca2ac7711a7c4a8ac39d88
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Thu, 4 Sep 2014 23:57:56 +0000 (16:57 -0700)]
Don't pass linker flags to compilation stages
Change-Id: Ie6d45dee1dd1919fb32a05c30143c4ba6ad085d8
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Thiago Macieira [Thu, 4 Sep 2014 23:40:39 +0000 (16:40 -0700)]
Add -p option to mkdir commands in the Makefiles
This makes mkdir not complain if it cannot create a directory that
already exists. The result of the command is ignored, but we don't need
the diagnostic either.
Change-Id: I25b9da5246e1469bccd5827886eecadd62ad319f
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Erich Keane [Thu, 11 Sep 2014 17:45:09 +0000 (10:45 -0700)]
Merge "Doxygen run for *.h files"
Erich Keane [Thu, 11 Sep 2014 17:44:23 +0000 (10:44 -0700)]
Merge "Add .gitignore file"
Erich Keane [Thu, 11 Sep 2014 16:50:10 +0000 (09:50 -0700)]
Merge "Added check for Time libraries in expected location for the Arduino Time Libraries from the TBStack makefile. If they are not there, the user will be told to put them where we expect them to be now."
Thiago Macieira [Thu, 4 Sep 2014 23:45:22 +0000 (16:45 -0700)]
Add .gitignore file
Change-Id: Ibb7a2b474bc4eb8a46654e2925fd73fddac0545a
Signed-off-by: Thiago Macieira <thiago.macieira@intel.com>
Joseph Morrow [Wed, 10 Sep 2014 21:36:18 +0000 (17:36 -0400)]
Added check for Time libraries in expected location for the Arduino
Time Libraries from the TBStack makefile. If they are not there,
the user will be told to put them where we expect them to be now.
Change-Id: I93693a4623d038e152559ff9d8260bdb91c9000f
Erich Keane [Wed, 10 Sep 2014 18:08:09 +0000 (11:08 -0700)]
Provided a mechanism for deleting the context pointer when the C Stack deletes callback. Otherwise, the callback context will be leaked.
Fixed some line endings.
Fixed some line lengths.
Fixed a series of memory leaks in C++ callbacks which assumed the C stack owned/deleted OCCallbackData
Patch 2: Removed deletion of function pointers becaues GCC says this violates the C spec.
Patch 3: Comments from Joey regarding the comment on the OCClientContextDeleter typedef
Patch 4: changed deleter to deleteCallback per sashi's suggestion
Change-Id: I0fb71896afac93356c1bd2f4916eb9f3eaa0ad13
Erich Keane [Wed, 10 Sep 2014 19:36:26 +0000 (12:36 -0700)]
Merge "IOT-31: Add copyright info to all makefiles"
Erich Keane [Wed, 10 Sep 2014 19:34:10 +0000 (12:34 -0700)]
Merge "A fix for building the Arduino TBStack without wifi libraries. This bug was apparent even when building for only ethernet, the makefile expected the wifi libraries to exist. Now the makefile only expects the wifi libraries to exist if you're building for wifi shield."
Charunethran PG [Wed, 10 Sep 2014 17:39:16 +0000 (19:39 +0200)]
Doxygen run for *.h files
Change-Id: Ia26ec0edc0f817ac872b261e9490c2319db466b4
Signed-off-by: Charunethran PG <charunethran.panchalam.govindarajan@intel.com>
Erich Keane [Wed, 10 Sep 2014 16:52:14 +0000 (09:52 -0700)]
IOT-31: Add copyright info to all makefiles
Change-Id: Idcdbf7e9a81b5f7f28e953cf31083d083c917aea
Erich Keane [Wed, 10 Sep 2014 16:35:04 +0000 (09:35 -0700)]
Merge "Just realized that these should NOT be PCF'd. If a server or client were using these as defined to be in program memory, the stack would have been storing the location the strings were stored in memory instead of their actual values when running on Arduino. PCF has no effect on Linux."
Joseph Morrow [Wed, 10 Sep 2014 16:20:47 +0000 (12:20 -0400)]
A fix for building the Arduino TBStack without wifi libraries. This bug
was apparent even when building for only ethernet, the makefile
expected the wifi libraries to exist. Now the makefile only expects
the wifi libraries to exist if you're building for wifi shield.
Patch 2: Rebase
Patch 3: Moved implementation in to just arduinomega.properties
as per Sachin's suggestion.
Change-Id: Id06387e0d10244d71ae83a00068a0ef2f615ee84
Erich Keane [Tue, 9 Sep 2014 22:36:33 +0000 (15:36 -0700)]
Cleaned up a number of line lengths. Cleaned up iterator-for-loops vs for:each syntax for clarity where easily possible. Switched to by ref where we were previously doing unnecessary pointer parameters
Went entirely thorugh InProcClientWrapper.cpp and fixed line lengths.
Change-Id: I69f706e64f67d9a822639f239575eb316675c1d7
Joseph Morrow [Tue, 9 Sep 2014 21:26:41 +0000 (17:26 -0400)]
Just realized that these should NOT be PCF'd. If a server or client
were using these as defined to be in program memory, the stack would
have been storing the location the strings were stored in memory
instead of their actual values when running on Arduino. PCF has no
effect on Linux.
Change-Id: Iada6282befa17878f87045141580c7e330c732c1
Sashi Penta [Tue, 9 Sep 2014 20:09:51 +0000 (13:09 -0700)]
Removing unwanted .h and .cpp files from C++ stack.
Change-Id: Ib130b2c6849b4b13683229fcf0811ad246dccd9a
Erich Keane [Tue, 9 Sep 2014 18:24:46 +0000 (11:24 -0700)]
Merge "With respect to Rally User Story: US1556"
Erich Keane [Tue, 9 Sep 2014 18:20:43 +0000 (11:20 -0700)]
Missed removing OCProperties.cpp earlier
Change-Id: I52eb9255cf7a75b312a8d556f2e53bcbe81e9512
Joseph Morrow [Tue, 9 Sep 2014 18:18:50 +0000 (14:18 -0400)]
With respect to Rally User Story: US1556
and Rally Task: US1596
Updated ocstack.c to notify observers of this 'stack's presence
whenever any modification or deletion to ANY of this stack's
resource(s) occurs.
Change-Id: I0bdb72f56fa36a94e3d6d6b548f87a6a75a362ca
Erich Keane [Tue, 9 Sep 2014 17:06:37 +0000 (10:06 -0700)]
Merge "Fixing a logging bug introduced in earlier WiFi related check-in. Patch 2 : Fixing the issue caught by Joey."
Erich Keane [Tue, 9 Sep 2014 16:54:53 +0000 (09:54 -0700)]
Merge "Deleting OCReflect and OCServer, which are boht dead code. Had to remove references from a few files and fix some other includes Fixed OCResource's line lengths Change-Id: Ica6e66a1ab3ecffd6c93d8f4787a3be8a9ed1128"
Erich Keane [Mon, 8 Sep 2014 22:37:28 +0000 (15:37 -0700)]
Deleting OCReflect and OCServer, which are boht dead code. Had to remove references from a few files and fix some other includes
Fixed OCResource's line lengths
Change-Id: Ica6e66a1ab3ecffd6c93d8f4787a3be8a9ed1128
Sachin Agrawal [Tue, 9 Sep 2014 16:21:42 +0000 (09:21 -0700)]
Fixing a logging bug introduced in earlier WiFi related check-in.
Patch 2 : Fixing the issue caught by Joey.
Change-Id: Ib254202e6d87fed70de2bf8d1db639b475d377f6
Sashi Penta [Mon, 8 Sep 2014 22:14:15 +0000 (15:14 -0700)]
OCStartPresence comments in header.
Every OCStartPresence sends a random nonce.
C++ Stack cleanup to use unsigned integer for nonce.
Change-Id: If83e0868bbc05fa641cd3a0758e250b59da6393c
Sashi Penta [Mon, 8 Sep 2014 22:27:21 +0000 (15:27 -0700)]
Sending NONCE and TTL in the payload for active presence:
Patch set #1:
1) The payload is formatted as NONCE#:TTL#
2) NONCE is a 32bit unisgned random int.
3) The client app is not called when there is not change in NONCE, OC_STACK_PRESENCE_NO_UPDATE was deleted
4) Code organization in occoap.c
5) Rebased on master
Change-Id: I2c4f8c4b30c3d692a38b3286001d55f3387f5a73
Erich Keane [Mon, 8 Sep 2014 21:37:35 +0000 (14:37 -0700)]
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."
Sashi Penta [Fri, 5 Sep 2014 04:36:27 +0000 (21:36 -0700)]
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
Erich Keane [Fri, 5 Sep 2014 23:43:55 +0000 (16:43 -0700)]
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
Erich Keane [Mon, 8 Sep 2014 19:50:07 +0000 (12:50 -0700)]
Merge "Get/Put overload implementation in OCResource. Allows for specifiying interface and type."
Erich Keane [Mon, 8 Sep 2014 18:28:30 +0000 (11:28 -0700)]
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."
Erich Keane [Mon, 8 Sep 2014 18:03:27 +0000 (11:03 -0700)]
Merge "Fix for arduinomega.properties. It was pointing at an invalid location for avrdude with arduino 1.0.5."
Erich Keane [Mon, 8 Sep 2014 17:57:13 +0000 (10:57 -0700)]
Merge "Stack/Observer: Free OCEntityHandlerRequest object after sent notification"
Erich Keane [Thu, 4 Sep 2014 18:23:16 +0000 (11:23 -0700)]
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
Sachin Agrawal [Mon, 8 Sep 2014 16:17:28 +0000 (09:17 -0700)]
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
Joseph Morrow [Fri, 5 Sep 2014 16:39:15 +0000 (12:39 -0400)]
Fix for arduinomega.properties. It was pointing at an invalid
location for avrdude with arduino 1.0.5.
Change-Id: I15878e29a2daa341886d0b8e0fa2010f21771f32
Erich Keane [Fri, 5 Sep 2014 17:35:33 +0000 (10:35 -0700)]
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
Erich Keane [Fri, 5 Sep 2014 16:18:02 +0000 (09:18 -0700)]
Merge "OCLib is now called src. Now liboc.a is generated instead OCLib.a ."
Joseph Morrow [Fri, 5 Sep 2014 15:17:52 +0000 (11:17 -0400)]
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
Sashi Penta [Fri, 5 Sep 2014 04:10:47 +0000 (21:10 -0700)]
OCLib is now called src. Now liboc.a is generated instead OCLib.a .
Change-Id: Id9aa7652987a0915842fa6441efa118809a0b01b
Sachin Agrawal [Fri, 5 Sep 2014 01:27:02 +0000 (18:27 -0700)]
Fixing broken TB stack due to compilation errors for Arduino platforms.
Change-Id: Id211b8f03ddb07dd8c50148ece513bd2548b447f
Wu Jiangbo [Wed, 3 Sep 2014 11:08:23 +0000 (19:08 +0800)]
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
Sashi Penta [Wed, 3 Sep 2014 21:01:11 +0000 (14:01 -0700)]
Minor formatting fix for the release notes.
Change-Id: Ieee8839edbaef07ae5ab18efb81162c99d70d440
Wu Jiangbo [Wed, 3 Sep 2014 07:09:02 +0000 (15:09 +0800)]
Stack/Observer: Free OCEntityHandlerRequest object after sent notification
Change-Id: I539778ed8ad65c45635ff64f0b71a5ade1743894
Sashi Penta [Wed, 3 Sep 2014 04:10:52 +0000 (21:10 -0700)]
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
Sashi Penta [Wed, 3 Sep 2014 03:17:01 +0000 (20:17 -0700)]
Includes C++ Presence Notification changes from Erich, Jesse, Sudarshan and Sashi.
Change-Id: I113cd2dc3051bace0d0111289184299e5fd21f87
Erich Keane [Wed, 3 Sep 2014 02:50:26 +0000 (19:50 -0700)]
Merge "Patch 1 (Yamin): This is where all work is merged from Sashi, Joey and myself"
Yamin S Al-mousa [Wed, 3 Sep 2014 02:16:08 +0000 (19:16 -0700)]
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
Jesse Williamson [Fri, 29 Aug 2014 23:53:30 +0000 (16:53 -0700)]
Checkpoint: monoprocess bug fixed-- need to improve CLI handling.
Change-Id: I73ad330df3716cda2f5907a62dc09478bc03fe5e
Erich Keane [Fri, 29 Aug 2014 16:27:09 +0000 (09:27 -0700)]
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."
Jesse Williamson [Tue, 26 Aug 2014 19:49:54 +0000 (12:49 -0700)]
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
Yamin S Al-mousa [Thu, 28 Aug 2014 01:02:52 +0000 (18:02 -0700)]
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
Patrick Lankswert [Thu, 28 Aug 2014 00:25:36 +0000 (17:25 -0700)]
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 "
Joseph Morrow [Wed, 27 Aug 2014 15:20:57 +0000 (11:20 -0400)]
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
clairiky [Tue, 26 Aug 2014 21:26:12 +0000 (14:26 -0700)]
Finished doing TODOs in ValidateQuery for container resources with default entity handler.
Updated Sashi's review.
Change-Id: I732bc7bf63acd7522051ce39a9f80c6a41df9b52
Joseph Morrow [Thu, 21 Aug 2014 21:04:55 +0000 (17:04 -0400)]
Adding API changes needed for Active Discovery in the TB Stack layer.
Change-Id: I8c457cccb2bbbfefedc6aa27a89bdc47ad7aa061
Joseph Morrow [Mon, 25 Aug 2014 19:23:30 +0000 (15:23 -0400)]
Fix for TB Unit Tests. This should make Jenkins build again.
Change-Id: I1dc0e13d9e2a56f5799f352d66b0916df869c738
William Dieter [Mon, 25 Aug 2014 17:12:01 +0000 (10:12 -0700)]
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.)."
Sachin Agrawal [Mon, 25 Aug 2014 07:43:06 +0000 (00:43 -0700)]
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
Sakthivel Samidurai [Mon, 25 Aug 2014 05:15:39 +0000 (22:15 -0700)]
Added helper function to escape a special character in a string.
Change-Id: Icdca0fe4fb1c959b374cd6afea57272576e86061
Joseph Morrow [Fri, 22 Aug 2014 21:43:36 +0000 (17:43 -0400)]
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
William Dieter [Fri, 22 Aug 2014 18:37:26 +0000 (11:37 -0700)]
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."
Sudarshan Prasad [Fri, 22 Aug 2014 09:49:18 +0000 (02:49 -0700)]
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
Sachin Agrawal [Fri, 22 Aug 2014 09:32:59 +0000 (02:32 -0700)]
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
Patrick Lankswert [Fri, 22 Aug 2014 05:40:09 +0000 (22:40 -0700)]
Merge "Fix for missing data in the observe notification. JSON response was not built correctly."
Patrick Lankswert [Fri, 22 Aug 2014 05:33:25 +0000 (22:33 -0700)]
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."
George Bowden [Fri, 22 Aug 2014 05:29:44 +0000 (22:29 -0700)]
Merge "TA1622 - Updated the copyright company from Intel Corporation to Intel Mobile Communications GmbH"
Sudarshan Prasad [Fri, 22 Aug 2014 05:13:26 +0000 (22:13 -0700)]
Fix for missing data in the observe notification. JSON response was not built correctly.
Change-Id: Idc7043228119854da497591cc468f95a6af38516
Sashi Penta [Fri, 22 Aug 2014 03:08:17 +0000 (20:08 -0700)]
Fix build issue.
Change-Id: I4e587768e921d9654d0473fa702bf35a2685211d
William Dieter [Fri, 22 Aug 2014 00:45:37 +0000 (17:45 -0700)]
Merge "Exposed APIs for getting resource types and resource interfaces in OCResource header. Updated samples accordingly."
Doug Hudson [Fri, 22 Aug 2014 00:44:13 +0000 (20:44 -0400)]
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
William Dieter [Fri, 22 Aug 2014 00:42:41 +0000 (17:42 -0700)]
Merge "Makefile again builds OCProperties."
William Dieter [Fri, 22 Aug 2014 00:37:20 +0000 (17:37 -0700)]
Merge "examples are created in just release/debug folder."
Sudarshan Prasad [Fri, 22 Aug 2014 00:14:01 +0000 (17:14 -0700)]
Exposed APIs for getting resource types and resource interfaces in OCResource header. Updated samples accordingly.
Change-Id: Iac6ac0dc064c62c419c1dee186f05d5aeb064623
Sashi Penta [Thu, 21 Aug 2014 22:58:17 +0000 (15:58 -0700)]
examples are created in just release/debug folder.
Change-Id: I9da8aab784d98fca7273dc2e9f03ba8dd8bda35c
William Dieter [Thu, 21 Aug 2014 21:28:19 +0000 (14:28 -0700)]
Merge "Incrementing the size of request/response packets for non-Arduino platforms. Also moved all the configurable items in a ocstackconfig.h include file"
Jesse Williamson [Thu, 21 Aug 2014 17:53:25 +0000 (10:53 -0700)]
Makefile again builds OCProperties.
AttributeMap -> OCRepresentation
Change-Id: I49effbd0a76845391a26aedcedacf783de8d0aa6
Sashi Penta [Thu, 21 Aug 2014 18:25:24 +0000 (11:25 -0700)]
Fix buildng C++ examples. Some examples are removed from the build script until those are fixed.
Change-Id: I706a770e9760250edc092ff2d3c78898f73b444f
Joseph Morrow [Thu, 21 Aug 2014 15:54:42 +0000 (11:54 -0400)]
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
Gene Chang [Thu, 21 Aug 2014 13:50:03 +0000 (09:50 -0400)]
TA1622 - Updated the copyright company from Intel Corporation to Intel Mobile Communications GmbH
Change-Id: Ia34c1254127b2c78f3a57aa89ae12b5805cb8be0
Sashi Penta [Thu, 21 Aug 2014 00:01:05 +0000 (17:01 -0700)]
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
Joseph Morrow [Mon, 18 Aug 2014 21:04:21 +0000 (17:04 -0400)]
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
Sashi Penta [Wed, 20 Aug 2014 01:49:02 +0000 (18:49 -0700)]
GET for CPP/Client
Change-Id: I1a6b9a087bec196387b4a8ab11f153451af54961
William Dieter [Wed, 20 Aug 2014 01:44:23 +0000 (18:44 -0700)]
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."
clairiky [Wed, 20 Aug 2014 01:39:46 +0000 (18:39 -0700)]
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
William Dieter [Wed, 20 Aug 2014 01:24:56 +0000 (18:24 -0700)]
Merge "CPP Collection"
William Dieter [Wed, 20 Aug 2014 01:23:13 +0000 (18:23 -0700)]
Merge "Added client/server multiple-resource program; fixed reflection exception namespaces."
William Dieter [Wed, 20 Aug 2014 01:11:10 +0000 (18:11 -0700)]
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."
Sashi Penta [Tue, 19 Aug 2014 06:02:54 +0000 (23:02 -0700)]
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
Jesse Williamson [Mon, 18 Aug 2014 17:29:23 +0000 (10:29 -0700)]
Added client/server multiple-resource program; fixed reflection exception namespaces.
Change-Id: Ibdcf3b722c31f5f5040e8b594431b943274b743d
William Dieter [Tue, 19 Aug 2014 16:28:48 +0000 (09:28 -0700)]
Merge "Check-in for container tests for C APIs. They were supposed to be in the previous commit, somehow they got missed during rebase."
clairiky [Tue, 19 Aug 2014 16:07:30 +0000 (09:07 -0700)]
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