Anonymous | Login | 03-04-2021 16:17 UTC |
Main | My View | View Issues | Change Log | Docs |
Viewing Issue Simple Details [ Jump to Notes ] | [ View Advanced ] [ Issue History ] [ Print ] | ||||||||||||
ID | Category | Severity | Reproducibility | Date Submitted | Last Update | ||||||||
0003759 | [Croquet] Hedgehog | tweak | always | 06-01-06 22:27 | 12-04-06 21:51 | ||||||||
Reporter | howardstearns | View Status | public | ||||||||||
Assigned To | howardstearns | ||||||||||||
Priority | normal | Resolution | fixed | ||||||||||
Status | resolved | ||||||||||||
Summary | 0003759: Full texture sometimes goes into island | ||||||||||||
Description | There are several ways to create TTextures. The one used by CroquetHarness>>dropForm: puts the whole high-res TForm into the island. (bthumb is false.) As a result, islands with stuff added this way become bloated and take longer to sync. | ||||||||||||
Additional Information |
The atttached changeset fixes this by refactoring TFormManager>>addFileNamed:mipmap:shrinkFit:extention:, and then using the new code in #dropForm:. However, there are two things to be aware of: 1. The TFormManager caches real image files (.jpeg, .gif, .bmp, etc.). It doesn't cache Squeak-native Form bits, presumably because we want to be able to tell what's what by using the operating system preview functionality when clicking on a file in the cache. #dropForm: is given a high-res form, not the original file, so all we can do at that point is guess at what format to write out to the cache. For now, I've just guessed at JPEG, but it's pretty clear how you can change that if you like. [Another approach would involve refactoring the upstream stuff, so that the bits are added to the cache earlier, when we still have the original file.] 2. With only the low-res bits in the image, a visitor needs to not only sync from a server, but also get the high-res bits from someone. These are similar, but not identical: i.e., when you've arranged for one of these to happen, the other doesn't automatically happen too. You have to arrange for both. Without this patch, this may or may not have been obvious in any given application, and it may be more obvious with the patch. |
||||||||||||
Attached Files |
![]() |
||||||||||||
|
![]() |
|
(0008540 - 73 - 73 - 73 - 73 - 73 - 73) howardstearns 12-04-06 21:51 |
This is fixed to my satisfaction in the K... classes (Wisconsin package). |
Mantis 1.0.8[^]
Copyright © 2000 - 2007 Mantis Group
36 total queries executed. 29 unique queries executed. |