Differential D27086

[Applet]Supplementary add port check
ClosedPublic
Actions

Authored by gvgeo on Feb 1 2020, 3:20 PM.

Details

Reviewers
drosca
ngraham
Commits
R115:ccdeb9327f17: [Applet]Supplementary add port check
Summary

Supplementary to the previous commit.
2 More ports needs check. The issue is the same, maybe crash or
display warning, if the port does not exit when try to read description.

Diff Detail

Repository
R115 Plasma Audio Volume Applet
Branch
fixes2 (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 21944
Build 21962: arc lint + arc unit
gvgeo created this revision.Feb 1 2020, 3:20 PM
Restricted Application added a project: Plasma. · View Herald TranscriptFeb 1 2020, 3:20 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
gvgeo requested review of this revision.Feb 1 2020, 3:20 PM
gvgeo edited the summary of this revision. (Show Details)Feb 1 2020, 3:23 PM
gvgeo added a comment.Feb 1 2020, 3:26 PM

previous commit is 8f0847af1d17.

ngraham accepted this revision.Feb 1 2020, 10:54 PM
This revision is now accepted and ready to land.Feb 1 2020, 10:54 PM
Closed by commit R115:ccdeb9327f17: [Applet]Supplementary add port check (authored by gvgeo, committed by ngraham). · Explain WhyFeb 1 2020, 10:54 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 74833

View Options

applet/contents/ui/DeviceListItem.qml

Loading...