Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Add listMappings command #366

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

terahlunah
Copy link

@terahlunah terahlunah commented May 2, 2016

Add the listMappings command which list all the mappings under a given path

Example with the MusicLibrary form the Getting Started wiki :

$ remote-process localhost 5000 listMappings /MusicLibrary/my_library/artists/IronMaiden

/MusicLibrary/my_library/artists/IronMaiden [Directory:/home/fponcabx/pfw/install-examples/libraries, File:ironMaiden, File:ironMaiden]

@@           master       #366   diff @@
========================================
Files         211        211
Lines        7169       7188    +19
Methods         0          0
Branches        0          0
========================================
Hits         5760       5760
- Misses       1409       1428    +19
Partials        0          0

Powered by Codecov. Last updated by 1d51444...757ae75

  •  File: parameter/ParameterMgr.cpp:L1822-1859
  • <img border=0 src='https://avatars.githubusercontent.com/u/124108?v=3' height=16 width=16'> Can you check if you can dump only the terminal element. Today you are listing the full list of element. So to many element without control are listed. See under:
    /Audio/REALTEK/SOUND_CARD/OUTPUT/HEADPHONE/ANALOG/VOLUME => Card:bytcrrt5640, Debug:, Amend1:HP, Volume:'%1 Playback Volume', Volume:HP Playback Volume
    /Audio/REALTEK/SOUND_CARD/OUTPUT/HEADPHONE/ANALOG/VOLUME/0 => Card:bytcrrt5640, Debug:, Amend1:HP, Volume:'%1 Playback Volume', Volume:'%1 Playback Volume'
    /Audio/REALTEK/SOUND_CARD/OUTPUT/EARPIECE/ANALOG => Card:bytcrrt5640, Debug:
    /Audio/REALTEK/SOUND_CARD/OUTPUT/EARPIECE/ANALOG/SWITCH => Card:bytcrrt5640, Debug:, Control:'Mono Playback Switch', Control:Mono Playback Switch
  • <img border=0 src='https://avatars.githubusercontent.com/u/6430928?v=3' height=16 width=16'> What you seem to want isn't the terminal element (i.e. leaves, i.e. parameters) but the SubsystemObjects (the elements with an instantiation mapping).

  

@dawagner
Copy link
Contributor

dawagner commented May 2, 2016

Please add the example's output in your commit message too.

List all the mapping under a given path

Example with the MusicLibrary form the Getting Started wiki :

$ remote-process localhost 5000 listMappings /MusicLibrary/my_library/artists/IronMaiden

/MusicLibrary/my_library/artists/IronMaiden [Directory:/home/fponcabx/pfw/install-examples/libraries, File:ironMaiden, File:ironMaiden]

Signed-off-by: Florian Poncabaré <[email protected]>
@codecov-io
Copy link

codecov-io commented May 2, 2016

Current coverage is 80.13%

Merging #366 into master will increase coverage by -0.19%

@@           master       #366   diff @@
========================================
  Files         211        211          
  Lines        7169       7188    +19   
  Methods         0          0          
  Branches        0          0          
========================================
  Hits         5760       5760          
- Misses       1409       1428    +19   
  Partials        0          0          

Powered by Codecov. Last updated by 1d51444...757ae75


for (string path : paths) {
string mapping;
if (getParameterMapping(path, mapping)) {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you check if you can dump only the terminal element. Today you are listing the full list of element. So to many element without control are listed. See under:

/Audio/REALTEK/SOUND_CARD/OUTPUT/HEADPHONE/ANALOG/VOLUME => Card:bytcrrt5640, Debug:, Amend1:HP, Volume:'%1 Playback Volume', Volume:HP Playback Volume
/Audio/REALTEK/SOUND_CARD/OUTPUT/HEADPHONE/ANALOG/VOLUME/0 => Card:bytcrrt5640, Debug:, Amend1:HP, Volume:'%1 Playback Volume', Volume:'%1 Playback Volume'

/Audio/REALTEK/SOUND_CARD/OUTPUT/EARPIECE/ANALOG => Card:bytcrrt5640, Debug:
/Audio/REALTEK/SOUND_CARD/OUTPUT/EARPIECE/ANALOG/SWITCH => Card:bytcrrt5640, Debug:, Control:'Mono Playback Switch', Control:Mono Playback Switch

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What you seem to want isn't the terminal element (i.e. leaves, i.e. parameters) but the SubsystemObjects (the elements with an instantiation mapping).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants