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

Поддержка для пользовательей ESP32 ;) #7

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

LexNastin
Copy link

Изменил пару библиотек под ESP32 и главный код тоже. В библиотеки EEManager, EEPROM.length() почемуто всегда возврощает 0, даже после EEPROM.begin(2048), по этому зделал чтобы функция EEManager.begin() сравнивала на примую с цифрой 2048 а не с EEPROM.length().

Изменил пару библиотек под ESP32 и главный код тоже. В библиотеки
EEManager, EEPROM.length() почемуто всегда возврощает 0, даже после
EEPROM.begin(2048), по этому зделал чтобы функция EEManager.begin()
сравнивала на примую с цифрой 2048 а не с EEPROM.length().
@GyverLibs
Copy link
Contributor

Я думаю это нужно писать в issue к косоруким индусам, которые делают ядро под есп32

@LexNastin
Copy link
Author

Я если что не только то изменил, пару других вешей для поддержи esp32 тоже изменил.

@LexNastin
Copy link
Author

LexNastin commented Dec 26, 2021

Это должно полностью поддерживать esp32, я на своем тестировал.

@LexNastin
Copy link
Author

LexNastin commented Dec 26, 2021

С ядром вообше проблема в том что в глобальной instance EEPROM, перемнная "_size" не когда не настраевается, .length() работает только когда мы создоём обект EEPROM и там делаем EEPROM.begin(2048)

@GyverLibs
Copy link
Contributor

Ну так надо писать в issue, пусть исправляют) потому что instance там один создан, begin ему мы делаем, значит _size должна получить значение

@LexNastin
Copy link
Author

LexNastin commented Dec 26, 2021

Открыл там issue)

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

Successfully merging this pull request may close these issues.

2 participants