Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • xfconf xfconf
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare
  • Issues 14
    • Issues 14
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • XfceXfce
  • xfconfxfconf
  • Merge requests
  • !28

xfconfd: Make xml backend case sensitive for channel names

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Gaël Bonithon requested to merge Tamaranch/xfconf:case-sensitive-backend into master Nov 21, 2022
  • Overview 0
  • Commits 1
  • Pipelines 2
  • Changes 1

Follow-up to !27 (comment 60205).

It was case insensitive for channel and property names since 2ab4e553, so almost originally. Then the property names became case sensitive at some point, and there are now several channels using property names containing capital letters, e.g. those of Xfce4-settings.

Channel names, on the other hand, remained case-insensitive, but only halfway: only the insertion into the hash table is done in lower case. But the fact that it is only halfway can cause various problems, including unexpected channel resets (see #31 (closed)).

Since there seems to be no good reason to enforce case insensitivity for channel names either, we simply remove this lowercase insertion into the hash table.

Fixes: #31 (closed)
Related: !27 (closed), 2ab4e553

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: case-sensitive-backend