dawagner / parameter-framework

73%

master: 78%

LAST BUILD ON BRANCH fix-enumparam-name-attribute

branch: fix-enumparam-name-attribute

CHANGE BRANCH

x

Reset

• fix-enumparam-name-attribute
• Disable_fix_point_test_if_no_python_bindings
• Wconversion-enabler
• access-elements-as-binary
• add-get-set-elementXML-commands
• add-missing-export-FormattedSubsystemObject
• android-darwin
• android-m
• appveyor
• appveyor-build-more-components
• appveyor-visualstudio-2015
• archive-export-version
• better-libxml2-error-messages
• c++11-ctors-and-dtors-features
• change-utility-header-install-dir
• check-duplicate-criterion-values
• client-simulator
• client-simulator-rework
• cmake-misc-fixes
• cmake-multiproject-fix
• cmake-prefix-path-usage
• component-arrays
• convert-to-bytes
• coveralls-test
• criterion-overflow-check
• criterion-rework-step-2
• criterion-rework-step-3
• criterion-unit-tests
• david-tag
• delme
• disambiguate-make-unique
• doc-add-missing-folders
• fix-android-clang+darwin
• fix-blackboard-api-inconsistency
• fix-coveralls
• fix-criterion-display
• fix-double-negative-infinity
• fix-dynamic-lib-utility-windows
• fix-enum-parameter-casts
• fix-floats-limits-tests
• fix-getElementBytes
• fix-log-on-list
• fix-logger-private-member-leakage
• fix-makeEncodable
• fix-python-sample-crash
• fix-setParameter-array
• fix-skeleton-plugin-build
• fix-uninitialized-members
• fix-unused-results
• fix_android_darwin
• fix_illegal_integer_conversions
• floating-point
• for-seb
• forward-libxml2-error-messages
• freeze-catch-version
• improve-build-experience
• includes-cleanup
• integerparam-template
• jules-command-parser
• jules-criterion-rework_rebased-next
• make-command-interface-public
• master
• master-candidate
• misc-doxygen-tweaks
• missing-client-doxygen
• more-compilation-help-in-readme
• move-tmpfile-out-of-libparameter
• new-c++-domain-generator
• no-Werror-by-default
• no-stlport
• optional-Werror-and-release-build
• poc-memory-profiling
• port-add-Wconversion-flag
• relax-windows-errors
• remove-big-endian-subsystems
• remove-binary-settings
• remove-binary-settings-storage
• remove-deprecated
• remove-mappingdata-xmlsink-inheritance
• remove-some-dead-code
• remove-some-useless-util-methods
• remove-unused-attribute
• remove-utility-test-from-coverage
• remove-utility-tostring-use-c++11
• requirements
• rework-getattribute
• rework-parameter-handle
• rework-plugin-entrypoint-api
• rework-plugin-entrypoint-api-WIP
• selectively-export-symbols
• simplify-empty-ctors-and-dtors
• soaresm-cmake_clientSimulator
• some-coveralls-fixes
• test-skeleton-plugin
• test-skeleton-plugin-WIP
• tokenizer-control-merging-separators
• travis-no-container-based
• uncaught-exception
• unify-access-contexts
• unsigned-int-criterion-api
• update-doxygen
• v2.6.0+no-stlport
• windows-fileversion-attribute
• windows_port
• wip
• xml-generation-fixes-and-tests
• xmlgeneration-bugfix
• xmlgeneration-check-conflicting-elements
• xmlgenerator-smoketest

Wrong XML "Name" attribute in enum value pairs

When exporting an element's structure (via the C++ API or via the tuning
interface), a spurious "Name" attribute appears for EnumPair nodes, while
only "Literal" and "Numerical" attributes should be present. This is because
the XML import / export feature assummes the existence of a Name attribute
for all XML nodes. The Name default processing can however be skipped from the
toXml() methods by calling the base class' childrenToXml() method instead
of the toXml() one.

Signed-off-by: Patrick Benavoli <patrick.benavoli@intel.com>