Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SIGSEGV with buggy FM envelope input editor, value mapping display misalignment #87

Closed
OPNA2608 opened this issue Mar 26, 2019 · 3 comments · Fixed by #95
Closed

SIGSEGV with buggy FM envelope input editor, value mapping display misalignment #87

OPNA2608 opened this issue Mar 26, 2019 · 3 comments · Fixed by #95

Comments

@OPNA2608
Copy link
Member

I've noticed two bugs within the Configuration->Input tab:


1) Name -> Specification association

While renaming a new or any of the existing FM envelope format specifications, if the currently entered name is equal to an existing specification's name, the currently edited specification as well as the one it shared its name with for any amount of time will be associated with each other: They share the same editable list of text -> FM operator mappings and deleting one of them will also delete the other.

Example: I want to add an alternate specification of the PMD format as described here and call it PMD-DT2. (because it adds a secondary Detune value to each operator specification)

  1. Have a FM envelope text specification called "PMD".
  2. Create a new FM envelope text specification.
  3. Edit the name from "Set …" to "PMD-DT2".

While typing, as soon as the textbox contains the string "PMD", PMD-DT2 will be linked to PMD.

  • If you try to edit PMD-DT2, it'll have the same content as PMD. A SIGSEGV is achievable here by changing a value, then pressing OK and selecting PMD. (valgrind trace below)
  • If you save, PMD-DT2 will replace PMD.
  • If you delete PMD-DT2, PMD will be deleted as well. (and vice versa)

Here is the valgrind log. It has alot of noise at the start, the SIGSEGV happens at line 1064.

2) Bugged mapping display

When adding a value while value number 0 or no value (when pressing Edit followed by Add without highlighting any existing value) is selected, the values will be misaligned. This is only a temporary visual bug, as closing the window and pressing Edit again will fix this.

grafik

@rerrahkr
Copy link
Member

Thanks for reports. I will fix them.

@rerrahkr
Copy link
Member

I fix to allow the same name envelope sets at f9018c8, and value type alignment at 4b6e56b.

@OPNA2608
Copy link
Member Author

OPNA2608 commented Apr 5, 2019

I can confirm that same name envelope sets work fine now, however the value type alignment, with enough additions, still bugs out.
grafik

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

Successfully merging a pull request may close this issue.

2 participants