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

[FIX] Paint Data: in-place output modification #1959

Merged
merged 3 commits into from
Jan 26, 2017

Conversation

ales-erjavec
Copy link
Contributor

Issue

The widget outputs a table created from a view of its data which should have been used only internally to the widget as it is modified in-place.

Description of changes

Ensure the output data is not a view of the working data.

Includes
  • Code changes
  • Tests
  • Documentation

The widget used to output a table created from a view of its
buffer which should have been used only internally to the widget
as it can be modified in-place.

Fixes biolabgh-1926
Fixes a 'QUndoStack::endMacro(): no matching beginMacro()' warning
from Qt and creates a single Undo command
@codecov-io
Copy link

codecov-io commented Jan 26, 2017

Current coverage is 89.50% (diff: 100%)

Merging #1959 into master will not change coverage

@@             master      #1959   diff @@
==========================================
  Files            89         89          
  Lines          9166       9166          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           8204       8204          
  Misses          962        962          
  Partials          0          0          

Sunburst

Powered by Codecov. Last update 4c29c3b...4faf8fa

@astaric astaric added this to the 3.3.11 milestone Jan 26, 2017
@lanzagar lanzagar merged commit 2ef581e into biolab:master Jan 26, 2017
@ales-erjavec ales-erjavec deleted the fixed/paint-data-inplace-mod branch January 27, 2017 10:34
astaric pushed a commit that referenced this pull request Feb 3, 2017
[FIX] Paint Data: in-place output modification
(cherry picked from commit 2ef581e)
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 this pull request may close these issues.

4 participants