Skip to content

Commit 939308b

Browse files
author
DenverM80
committed
Visual studio does not consider static const to be Constant; use #define instead
1 parent 7b7291a commit 939308b

File tree

2 files changed

+8
-12
lines changed

2 files changed

+8
-12
lines changed

src/ds3_init_requests.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
#include "ds3_request.h"
2727
#include "ds3_net.h"
2828

29-
static const unsigned int STRING_BUFFER_SIZE = 32;
29+
#define STRING_BUFFER_SIZE 32
3030

3131
static char* _get_ds3_bucket_acl_permission_str(ds3_bucket_acl_permission input) {
3232
if (input == DS3_BUCKET_ACL_PERMISSION_LIST) {

src/ds3_requests.c

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -34,13 +34,9 @@
3434
#endif
3535

3636

37-
//The max size of an uint32_t should be 10 characters + NULL
38-
static const char UNSIGNED_LONG_BASE_10[] = "4294967296";
39-
static const unsigned int UNSIGNED_LONG_BASE_10_STR_LEN = sizeof(UNSIGNED_LONG_BASE_10) + 1;
40-
//The max size of an uint64_t should be 20 characters + NULL
41-
static const char UNSIGNED_LONG_LONG_BASE_10[] = "18446744073709551615";
42-
static const unsigned int UNSIGNED_LONG_LONG_BASE_10_STR_LEN = sizeof(UNSIGNED_LONG_LONG_BASE_10) + 1;
43-
37+
//The max size of an uint32_t is 10 characters + NULL
38+
//The max size of an uint64_t is 20 characters + NULL
39+
#define STRING_BUFFER_SIZE 32
4440

4541

4642
struct _ds3_metadata {
@@ -408,7 +404,7 @@ static ds3_error* _get_request_xml_nodes(
408404
}
409405

410406
static xmlDocPtr _generate_xml_bulk_objects_list(const ds3_bulk_object_list_response* obj_list, object_list_type list_type, ds3_job_chunk_client_processing_order_guarantee order) {
411-
char size_buff[UNSIGNED_LONG_LONG_BASE_10_STR_LEN];
407+
char size_buff[STRING_BUFFER_SIZE];
412408
xmlDocPtr doc;
413409
ds3_bulk_object_response* obj;
414410
xmlNodePtr objects_node, object_node;
@@ -431,7 +427,7 @@ static xmlDocPtr _generate_xml_bulk_objects_list(const ds3_bulk_object_list_resp
431427

432428
for (obj_index = 0; obj_index < obj_list->num_objects; obj_index++) {
433429
obj = obj_list->objects[obj_index];
434-
g_snprintf(size_buff, sizeof(char) * UNSIGNED_LONG_LONG_BASE_10_STR_LEN, "%" PRIu64, obj->length);
430+
g_snprintf(size_buff, STRING_BUFFER_SIZE, "%" PRIu64, obj->length);
435431

436432
object_node = xmlNewNode(NULL, (xmlChar*) "Object");
437433
xmlAddChild(objects_node, object_node);
@@ -448,7 +444,7 @@ static xmlDocPtr _generate_xml_bulk_objects_list(const ds3_bulk_object_list_resp
448444
}
449445

450446
static xmlDocPtr _generate_xml_complete_mpu(const ds3_complete_multipart_upload_response* mpu_list) {
451-
char size_buff[UNSIGNED_LONG_LONG_BASE_10_STR_LEN];
447+
char size_buff[STRING_BUFFER_SIZE];
452448
xmlDocPtr doc;
453449
ds3_multipart_upload_part_response* part;
454450
xmlNodePtr parts_node, part_node;
@@ -464,7 +460,7 @@ static xmlDocPtr _generate_xml_complete_mpu(const ds3_complete_multipart_upload_
464460
part_node = xmlNewNode(NULL, (xmlChar*) "Part");
465461
xmlAddChild(parts_node, part_node);
466462

467-
g_snprintf(size_buff, sizeof(char) * UNSIGNED_LONG_LONG_BASE_10_STR_LEN, "%d", part->part_number);
463+
g_snprintf(size_buff, STRING_BUFFER_SIZE, "%d", part->part_number);
468464
xmlNewTextChild(part_node, NULL, (xmlChar*) "PartNumber", (xmlChar*) size_buff);
469465

470466
xmlNewTextChild(part_node, NULL, (xmlChar*) "ETag", (xmlChar*) part->etag->value);

0 commit comments

Comments
 (0)