Skip to content

Commit 0497941

Browse files
Chr1sNoCalcProgrammer1
authored andcommitted
Adjusted code to remove warnings in libcmmk
1 parent cacfffd commit 0497941

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

dependencies/libcmmk/include/libcmmk/libcmmk.h

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,9 @@ extern "C" {
3030
#define CMMK_ROWS_MAX 7
3131
#define CMMK_COLS_MAX 22
3232

33-
#define CMMK_KEYLIST_SIZE 256
34-
#define CMMK_BUFFER_SIZE 65
33+
#define CMMK_FW_SIZE 16
34+
#define CMMK_KEYLIST_SIZE 256
35+
#define CMMK_BUFFER_SIZE 65
3536

3637
/*
3738
* If we have C99 support (which we do, because libusb-1.0 requires it...), define some handy

dependencies/libcmmk/src/libcmmk.c

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -236,12 +236,12 @@ int cmmk_find_device(int *product)
236236

237237
static int cmmk_try_determine_layout(struct cmmk *dev, int product)
238238
{
239-
char fw[16];
239+
char fw[CMMK_FW_SIZE];
240240

241-
enum cmmk_layout_type general_layout = CMMK_LAYOUT_TYPE_ANSI;
242-
enum cmmk_product_type device_model;
241+
enum cmmk_layout_type general_layout = CMMK_LAYOUT_TYPE_ANSI;
242+
enum cmmk_product_type device_model = 0;
243243

244-
if (cmmk_get_firmware_version(dev, fw, sizeof(fw)) == 0) {
244+
if (cmmk_get_firmware_version(dev, fw, CMMK_FW_SIZE) == 0) {
245245
if (fw[0] == '1') {
246246
/* ANSI firmware */
247247
general_layout = CMMK_LAYOUT_TYPE_ANSI;
@@ -399,7 +399,7 @@ int cmmk_get_firmware_version(struct cmmk *dev, char *fw, size_t fwsiz)
399399
fwsiz = 60;
400400
}
401401

402-
strncpy(fw, (char *)data + 4, fwsiz);
402+
memcpy(fw, (char *)data + 4, fwsiz);
403403

404404
return CMMK_OK;
405405
}
@@ -890,7 +890,6 @@ int cmmk_get_multilayer_map(struct cmmk *dev, struct cmmk_effect_matrix *effmap)
890890
{
891891
const unsigned char HEADER_SIZE = 9;
892892
unsigned char data_size = CMMK_BUFFER_SIZE - HEADER_SIZE;
893-
unsigned char data_size_in_bytes = data_size / BYTE_SIZE;
894893
int r;
895894

896895
unsigned char data[CMMK_BUFFER_SIZE];
@@ -939,7 +938,6 @@ int cmmk_set_multilayer_map(struct cmmk *dev, struct cmmk_effect_matrix const *e
939938
{
940939
const unsigned char HEADER_SIZE = 9;
941940
unsigned char data_size = CMMK_BUFFER_SIZE - HEADER_SIZE;
942-
unsigned char data_size_in_bytes = data_size / BYTE_SIZE;
943941
int r;
944942

945943
unsigned char data[CMMK_BUFFER_SIZE];
@@ -1035,7 +1033,7 @@ int cmmk_set_all_single(struct cmmk *dev, struct rgb const *col)
10351033
*/
10361034
int cmmk_set_leds(struct cmmk *dev, struct cmmk_color_matrix const *colmap)
10371035
{
1038-
unsigned char data[CMMK_BUFFER_SIZE];
1036+
unsigned char data[CMMK_BUFFER_SIZE];
10391037

10401038
int i;
10411039
int j;

0 commit comments

Comments
 (0)