View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002384 | Scribus | Internal | public | 2005-08-03 16:38 | 2006-07-03 20:17 |
Reporter | Assigned To | avox | |||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Platform | x86 Linux | OS | Fedora Core | OS Version | 3 |
Fixed in Version | 1.3.4cvs | ||||
Summary | 0002384: PageItem::DrawObj does lots of non-drawing tasks, gets very convoluted | ||||
Description | PageItem::DrawObj is large and difficult to follow. It's very deeply nested in places, lacks documentation on what it's doing (some comments on branches and loops would help a *LOT*), and appears to do rather more than redraw the object. In particular, it seems to handle text reflow. That's a performance issue as well as a code readability one. This has been known for ages, but I'm filing it here "for posterity", so someone gets the pleasure of closing it, and so people hunting for interesting challenges will stumble across it. | ||||
Tags | No tags attached. | ||||
Patch | |||||
child of | 0002382 | acknowledged | Metabug: Code quality, structure, readability |
Date Modified | Username | Field | Change |
---|---|---|---|
2005-08-03 16:38 |
|
New Issue | |
2005-08-03 16:38 |
|
Status | new => confirmed |
2005-08-03 16:38 |
|
Summary | PageItem::DrawObj does lots of non-drawing tasks => PageItem::DrawObj does lots of non-drawing tasks, gets very convoluted |
2005-08-03 16:38 |
|
Relationship added | related to 0002382 |
2005-08-03 16:41 |
|
Relationship replaced | child of 0002382 |
2006-07-02 22:25 | avox | Status | confirmed => resolved |
2006-07-02 22:25 | avox | Fixed in Version | => 1.3.4cvs |
2006-07-02 22:25 | avox | Resolution | open => fixed |
2006-07-02 22:25 | avox | Assigned To | => avox |
2006-07-03 20:17 | cbradney | Status | resolved => closed |