GyverLibs / GyverOLED

Лёгкая и быстрая библиотека для OLED дисплея
MIT License
123 stars 20 forks source link

Update GyverOLED.h #21

Closed izayash closed 1 year ago

izayash commented 1 year ago

добавлена возможность переназначения портов I2c, только при использовании библиотеки Wire

GyverLibs commented 1 year ago

Господи, ещё один. В таком виде библиотека перестанет компилироваться для avr arduino

izayash commented 1 year ago

"ещё один"? Не очень вежливо звучит конечно. Давайте добавим " && !defined(AVR)" в строке 183? С такими правками она компилится. Методом тыка не нашел, как исправить свои правки в gite.

Редактор гита скрывает __

GyverLibs commented 1 year ago

Да, уже третий) не думает о юзерах других платформ. Я PR мержить не буду, есть ещё пара моментов которые нужно исправить к обновлению библиотеки

izayash commented 1 year ago

так какие дальше действия то? я пытался закрыть свой же issue https://github.com/GyverLibs/GyverOLED/issues/20 Вы вроде написали в нем - "да, можно". Но как это понимать, неясно. Мне новый исправленный реквест сделать? или что?

GyverLibs commented 1 year ago

"да, можно" означало как "отличная идея, сделаю обязательно". Уже сделал, версия 1.6

izayash commented 1 year ago

Понятно, спасибо за оперативность. Из кода не могу понять, как переназначать пины. Может стоит добавить пример в README.md?

GyverLibs commented 1 year ago

init(sda, scl)