search

espressif / esp-nimble

A fork of NimBLE stack, for use with ESP32 and ESP-IDF
Apache License 2.0
76 stars 50 forks source link

porting: use macros for log level filtering #6

Closed xentec closed 4 years ago

xentec commented 4 years ago

This reduces the size of the compiled binary by 7856 bytes.

CLAassistant commented 4 years ago

CLA assistant check
All committers have signed the CLA.

dhrishi commented 4 years ago

@xentec Thanks for the contribution. We will review this PR and get back to you.

prasad-alatkar commented 4 years ago

LGTM

dhrishi commented 4 years ago

@xentec This looks great! We will get this merged.

xentec commented 4 years ago

Nice. Thanks!

prasad-alatkar commented 4 years ago

Hi @xentec the changes are included in esp-nimble, commit: 6d659067 . I cherry-picked your commit in my existing MR, that is why the commit SHA is changed, and I suppose that may be the reason the PR has not been marked as merged. Nevertheless your contribution is visible in commit history. Thank you for your contribution.