Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

's_boot_time_lock' , 's_boot_time', undeclared, ) newlib\time.c #238

Closed
ESP32DE opened this issue Jan 15, 2017 · 2 comments
Closed

's_boot_time_lock' , 's_boot_time', undeclared, ) newlib\time.c #238

ESP32DE opened this issue Jan 15, 2017 · 2 comments
Milestone

Comments

@ESP32DE
Copy link
Contributor

ESP32DE commented Jan 15, 2017

hi

C:/sdk32/esp-idf/components/newlib/time.c: In function 'set_boot_time':
C:/sdk32/esp-idf/components/newlib/time.c:94:20: error: 's_boot_time_lock' undeclared (first use in this function)
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L94

_lock_acquire(&s_boot_time_lock);
               ^

C:/sdk32/esp-idf/components/newlib/time.c:94:20: note: each undeclared identifier is reported only once for each function it appears in
C:/sdk32/esp-idf/components/newlib/time.c:99:5: error: 's_boot_time' undeclared (first use in this function)
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L99

s_boot_time = time_us;
^

C:/sdk32/esp-idf/components/newlib/time.c: In function 'get_boot_time':
C:/sdk32/esp-idf/components/newlib/time.c:107:20: error: 's_boot_time_lock' undeclared (first use in this function)
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L107

_lock_acquire(&s_boot_time_lock);
               ^

C:/sdk32/esp-idf/components/newlib/time.c:111:14: error: 's_boot_time' undeclared (first use in this function)
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L111

result = s_boot_time;
         ^

C:/sdk32/esp-idf/components/newlib/time.c: At top level:

C:/sdk32/esp-idf/components/newlib/time.c:92:13: warning: 'set_boot_time' defined but not used [-Wunused-function]
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L92

static void set_boot_time(uint64_t time_us)
^
C:/sdk32/esp-idf/components/newlib/time.c:104:17: warning: 'get_boot_time' defined but not used [-Wunused-function]
https://github.com/espressif/esp-idf/blob/master/components/newlib/time.c#L104

static uint64_t get_boot_time()
^

best wishes
rudi ;-)

@negativekelvin
Copy link
Contributor

Is this with menuconfig Timers used for gettimeofday function = none?

@ESP32DE
Copy link
Contributor Author

ESP32DE commented Jan 15, 2017

had a look - yes there stands = none
for test i change to FRC1 and i can compile. ( txs for the hint )
it seems if gettimeofday function = none the things are aktive without declaration
best wishes
rudi ;-)

@igrr igrr added the Status: In Progress Work is in progress label Jan 16, 2017
@igrr igrr closed this as completed in 7776d3e Jan 17, 2017
igrr added a commit that referenced this issue Jan 17, 2017
Fix issues with time syscalls

- fix compilation error when no time source is selected (#238)
- work around a bug that time speeds up when RTC registers are read on the APP CPU (espressif/arduino-esp32#120)


See merge request !427
@igrr igrr removed the Status: In Progress Work is in progress label Jan 20, 2017
@igrr igrr modified the milestone: v2.0 Jan 20, 2017
0xFEEDC0DE64 pushed a commit to 0xFEEDC0DE64/esp-idf that referenced this issue May 5, 2021
* Rework WiFiClient

Rework WiFiClient to correct error where making a copy of a WiFiClient object resulted in the socket being closed prematurely.

Added loop and select to write to handle/prevent EAGAIN errors.

* Rework WiFiClient to use shared_ptr

Rework changes to utilize shared_ptr rather than manually maintaining reference count. 

Revert changes to write

* Incorporate comments from review

Move WiFiClientSocketHandle and fd() into WiFiClient.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants