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

Release 5.4.0 #1795

Merged
merged 130 commits into from
Apr 13, 2022
Merged

Release 5.4.0 #1795

merged 130 commits into from
Apr 13, 2022

Conversation

JulianKast
Copy link
Contributor

@JulianKast JulianKast commented Mar 8, 2022

5.4.0 RC

This PR is [ready] for review.

Risk

This PR makes [minor] API changes.

Testing Plan

  • Full test plan
  • Run all unit tests
  • Additional smoke tests

Summary

See the 5.4.0 Project

CLA

bilal-alsharifi and others added 30 commits July 28, 2021 14:54
# Conflicts:
#	base/src/main/java/com/smartdevicelink/managers/screen/menu/MenuReplaceUtilities.java
@codecov
Copy link

codecov bot commented Mar 8, 2022

Codecov Report

Merging #1795 (81f6273) into master (db5c943) will decrease coverage by 0.35%.
The diff coverage is 23.12%.

❗ Current head 81f6273 differs from pull request most recent head a174b5d. Consider uploading reports for the commit a174b5d to get more accurate results

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1795      +/-   ##
============================================
- Coverage     54.38%   54.02%   -0.36%     
- Complexity     5501     5520      +19     
============================================
  Files           562      562              
  Lines         25467    25757     +290     
  Branches       3317     3382      +65     
============================================
+ Hits          13849    13915      +66     
- Misses        10358    10579     +221     
- Partials       1260     1263       +3     
Impacted Files Coverage Δ
...icelink/managers/lockscreen/LockScreenManager.java 46.74% <0.00%> (ø)
...tdevicelink/managers/video/VideoStreamManager.java 21.87% <0.00%> (-0.53%) ⬇️
...managers/video/resolution/VideoStreamingRange.java 52.72% <0.00%> (ø)
...icelink/transport/MultiplexBluetoothTransport.java 4.23% <ø> (ø)
...link/transport/USBAccessoryAttachmentActivity.java 0.00% <0.00%> (ø)
...artdevicelink/transport/utl/SdlDeviceListener.java 7.81% <0.00%> (-0.13%) ⬇️
...in/java/com/smartdevicelink/util/AndroidTools.java 19.17% <0.00%> (-1.72%) ⬇️
...a/com/smartdevicelink/managers/BaseSdlManager.java 49.24% <0.00%> (-0.57%) ⬇️
...tdevicelink/managers/file/UploadFileOperation.java 67.83% <0.00%> (ø)
...anagers/lifecycle/BaseSystemCapabilityManager.java 67.67% <ø> (ø)
... and 48 more

@JulianKast JulianKast changed the title 5.4.0 RC Release 5.4.0 Mar 8, 2022
joeygrover and others added 14 commits March 23, 2022 11:52
* Add UAI request to clean method in LCM

This makes sure to send an UAI request when the LCM is shutting down to clear it from the head unit. Also removed redundent code to cose services and ignore the UAI response because there’s nothing the library can do at that point anyways.

* Add missing initilize to JavaSE LCM

* Fix incorrect start call when should stop in SdlManager

* Add states to LCM

* Prevent duplicate calls to SdlManager.dispose

* Fix formatting issues for shutdown fix

#1796

Co-authored-by: Julian Kast <[email protected]>

Co-authored-by: Julian Kast <[email protected]>
…d strings (#1799)

* Add error logging when removing empty commands

* Switch from logging error to logging warnings

* Update warning messages based on PR suggestions
* Fix RS exit foreground params

* Refactor to call extiForeground for every close()

In the RS

* Fix altTransport timer issues preventing RS close

* Fix incorrect spacing PR #1797

Co-authored-by: Julian Kast <[email protected]>

Co-authored-by: Julian Kast <[email protected]>
This commit adds null checks and a List.size() check to PresentAlertOperation.supportsSoftButtonImages()
These checks prevent crashes and returns true when soft button capabilities are unavailable
This new behavior improves alignment with the other SDL libraries
* Improve handling of unsupported resolutions

Adds a check in VirtualDisplayEncoder for unsupported resolutions that makes the virtual display encoder fail to start, prints a more helpful error message, and propagates an exception to the video stream manager
Changes in VideoStreamManager now kill the video stream service when the encoder fails to start, allowing for subsequent streams to start after an encoder error

* Add missing spaces to if statements
…icitly released -- state may be leaked", So I modified logic to release it if we have already created EglCore object and are creating a new one
…form an HMI state that was not streamable to one that is.
…_error

Fix auto restart video stream to ensure hmi level is streamable
@joeygrover joeygrover merged commit 16b2ab6 into master Apr 13, 2022
@joeygrover joeygrover deleted the 5.4.0_RC branch April 13, 2022 15:03
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.

7 participants