Skip to content

Commit 6ab2b99

Browse files
MikhailNatalenkoMikhailNatalenko
committed
Refactor compile warnings. Encoding fix
1 parent 6723b1d commit 6ab2b99

File tree

8 files changed

+68
-66
lines changed

8 files changed

+68
-66
lines changed

‎Widgets/Button.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
#include "Widgets/Button.h"
2424
#include <malloc.h>
2525

26-
bool static vButtonDraw(xButton *pxW) {
26+
static bool vButtonDraw(xButton *pxW) {
2727
xButtonProps *xP;
2828

2929
if (!pxW)
@@ -57,7 +57,7 @@ bool static vButtonDraw(xButton *pxW) {
5757
return true;
5858
}
5959

60-
bool static bButtonCheckTSRoutine(xButton *pxW, xTouchEvent *pxTouchScreenEv) {
60+
static bool bButtonCheckTSRoutine(xButton *pxW, xTouchEvent *pxTouchScreenEv) {
6161
xButtonProps *xP;
6262

6363
if (!pxW)
@@ -106,6 +106,7 @@ xButton * pxButtonCreate(uint16_t usX, uint16_t usY, xPicture pusPic, xWidget *p
106106
}
107107
}
108108

109+
//TODO: Test this code (should work)
109110
bool bButtonSetPushPic(xButton *pxW, xPicture pusPic) {
110111
//xButtonProps *xP;
111112

‎Widgets/Button.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ extern "C" {
4040

4141
xButton * pxButtonCreate(uint16_t usX, uint16_t usY, xPicture pusPic, xWidget *pxWidParent);
4242
bool bButtonSetPushPic(xButton *pxW, xPicture pusPic);
43-
void inline bButtonSetOnClickHandler(xWidget *pxW, bool(*pxCallback)(xWidget *)) {
43+
inline void bButtonSetOnClickHandler(xWidget *pxW, bool(*pxCallback)(xWidget *)) {
4444
vWidgetSetOnClickHandler(pxW, pxCallback);
4545
}
4646

‎Widgets/Interface.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ xWindow * pxInterfaceIsWindowActive(eWindow eWnd) {
9595
return NULL;
9696
}
9797

98-
void inline prvDelWndFromStack(xWindow *pxN, xWindow *pxNext) {
98+
inline void prvDelWndFromStack(xWindow *pxN, xWindow *pxNext) {
9999

100100
xWidget * pxPrev; //пред. окно в стеке
101101
xWindowProps *xP;

‎Widgets/Interface.h

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,22 +39,22 @@ extern "C" {
3939
xInterface * pxInterfaceCreate(bool(*pxOnCreateHandler)(xWidget *));
4040
void vInterfaceDraw();
4141
xInterface *pxInterfaceGet();
42-
uint16_t inline usInterfaceGetW() {
42+
inline uint16_t usInterfaceGetW() {
4343
return LCD_SizeX;
4444
}
45-
uint16_t inline usInterfaceGetH() {
45+
inline uint16_t usInterfaceGetH() {
4646
return LCD_SizeY;
4747
}
48-
uint16_t inline usInterfaceGetWindowH() {
48+
inline uint16_t usInterfaceGetWindowH() {
4949
return LCD_SizeY - LCD_STATUS_BAR_HEIGHT;
5050
}
51-
uint16_t inline usInterfaceGetWindowW() {
51+
inline uint16_t usInterfaceGetWindowW() {
5252
return LCD_SizeX;
5353
}
54-
uint16_t inline usInterfaceGetWindowX() {
54+
inline uint16_t usInterfaceGetWindowX() {
5555
return 0;
5656
}
57-
uint16_t inline usInterfaceGetWindowY() {
57+
inline uint16_t usInterfaceGetWindowY() {
5858
return LCD_STATUS_BAR_HEIGHT;
5959
}
6060
void vInterfaceInvalidate();

‎Widgets/Label.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ static char* prvCountLine(char *pcLine, uint16_t uXFrom, uint16_t uXTo, uint16_t
4444
pcSeparator = NULL;
4545
pcCharCount = pcLine;
4646
uXBefor = uXFrom;
47+
uXSeparator = uXBefor;
4748

4849
while ((uXBefor + ucFontGetCharW(*pcCharCount, pubFont)) <= uXTo && uMaxLineLen) {
4950
if (((*pcCharCount == ' ' || *pcCharCount == '\t') && pcCharCount != pcLine) // и разделитель будет не первый символ
@@ -140,7 +141,7 @@ static void prvPrintPage(char *pcPage, uint16_t uXFrom, uint16_t uXTo, uint16_t
140141
}
141142
}
142143

143-
bool static prvDraw(xWidget *pxW) {
144+
static bool prvDraw(xWidget *pxW) {
144145
xLabelProps *xP;
145146
uint16_t uY, uLineCount;
146147

‎Widgets/MenuButton.h

Lines changed: 29 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
* @file MenuButton.h
33
* @author Lemeshev K ([email protected])
44
* @date 21.02.2014
5-
* @brief Ðåàëèçàöèÿ âèäæåòà êíîïêè äëÿ ìåíþ è äèàëîãîâ
5+
* @brief Реализация виджета кнопки для меню и диалогов
66
*
77
*/
88
#ifndef MENUBUTTON_H_
@@ -26,38 +26,39 @@
2626
extern "C" {
2727
#endif // __cplusplus
2828

29-
typedef xWidget xMenuButton; ///< òèï äëÿ êíîïêè ìåíþ
29+
typedef xWidget xMenuButton; ///< тип для кнопки меню
3030

31-
/**
32-
* @brief ñîçäàåò âèäæåò êíîïêè
33-
*
34-
* @param uX - ïîëîæåíèå ïî ãîðèçîíòàëè îòíîñèòåëüíî ðîäèòåëüñêîãî ýëåìåíòà
35-
* @param uY - ïîëîæåíèå ïî âåðêèêàëè îòíîñèòåëüíî ðîäèòåëüñêîãî ýëåìåíòà
36-
* @param xPicFG - óêàçàòåëü íà êàðòèíêó
37-
* @param strLabel - ïîäïèñü ïîä êíîïêîé
38-
* @param pvClickHanlder - îáðàáîò÷èê íàæàòèÿ
39-
* @param pxWidParent - ðîäèòåëü âèäæåòà
40-
*
41-
* @returns óêàçàòåëü íà ñîçäàííóþ êíîïêó
42-
*/
31+
/**
32+
* @brief создает виджет кнопки
33+
*
34+
* @param uX - положение по горизонтали относительно родительского элемента
35+
* @param uY - положение по веркикали относительно родительского элемента
36+
* @param xPicFG - указатель на картинку
37+
* @param strLabel - подпись под кнопкой
38+
* @param pvClickHanlder - обработчик нажатия
39+
* @param pxWidParent - родитель виджета
40+
*
41+
* @returns указатель на созданную кнопку
42+
*/
4343
xMenuButton * pxMenuButtonCreate(uint16_t usX, uint16_t usY, xPicture pusPic, char* strLabel, bool(*pvClickHanlder) (xWidget *), xWidget *pxWidParent);
4444

45-
/**
46-
* @brief óñòàíàâëèâàåò êàðòèíêó íà êíîïêå
47-
*
48-
* @param pxW - âèäæåò
49-
* @param xPic - óêàçàòåëü íà êàðòèíêó
50-
*
51-
*/
45+
46+
/**
47+
* @brief устанавливает картинку на кнопке
48+
*
49+
* @param pxW - виджет
50+
* @param xPic - указатель на картинку
51+
*
52+
*/
5253
void pxMenuButtonSetMainPic(xWidget * pxW, xPicture pusPic);
5354

54-
/**
55-
* @brief óñòàíàâëèâàåò ïîäïèñü ïîä êíîïêîé
56-
*
57-
* @param pxW - âèäæåò
58-
* @param xPic - óêàçàòåëü ñòðîêó
59-
*
60-
*/
55+
/**
56+
* @brief устанавливает подпись под кнопкой
57+
*
58+
* @param pxW - виджет
59+
* @param xPic - указатель строку
60+
*
61+
*/
6162
void pxMenuButtonSetLabelText(xWidget * pxW, char const* strL);
6263

6364
/**

‎Widgets/ModalDialog.c

Lines changed: 24 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,9 @@ extern "C" {
3838

3939
static xLabel *xMessage;
4040
//static xProgressBar *xPBar;
41-
static xMenuButton *xButtons[MODAL_DIALOG_MAX_BUTTONS]; // y(ok)/n/c Ìàêñèìóì â äèàëîãå âèäíî 4 êíîïêè.
41+
static xMenuButton *xButtons[MODAL_DIALOG_MAX_BUTTONS]; // y(ok)/n/c Максимум в диалоге видно 4 кнопки.
4242

43-
//àâòîíóìåðàöèÿ äëÿ àâòîìàòè÷åñêèõ äèàëîãîâ
43+
//автонумерация для автоматических диалогов
4444
uint16_t usDlgID = MODAL_AUTO + 1;
4545

4646
typedef struct xModalDialog_t xModalDialog;
@@ -90,23 +90,23 @@ extern "C" {
9090
//vWidgetHide(xPBar);
9191
}
9292

93-
bool static prvOnOpenHandler(xWidget *pxW) {
93+
static bool prvOnOpenHandler(xWidget *pxW) {
9494
prvDlgShowActive();
9595
return true;
9696
}
9797

98-
bool static prvOnOpenRequestHandler(xWidget *pxW) {
98+
static bool prvOnOpenRequestHandler(xWidget *pxW) {
9999
if (xMDActive)
100100
return true;
101101
else
102102
return false;
103103
}
104104

105-
bool static prvOnCloseHandler(xWidget *pxW) {
105+
static bool prvOnCloseHandler(xWidget *pxW) {
106106
return true;
107107
}
108108

109-
bool static prvOnCloseRequestHandler(xWidget *pxW) {
109+
static bool prvOnCloseRequestHandler(xWidget *pxW) {
110110
if (xMDActive)
111111
vModalDialogClose(xMDActive->usDlgID, true);
112112

@@ -116,10 +116,10 @@ extern "C" {
116116
return true;
117117
}
118118

119-
bool static prvButtonHandler(xWidget *pxW) {
120-
//äèàëîã â îáðàáîò÷èêå ìîæåò îòêðûòü åùå îäèí äèàëîã.
121-
//ïðè ýòîì ID àêòîâíîãî èçìåíÿåòñÿ. Äëÿ ñîõðàíåíèÿ èñïîëüçóåì
122-
//ýòó ïåðåìåííóþ.
119+
static bool prvButtonHandler(xWidget *pxW) {
120+
//диалог в обработчике может открыть еще один диалог.
121+
//при этом ID актовного изменяется. Для сохранения используем
122+
//эту переменную.
123123
int usDlgId = xMDActive->usDlgID;
124124
if (!xMDActive)
125125
return false;
@@ -135,7 +135,7 @@ extern "C" {
135135

136136
xWidget * pxModalDialogWindowCreate() {
137137

138-
// X0, Y0 - êîîðäèíàòû ðàñïîëîæåíèÿ âèäæåòîâ
138+
// X0, Y0 - координаты расположения виджетов
139139
uint16_t usX, usY;
140140

141141
xThisWnd = pxWindowCreate(WINDOW_MODAL);
@@ -217,7 +217,7 @@ extern "C" {
217217
//return;
218218
vInterfaceCloseWindow(WINDOW_MODAL);
219219
return;
220-
//TODO: âûñòàâèòü êîë-âî àêòèâíûõ äèàëîãîâ â 0
220+
//TODO: выставить кол-во активных диалогов в 0
221221
}
222222

223223
uint16_t cBtnCnt = (uint16_t)strlen(xDlg->sDialogConfig);
@@ -291,8 +291,7 @@ extern "C" {
291291

292292
inline xModalDialog *prvDelDlgFromStack(xModalDialog *pxN, xModalDialog *pxNext) {
293293

294-
xModalDialog * pxPrev; //ïðåä. äèàëîã â ñòåêå
295-
294+
xModalDialog * pxPrev; //пред. диалог в стеке
296295
pxPrev = pxN->pxPrev;
297296
pxNext->pxPrev = pxPrev;
298297

@@ -320,27 +319,27 @@ extern "C" {
320319
xModalDialog * xDlg;
321320
xModalDialog * xDlgNext;
322321

323-
/*/Ïðîâåðêà îãðàíè÷åíèÿ ìàêñ. êîëè÷åñòâà îòêðûòûõ äèàëîãîâ
322+
/*/Проверка ограничения макс. количества открытых диалогов
324323
if(cDialogCount >= MODAL_DIALOG_MAX_COUNT){
325324
vInterfaceOpenWindow(WINDOW_MODAL);
326325
return -1;
327326
}*/
328327

329-
//Åñëè òàêîé äèàëîã óæå àêòèâåí - Ñáðîñèòü åãî îáðàáîò÷èêè è êîíôèãóðàöèþ.
328+
//Если такой диалог уже активен - Сбросить его обработчики и конфигурацию.
330329
if ((xDlg = prvDlgIsActive(iDlgId))) {
331330
prvDlgRefresh(xDlg, sBtns, sHdr, sMsg);
332331
prvDlgShowActive();
333332
vInterfaceOpenWindow(WINDOW_MODAL);
334333
return -1;
335334
}
336335

337-
//Äèàëîã åñòü ãäå-òî â ñòåêå, åãî íàäî ïîäíÿòü
336+
//Диалог есть где-то в стеке, его надо поднять
338337
if ((xDlg = prvDlgIsOpened(iDlgId, &xDlgNext))) {
339338
prvDlgRefresh(xDlg, sBtns, sHdr, sMsg);
340339
prvDelDlgFromStack(xDlg, xDlgNext);
341340
}
342341
else {
343-
//Äèàëîãà â ñòåêå íåò, ñîçäàåì íîâóþ ñòðóêòóðó
342+
//Диалога в стеке нет, создаем новую структуру
344343
xDlg = malloc(sizeof(xModalDialog));
345344
//cDialogCount ++;
346345
memcpy(xDlg->sDialogConfig, sBtns, MODAL_DIALOG_MAX_BUTTONS + 1);
@@ -354,8 +353,8 @@ extern "C" {
354353
for (int c = 0; c < MODAL_DIALOG_MAX_BUTTONS; c++)
355354
xDlg->pxClickHandlers[c] = NULL;
356355

357-
//Âûñòàâëÿåì èäåíòèôèêàòîð äèàëîãà, ïî êîòîðîìó
358-
//áóäåò âîçìîæíîñòü äàëüíåéøåé ðàáîòû ñ ýòèì äèàëîãîì.
356+
//Выставляем идентификатор диалога, по которому
357+
//будет возможность дальнейшей работы с этим диалогом.
359358
if (iDlgId != MODAL_AUTO)
360359
xDlg->usDlgID = iDlgId;
361360
else {
@@ -364,7 +363,7 @@ extern "C" {
364363
}
365364
}
366365

367-
//äåëàåì äèàëîã àêòèâíûì
366+
//делаем диалог активным
368367
xDlg->pxPrev = xMDActive;
369368
xMDActive = xDlg;
370369

@@ -432,22 +431,22 @@ extern "C" {
432431

433432
xDlg = prvDlgIsOpened(iDlgID, &xDlgNext);
434433

435-
//òàêîé äèàëîã íå áûë â ñòåêå
434+
//такой диалог не был в стеке
436435
if (!xDlg)
437436
return;
438437

439-
//åñëè äèàëîã íåëüçÿ çàêðûâàòü
438+
//если диалог нельзя закрывать
440439
if (!xDlg->bCanClose)
441440
return;
442441

443-
//åñëè äèàëîã àêòèâíûé, òî ñòàâèì àêòèâíûì ïðåäûäóùèé.
442+
//если диалог активный, то ставим активным предыдущий.
444443
if (prvDlgIsActive(iDlgID)) {
445444
xMDActive = xMDActive->pxPrev;
446445
}
447446
else
448447
prvDelDlgFromStack(xDlg, xDlgNext);
449448

450-
//ãåíåðèðóåì ñîáûòèå ïî óìîë÷àíèþ, åñëè íàäî è îíî åñòü.
449+
//генерируем событие по умолчанию, если надо и оно есть.
451450
if (bFireDefault) {
452451
if (xDlg->pxDefaultHandler)
453452
xDlg->pxDefaultHandler(NULL);

‎Widgets/StatusBar.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ extern "C" {
4949
*
5050
* @returns ширина в пикселях
5151
*/
52-
uint16_t inline usStatusBarGetW() {
52+
inline uint16_t usStatusBarGetW() {
5353
return LCD_SizeX;
5454
}
5555

@@ -58,7 +58,7 @@ extern "C" {
5858
*
5959
* @returns высота в пикселях
6060
*/
61-
uint16_t inline usStatusBarGetH() {
61+
inline uint16_t usStatusBarGetH() {
6262
return LCD_STATUS_BAR_HEIGHT;
6363
}
6464

0 commit comments

Comments
 (0)