Mantis Bugtracker
  

Viewing Issue Advanced Details Jump to Notes ] View Simple ] Issue History ] Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0002568 [Squeak] Graphics minor have not tried 01-23-06 23:50 07-22-13 02:23
Reporter wiz View Status public  
Assigned To andreas
Priority normal Resolution no change required Platform
Status closed   OS
Projection none   OS Version
ETA none Fixed in Version Product Version
  Product Build
Summary 0002568: How much should warpblt inherit from BitBlt???
Description As I've been looking into rounding problems see Mantis 0002453. I've realized that there has been a culture of rounding parameters for both source and destination for both bitblt and warpblt.

Now both need integral destination rectangles, and bitblt needs an integral source. Warpblts source does not and is useful when it is not integral.

Yet it inherits routines from Bitblt that imply they should be.

The final straw was when I relized I could send a warpblt instance the method copybits. Now I am new enough to have to guess at this point whether you would ever want to do such a thing so I can't say there is not a legit reason to do so. But I'm betting it really is not -to-be-done.

And written correctly warpblt would have to have a lot of shouldNotBeImplementeds for some BitBlt methods.

Which really means they should probably both inherit from a common ancestor and be separte branches to say MoveBlts or ???.

The bottom line is the way it is now it is confusing to write good code. The looks of squeak suffer from good people doing their best to understand and failing. We should clean up the code and get the benifits of a wider body of competent maintainers.
Steps To Reproduce
Additional Information Severity does not make a distinction between Urgency and Improtance.

This has major long range improtance (and I believe urgency)

It is minor in immediate urgency.

Its immediate improtance is that once some of the rounding problems have been fixed. Keeping them fixed will depend on the clarity of the code.


I also think this is part of another major issue that squeak is long overdue for a Refactoring only pass. This Is of high urgency and importance. Lots has been added to squeak and even some of the code that hasn't rotted totally would rate a high stink score on any oop stink test.

But that of course, is another issue.

Yours in service, -- Jerome Peace

Attached Files

- Relationships

- Notes
(0014405 - 29 - 29 - 29 - 29 - 29 - 29)
tim
07-22-13 02:23

No reason to change this part
 

- Issue History
Date Modified Username Field Change
01-23-06 23:50 wiz New Issue
01-23-06 23:50 wiz Status new => assigned
01-23-06 23:50 wiz Assigned To  => andreas
07-22-13 02:23 tim Status assigned => closed
07-22-13 02:23 tim Note Added: 0014405
07-22-13 02:23 tim Resolution open => no change required


Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
38 total queries executed.
32 unique queries executed.
Powered by Mantis Bugtracker