Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

can't build with -DREVERSED #27

Closed
jpwsutton opened this issue Feb 4, 2016 · 0 comments
Closed

can't build with -DREVERSED #27

jpwsutton opened this issue Feb 4, 2016 · 0 comments

Comments

@jpwsutton
Copy link
Member

migrated from Bugzilla #445038
status RESOLVED severity blocker in component MQTT-Embedded-C for 1.1
Reported in version unspecified on platform Other
Assigned to: Al Stockdill-Mander

On 2014-09-25 02:57:47 -0400, Akihiro Yamazaki wrote:

How to repeat

$ cd paho/MQTTPacket/samples

$ git remote -v
origin http://git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.embedded-c.git (fetch)
origin http://git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.embedded-c.git (push)

$ git log -1 --pretty=short
commit SHA: 8e6d4a4
Author: Al S-M asm@uk.ibm.com

c client layer and cc3200 implementation

$ cc qos0pub.c -DREVERSED -I ../src ../src/MQTTConnectClient.c
../src/MQTTSerializePublish.c ../src/MQTTPacket.c -o qos0pub -Os -s
In file included from ../src/MQTTPacket.h:91:0,
from qos0pub.c:17:
../src/MQTTConnect.h:113:18: error: �y� undeclared here (not in a function)
unsigned int : y; /**< unused */
^
(snip)

Patch

diff --git a/MQTTPacket/src/MQTTConnect.h b/MQTTPacket/src/MQTTConnect.h
index 8577289..db90251 100644
--- a/MQTTPacket/src/MQTTConnect.h
+++ b/MQTTPacket/src/MQTTConnect.h
@@ -110,7 +110,7 @@ typedef union
struct
{
unsigned int sessionpresent : 1; /**< session present flag */

  •           unsigned int : y;                         /**< unused */
    
  •           unsigned int : 7;                         /**< unused */
    } bits;
    
    #else
    struct

On 2014-09-25 06:00:34 -0400, Ian Craggs wrote:

Assigning to Al.

On 2014-09-25 06:55:10 -0400, Al Stockdill-Mander wrote:

I have pushed a fix for this to Gerrit

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

No branches or pull requests

1 participant