[jira] [Updated] (PROTON-98) Porting Issue -- Visual Studio compiler requires explicit casts

2013-02-11 Thread Rob Godfrey (JIRA)

 [ 
https://issues.apache.org/jira/browse/PROTON-98?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rob Godfrey updated PROTON-98:
--

Fix Version/s: 0.4

 Porting Issue -- Visual Studio compiler requires explicit casts
 ---

 Key: PROTON-98
 URL: https://issues.apache.org/jira/browse/PROTON-98
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-c
 Environment: Windows using Visual Studio 2010
Reporter: Mary hinton
  Labels: build
 Fix For: 0.4

 Attachments: explicitCasts2.patch, ExplicitCasts.patch


 I would like to get the code changed to add explicit casts where the Visual 
 Studio compiler requires it. 
 The GNU compiler isn’t so picky, but we need the explicit casts for Visual 
 Studio tools.
 The problem is in many of the files.
   C:\qpid\qpid\proton\proton-c\src\proton.c(278):  struct client_context *ctx 
 = (client_context *) pn_connector_context(ctor);// explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\buffer.c(41):  pn_buffer_t *buf = 
 (pn_buffer_t *) malloc(sizeof(pn_buffer_t)); // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\buffer.c(45):  buf-bytes = capacity ? 
 (char *) malloc(capacity) : NULL;   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(706):return (pn_type_t) 
 PN_ARG_ERR;  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(758):return (pn_type_t) 
 PN_ARG_ERR;  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1210):atom-u.type = 
 (pn_type_t) va_arg(*ap, int);   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1311):  char **sptr 
 = (char **) ptr; // explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1875):  pn_data_t *data = 
 (pn_data_t *) malloc(sizeof(pn_data_t));   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1878):  data-nodes = 
 capacity ? (pn_node_t *) malloc(capacity * sizeof(pn_node_t)) : NULL;   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1931):  data-nodes = 
 (pn_node_t *) realloc(data-nodes, data-capacity * sizeof(pn_node_t));   
  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(2151):char 
 **sptr = (char**) ptr;
 // explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(3114):return (pn_type_t) 
 -1; // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(3764):return (pn_type_t) 
 -1; // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(32):  
 pn_dispatcher_t *disp = (pn_dispatcher_t *) calloc(sizeof(pn_dispatcher_t), 
 1);  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(47):  disp-output 
 = (char *) malloc(disp-capacity);  // explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(248):
 disp-output = (char *) realloc(disp-output, disp-capacity);// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(193):  pn_listener_t *l = 
 (pn_listener_t *) malloc(sizeof(pn_listener_t));// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(410):  pn_connector_t *c = 
 (pn_connector_t *) malloc(sizeof(pn_connector_t)); // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(656):  pn_driver_t *d = 
 (pn_driver_t *) malloc(sizeof(pn_driver_t));  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(748):d-fds = (pollfd *) 
 realloc(d-fds, d-capacity*sizeof(struct pollfd));  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(47):  db-deliveries = 
 (pn_delivery_state_t *) malloc(sizeof(pn_delivery_state_t) * capacity);// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(438):  pn_connection_t 
 *conn = (pn_connection_t *)malloc(sizeof(pn_connection_t)); 
 // explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(730):  pn_session_t *ssn = 
 (pn_session_t *) malloc(sizeof(pn_session_t));  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(866):  pn_transport_t 
 *transport = (pn_transport_t *) malloc(sizeof(pn_transport_t));   
// 

[jira] [Updated] (PROTON-98) Porting Issue -- Visual Studio compiler requires explicit casts

2013-02-11 Thread Rafael H. Schloming (JIRA)

 [ 
https://issues.apache.org/jira/browse/PROTON-98?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Rafael H. Schloming updated PROTON-98:
--

Assignee: Cliff Jansen

 Porting Issue -- Visual Studio compiler requires explicit casts
 ---

 Key: PROTON-98
 URL: https://issues.apache.org/jira/browse/PROTON-98
 Project: Qpid Proton
  Issue Type: Improvement
  Components: proton-c
 Environment: Windows using Visual Studio 2010
Reporter: Mary hinton
Assignee: Cliff Jansen
  Labels: build
 Fix For: 0.4

 Attachments: explicitCasts2.patch, ExplicitCasts.patch


 I would like to get the code changed to add explicit casts where the Visual 
 Studio compiler requires it. 
 The GNU compiler isn’t so picky, but we need the explicit casts for Visual 
 Studio tools.
 The problem is in many of the files.
   C:\qpid\qpid\proton\proton-c\src\proton.c(278):  struct client_context *ctx 
 = (client_context *) pn_connector_context(ctor);// explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\buffer.c(41):  pn_buffer_t *buf = 
 (pn_buffer_t *) malloc(sizeof(pn_buffer_t)); // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\buffer.c(45):  buf-bytes = capacity ? 
 (char *) malloc(capacity) : NULL;   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(706):return (pn_type_t) 
 PN_ARG_ERR;  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(758):return (pn_type_t) 
 PN_ARG_ERR;  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1210):atom-u.type = 
 (pn_type_t) va_arg(*ap, int);   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1311):  char **sptr 
 = (char **) ptr; // explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1875):  pn_data_t *data = 
 (pn_data_t *) malloc(sizeof(pn_data_t));   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1878):  data-nodes = 
 capacity ? (pn_node_t *) malloc(capacity * sizeof(pn_node_t)) : NULL;   // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(1931):  data-nodes = 
 (pn_node_t *) realloc(data-nodes, data-capacity * sizeof(pn_node_t));   
  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(2151):char 
 **sptr = (char**) ptr;
 // explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(3114):return (pn_type_t) 
 -1; // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\codec\codec.c(3764):return (pn_type_t) 
 -1; // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(32):  
 pn_dispatcher_t *disp = (pn_dispatcher_t *) calloc(sizeof(pn_dispatcher_t), 
 1);  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(47):  disp-output 
 = (char *) malloc(disp-capacity);  // explicit 
 cast
   C:\qpid\qpid\proton\proton-c\src\dispatcher\dispatcher.c(248):
 disp-output = (char *) realloc(disp-output, disp-capacity);// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(193):  pn_listener_t *l = 
 (pn_listener_t *) malloc(sizeof(pn_listener_t));// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(410):  pn_connector_t *c = 
 (pn_connector_t *) malloc(sizeof(pn_connector_t)); // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(656):  pn_driver_t *d = 
 (pn_driver_t *) malloc(sizeof(pn_driver_t));  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\driver.c(748):d-fds = (pollfd *) 
 realloc(d-fds, d-capacity*sizeof(struct pollfd));  // explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(47):  db-deliveries = 
 (pn_delivery_state_t *) malloc(sizeof(pn_delivery_state_t) * capacity);// 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(438):  pn_connection_t 
 *conn = (pn_connection_t *)malloc(sizeof(pn_connection_t)); 
 // explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(730):  pn_session_t *ssn = 
 (pn_session_t *) malloc(sizeof(pn_session_t));  // 
 explicit cast
   C:\qpid\qpid\proton\proton-c\src\engine\engine.c(866):  pn_transport_t 
 *transport = (pn_transport_t *)