#1 2017-02-15 22:48:42

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Вопрос к знаоткам С++ по макросам

Всем привет.
Гуру С++, объясните пожалуйста почему не компилится скетч.
Вот код:

#include <TimeLib.h>
void setup() {
  unsigned long t=minutesToTime_t(1);
  Serial.println(t);
}

void loop() {
}

в TimeLib.h есть такой макрос:

/* Useful Macros for converting elapsed time to a time_t */
#define minutesToTime_t ((M)) ( (M) * SECS_PER_MIN)   

вот его я и вызываю в своем скетче, но он не компилится, выдает ошибку:
TimeLib.h:92:27: error: 'M' was not declared in this scope
#define minutesToTime_t ((M)) ( (M) * SECS_PER_MIN) 
то есть он видит, что есть такой макрос, но почему-то не хочет его исполнять, а расценивает его как функцию, как я понимаю? Или макрос можно использовать только в том же файле в котором он объявлен?

Неактивний

#2 2017-02-16 07:57:58

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Должно быть так

#define minutesToTime_t((M)) ( (M) * SECS_PER_MIN) 

Неактивний

#3 2017-02-16 17:52:12

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

В смысле пробел убрать? Пробовал - результат тот же. Да и сам TimeLib.h успешно компилится, если не вызывать его макросы

Неактивний

#4 2017-02-16 18:45:04

NoName
Customer
З Київ
Зареєстрований: 2014-07-08
Повідомлень: 1,446

Re: Вопрос к знаоткам С++ по макросам

void main()
{

#define  SECS_PER_MIN  60
#define minutesToTime_t(M)  ((M)*SECS_PER_MIN)
	unsigned long  foo;

	foo =  minutesToTime_t (10);

foo = 600

не дефайн,  а целый синтаксический сахар )

Неактивний

#5 2017-02-16 19:28:40

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Нормал. Секс пер мин может быть любой, согласны?))
Я обычно использую MS2SYS(x) (x)/T_SYS, то бишь перевожу МС в системные тики.

Неактивний

#6 2017-02-16 20:48:01

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Андрей пише:

В смысле пробел убрать? Пробовал - результат тот же. Да и сам TimeLib.h успешно компилится, если не вызывать его макросы

Что пробовал?

Неактивний

#7 2017-02-17 11:14:13

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Green пише:

Что пробовал?

Заменил на вот так (то есть убрал пробел) в TimeLib.h

Green пише:

Должно быть так

#define minutesToTime_t((M)) ( (M) * SECS_PER_MIN) 

Неактивний

#8 2017-02-17 12:01:51

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Ну, и... Партизан?

Неактивний

#9 2017-02-17 22:48:01

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Так говорю ж - результат тот же. Что с тем вариантом что у меня, что с тем, что вы предложили.

Неактивний

#10 2017-02-18 07:22:32

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Не верю! Хочу сообщение об ошибке.

Неактивний

#11 2017-02-18 08:50:50

Roman1984
Учасник
Зареєстрований: 2017-02-17
Повідомлень: 39

Re: Вопрос к знаоткам С++ по макросам

А в каком месте кода вы привязывае макрос? Можно увидеть код полностью?

Остання редакція Roman1984 (2017-02-18 08:53:06)

Неактивний

#12 2017-02-18 12:07:03

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Давайте я еще раз опишу проблему.
Если я делаю вот так:

#define SECS_PER_MIN  (60UL)
#define minutesToTime_t(M) ( (M) * SECS_PER_MIN) 
void setup() {
  unsigned long t=minutesToTime_t(1);
  Serial.println(t);
}

void loop() {
}

то все компилится ок.
Но я хочу использовать дефайны из внешнего файла библиотеки времени TimeLib.h без необходимости копировать их в основной скетч. Методы этой библиотеки вызываются нормально из чего я делаю вывод, что с ней все ок и она тоже компилится без ошибок. Но если я пытаюсь получить доступ к макросам, которые в ней используются, то получаю ошибку. Чтобы не грузить вас избыточным кодом в первом посте простой скетч, который это демонстрирует.
На всякий вот сам файл TimeLib.h

/*
  time.h - low level time and date functions
*/

/*
  July 3 2011 - fixed elapsedSecsThisWeek macro (thanks Vincent Valdy for this)
              - fixed  daysToTime_t macro (thanks maniacbug)
*/     

#ifndef _Time_h
#ifdef __cplusplus
#define _Time_h

#include <inttypes.h>
#ifndef __AVR__
#include <sys/types.h> // for __time_t_defined, but avr libc lacks sys/types.h
#endif


#if !defined(__time_t_defined) // avoid conflict with newlib or other posix libc
typedef unsigned long time_t;
#endif


// This ugly hack allows us to define C++ overloaded functions, when included
// from within an extern "C", as newlib's sys/stat.h does.  Actually it is
// intended to include "time.h" from the C library (on ARM, but AVR does not
// have that file at all).  On Mac and Windows, the compiler will find this
// "Time.h" instead of the C library "time.h", so we may cause other weird
// and unpredictable effects by conflicting with the C library header "time.h",
// but at least this hack lets us define C++ functions as intended.  Hopefully
// nothing too terrible will result from overriding the C library header?!
extern "C++" {
typedef enum {timeNotSet, timeNeedsSync, timeSet
}  timeStatus_t ;

typedef enum {
    dowInvalid, dowSunday, dowMonday, dowTuesday, dowWednesday, dowThursday, dowFriday, dowSaturday
} timeDayOfWeek_t;

typedef enum {
    tmSecond, tmMinute, tmHour, tmWday, tmDay,tmMonth, tmYear, tmNbrFields
} tmByteFields;	   

typedef struct  { 
  uint8_t Second; 
  uint8_t Minute; 
  uint8_t Hour; 
  uint8_t Wday;   // day of week, sunday is day 1
  uint8_t Day;
  uint8_t Month; 
  uint8_t Year;   // offset from 1970; 
} 	tmElements_t, TimeElements, *tmElementsPtr_t;

//convenience macros to convert to and from tm years 
#define  tmYearToCalendar(Y) ((Y) + 1970)  // full four digit year 
#define  CalendarYrToTm(Y)   ((Y) - 1970)
#define  tmYearToY2k(Y)      ((Y) - 30)    // offset is from 2000
#define  y2kYearToTm(Y)      ((Y) + 30)   

typedef time_t(*getExternalTime)();
//typedef void  (*setExternalTime)(const time_t); // not used in this version


/*==============================================================================*/
/* Useful Constants */
#define SECS_PER_MIN  ((time_t)(60UL))
#define SECS_PER_HOUR ((time_t)(3600UL))
#define SECS_PER_DAY  ((time_t)(SECS_PER_HOUR * 24UL))
#define DAYS_PER_WEEK ((time_t)(7UL))
#define SECS_PER_WEEK ((time_t)(SECS_PER_DAY * DAYS_PER_WEEK))
#define SECS_PER_YEAR ((time_t)(SECS_PER_WEEK * 52UL))
#define SECS_YR_2000  ((time_t)(946684800UL)) // the time at the start of y2k
 
/* Useful Macros for getting elapsed time */
#define numberOfSeconds(_time_) (_time_ % SECS_PER_MIN)  
#define numberOfMinutes(_time_) ((_time_ / SECS_PER_MIN) % SECS_PER_MIN) 
#define numberOfHours(_time_) (( _time_% SECS_PER_DAY) / SECS_PER_HOUR)
#define dayOfWeek(_time_)  ((( _time_ / SECS_PER_DAY + 4)  % DAYS_PER_WEEK)+1) // 1 = Sunday
#define elapsedDays(_time_) ( _time_ / SECS_PER_DAY)  // this is number of days since Jan 1 1970
#define elapsedSecsToday(_time_)  (_time_ % SECS_PER_DAY)   // the number of seconds since last midnight 
// The following macros are used in calculating alarms and assume the clock is set to a date later than Jan 1 1971
// Always set the correct time before settting alarms
#define previousMidnight(_time_) (( _time_ / SECS_PER_DAY) * SECS_PER_DAY)  // time at the start of the given day
#define nextMidnight(_time_) ( previousMidnight(_time_)  + SECS_PER_DAY )   // time at the end of the given day 
#define elapsedSecsThisWeek(_time_)  (elapsedSecsToday(_time_) +  ((dayOfWeek(_time_)-1) * SECS_PER_DAY) )   // note that week starts on day 1
#define previousSunday(_time_)  (_time_ - elapsedSecsThisWeek(_time_))      // time at the start of the week for the given time
#define nextSunday(_time_) ( previousSunday(_time_)+SECS_PER_WEEK)          // time at the end of the week for the given time


/* Useful Macros for converting elapsed time to a time_t */
#define minutesToTime_t((M)) ( (M) * SECS_PER_MIN) 
#define hoursToTime_t   ((H)) ( (H) * SECS_PER_HOUR)  
#define daysToTime_t    ((D)) ( (D) * SECS_PER_DAY) // fixed on Jul 22 2011
#define weeksToTime_t   ((W)) ( (W) * SECS_PER_WEEK)   

/*============================================================================*/
/*  time and date functions   */
int     hour();            // the hour now 
int     hour(time_t t);    // the hour for the given time
int     hourFormat12();    // the hour now in 12 hour format
int     hourFormat12(time_t t); // the hour for the given time in 12 hour format
uint8_t isAM();            // returns true if time now is AM
uint8_t isAM(time_t t);    // returns true the given time is AM
uint8_t isPM();            // returns true if time now is PM
uint8_t isPM(time_t t);    // returns true the given time is PM
int     minute();          // the minute now 
int     minute(time_t t);  // the minute for the given time
int     second();          // the second now 
int     second(time_t t);  // the second for the given time
int     day();             // the day now 
int     day(time_t t);     // the day for the given time
int     weekday();         // the weekday now (Sunday is day 1) 
int     weekday(time_t t); // the weekday for the given time 
int     month();           // the month now  (Jan is month 1)
int     month(time_t t);   // the month for the given time
int     year();            // the full four digit year: (2009, 2010 etc) 
int     year(time_t t);    // the year for the given time

time_t now();              // return the current time as seconds since Jan 1 1970 
void    setTime(time_t t);
void    setTime(int hr,int min,int sec,int day, int month, int yr);
void    adjustTime(long adjustment);

/* date strings */ 
#define dt_MAX_STRING_LEN 9 // length of longest date string (excluding terminating null)
char* monthStr(uint8_t month);
char* dayStr(uint8_t day);
char* monthShortStr(uint8_t month);
char* dayShortStr(uint8_t day);
	
/* time sync functions	*/
timeStatus_t timeStatus(); // indicates if time has been set and recently synchronized
void    setSyncProvider( getExternalTime getTimeFunction); // identify the external time provider
void    setSyncInterval(time_t interval); // set the number of seconds between re-sync

/* low level functions to convert to and from system time                     */
void breakTime(time_t time, tmElements_t &tm);  // break time_t into elements
time_t makeTime(tmElements_t &tm);  // convert time elements into time_t

} // extern "C++"
#endif // __cplusplus
#endif /* _Time_h */

Код ошибки:

"C:\Users\Home\AppData\Local\Arduino15\packages\esp8266\tools\xtensa-lx106-elf-gcc\1.20.0-26-gb404fb9-2/bin/xtensa-lx106-elf-g++" -D__ets__ -DICACHE_FLASH -U__STRICT_ANSI__ "-IC:\Users\Home\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0/tools/sdk/include" "-IC:\Users\Home\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0/tools/sdk/lwip/include" "-IC:\Users\Home\AppData\Local\Temp\build27c3b10ae9e4832c957ff9f5660570ad.tmp/core" -c -w -Os -g -mlongcalls -mtext-section-literals -fno-exceptions -fno-rtti -falign-functions=4 -std=c++11  -ffunction-sections -fdata-sections -w -x c++ -E -CC -DF_CPU=80000000L -DLWIP_OPEN_SRC   -DARDUINO=10611 -DARDUINO_ESP8266_NODEMCU -DARDUINO_ARCH_ESP8266 -DARDUINO_BOARD="ESP8266_NODEMCU"  -DESP8266 "-IC:\Users\Home\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\cores\esp8266" "-IC:\Users\Home\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\variants\nodemcu" "-IC:\Users\Home\Documents\Arduino\libraries\Time" "C:\Users\\AppData\Local\Temp\build27c3b10ae9e4832c957ff9f5660570ad.tmp\sketch\sketch_feb18a.ino.cpp" -o "C:\Users\Home\AppData\Local\Temp\build27c3b10ae9e4832c957ff9f5660570ad.tmp\preproc\ctags_target_for_gcc_minus_e.cpp"
In file included from C:\Users\Home\AppData\Local\Temp\arduino_modified_sketch_214122\sketch_feb18a.ino:1:0:

C:\Users\Home\Documents\Arduino\libraries\Time/TimeLib.h:92:25: error: "(" may not appear in macro parameter list

#define minutesToTime_t((M)) ( (M) * SECS_PER_MIN)

                         ^

Используем библиотеку Time версии 1.5 из папки: C:\Users\Home\Documents\Arduino\libraries\Time
exit status 1
Ошибка компиляции для платы NodeMCU 1.0 (ESP-12E Module).

Неактивний

#13 2017-02-18 12:15:56

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

#include <TimeLib.h> где?

Остання редакція Green (2017-02-18 12:21:43)

Неактивний

#14 2017-02-18 12:41:08

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

И пару скобок вокруг М нужно убрать.

Неактивний

#15 2017-02-18 14:09:49

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Green пише:

#include <TimeLib.h> где?

В смысле где лежит? В хомах арудино ИДЕ libraries\Time\TimeLib.h

Неактивний

#16 2017-02-18 20:10:17

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Шевелитесь, Андрей, шевелитесь. Спасибо хочу услышать.))

Неактивний

#17 2017-02-22 14:06:35

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Green пише:

Шевелитесь, Андрей, шевелитесь. Спасибо хочу услышать.))

Спасибо! Так и правда работает. Не могу только понять как оно компилилось со скобками без ошибок, когда я просто вставлял TimeLib.h в свой скетч.

Неактивний

#18 2017-02-22 14:57:30

NoName
Customer
З Київ
Зареєстрований: 2014-07-08
Повідомлень: 1,446

Re: Вопрос к знаоткам С++ по макросам

" Не могу только понять как оно компилилось со скобками без ошибок, когда я просто вставлял TimeLib.h в свой скетч."
индивидуальные ключи при компиляции файлов , я так думаю )
ну порядок видимости define и т.д.

"lf-g++" -D__ets__ -DICACHE_FLASH -U__STRICT_ANSI__ " 

почитать тут )
http://microelectronics.esa.int/erc32/doc/gcc.pdf
https://groups.google.com/forum/#!forum/gnu.g++.help

Остання редакція NoName (2017-02-22 15:02:16)

Неактивний

#19 2017-02-22 22:07:58

Green
Учасник
Зареєстрований: 2015-11-08
Повідомлень: 591

Re: Вопрос к знаоткам С++ по макросам

Андрей пише:
Green пише:

Шевелитесь, Андрей, шевелитесь. Спасибо хочу услышать.))

Спасибо! Так и правда работает. Не могу только понять как оно компилилось со скобками без ошибок, когда я просто вставлял TimeLib.h в свой скетч.

TimeLib.h то Вы вставляли, а сам minutesToTime_t(x) ведь не использовали! Вот ему и до лампочки. А там (в #define minutesToTime_t(x) бла-бла-бла) могли даже слова неприличные быть и ничо. До тех пор пока не начнёте его использовать. Кстати, попробуйте. А про макро почитайте. Что бы хотя б слегка представлять.
А либо-писатели поражают, конечно.((

Неактивний

#20 2017-02-22 22:31:02

Андрей
Учасник
Зареєстрований: 2014-06-17
Повідомлень: 46

Re: Вопрос к знаоткам С++ по макросам

Блин, точно.
Я не обратил внимания, что макрос объявлен, но не используется в самом TimeLib.h
Еще раз спасибо за разъяснения.

Неактивний

Швидке повідомлення

Введіть повідомлення і натисніть Надіслати

Підвал форуму