Bug #281
Race condition in PangoTools.ToMarkup
Status: | Closed | Start: | 01/07/2010 | |
Priority: | Immediate | Due date: | 01/07/2010 | |
Assigned to: | David Paleino | % Done: | 100% |
|
Category: | Frontend GNOME | |||
Target version: | 0.7 | |||
Complexity: | Low |
Found in Version: | ||
Votes: | 0 |
Description
The for loop (the one building the string with the Pango Markup), relies on tags.Count.
Inside the loop, there's tags.Pop(), which modified the contents of the stack. This means that the for loop effectively "catches" only the first half+1 of the elements in the Stack.
Associated revisions
Revision 4717e91e29c48c02448deb4829b8656b8700b5da
Fix race condition in PangoTools.ToMarkup (closes: #281)
Also, use List<> instead of Stack<>, since there's no point in having
a Stack anymore.
Signed-off-by: Mirco Bauer <[email protected]>
History
Updated by David Paleino 5446 days ago
- % Done changed from 0 to 100
- 1 set to fixed
Updated by David Paleino 5446 days ago
- Status changed from New to Closed
Applied in changeset 4717e91e29c48c02448deb4829b8656b8700b5da.