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 <meebey@meebey.net>
History
Updated by David Paleino 5431 days ago
- Status changed from New to Closed
Applied in changeset 4717e91e29c48c02448deb4829b8656b8700b5da.