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

v5.18.0 #1573

Merged
merged 21 commits into from
Jan 5, 2022
Merged

v5.18.0 #1573

merged 21 commits into from
Jan 5, 2022

Conversation

lemnik
Copy link
Contributor

@lemnik lemnik commented Jan 5, 2022

Enhancements

  • Improve the memory use and performance overhead when handling the delivery response status codes
    #1558

  • Harden ndk layer through use of const keyword
    #1566

Bug fixes

  • Delete persisted NDK events earlier in delivery process
    #1562

  • Add null checks for strlen()
    #1563

  • Catch IOException when logging response status code
    #1567

lemnik and others added 20 commits December 14, 2021 10:29
Improve the memory and performance handling delivery response status codes
Delete persisted NDK events earlier in delivery process
Harden ndk layer through use of const keyword
Avoid dependencies on the global namespace
[quick ci]
[PLAT-6090] Harmonize all string copying in the NDK layer
…heck

Catch IOException when logging response status code
@lemnik lemnik requested a review from kstenerud January 5, 2022 11:39
@lemnik lemnik closed this Jan 5, 2022
@lemnik lemnik deleted the releases/5.18.0 branch January 5, 2022 11:41
@lemnik lemnik restored the releases/5.18.0 branch January 5, 2022 12:07
@lemnik lemnik reopened this Jan 5, 2022
@bugsnagbot
Copy link
Collaborator

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1353.03 1167.83
arm64_v8a 483.72 299.4
armeabi -492.59 -21.55
armeabi_v7a 455.06 270.74
x86 520.57 340.35
x86_64 508.29 323.97

Generated by 🚫 Danger

@lemnik lemnik merged commit 2fd4ec6 into master Jan 5, 2022
@lemnik lemnik deleted the releases/5.18.0 branch January 5, 2022 15:46
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.

6 participants