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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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 74846

View Options

applet/contents/ui/DeviceListItem.qml

Loading...