Skip to content
Snippets Groups Projects

Fix saving/loading of Coordinates setting

Closed zoner requested to merge zoner/grafX2:fix-coords_rel-setting into master

Saving ini file messes up the Coordinates setting. This patch fixes this.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • I don't understand why this would change something...

  • Coords_rel is a boolean it must be 0 for false, anything else for true. If it is either 1 or 2, it is always true...

  • Author Contributor

    It's that "2-Values[0]" part that is messing things up. I don't recall what compelled me to change the 0 and 1 to 1 and 2 (There's a real chance i tried to fix this from both sides independently, reading and writing the .ini, and maybe my .ini file had already been invalid when i tested my fix). I'll revisit this now.

  • Author Contributor

    I figured out a cleaner fix; I'll close this merge request and submit a new one.

  • closed

  • can you explain what needs to be fixed ?

  • Author Contributor

    The new fix got merged earlier today (and my description for it is moderately verbose): link

    Edited by zoner
Please register or sign in to reply
Loading