CCSM->Put plugin: the "Put to arbitrary viewport" bindings aren't working

Bug #1160710 reported by Micheal Hsu
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Compiz
Fix Committed
Undecided
Unassigned
compiz (Ubuntu)
New
Undecided
Unassigned

Bug Description

CCSM--> Put plugin:
set key bindings in "Put to arbitrary viewport" section.

but the key bindings only work once.

this is caused a programming error in put.cpp:

bool PutScreen::toViewport (....)

the function resized the CompOption every time it's called.(this can cause memory leak).
subsequent calls to getIntOptionNamed(..) always return the first "viewport" value instead of of the correct value, which is appended to the tail of the vector.

the correct way is to find the "viewport" option and then set its value.

Related branches

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:compiz at revision None, scheduled for release in compiz, milestone 0.9.9.2

Changed in compiz:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.