On Thu, Jan 18, 2024 at 3:22 AM Thomas Huth <th.huth@xxxxxxxxx
<mailto:th.huth@xxxxxxxxx>> wrote:
> On 17.1.2024 3.02, Brad Smith wrote:
>> The NEO format seemed to have a header that specifies dimensions
of the
>> picture, resolution (low/medium/high), and a 16-colour palette,
followed by
>> a 32k dump of screen data. The original NEOChrome program itself
only works
>> with 320x200x4bpp images, however.
>>
>> So, the question becomes what to do with the other screen modes
available
>> for STE/TT/Falcon. My thought was rather than dumping nothing,
or something
>> incomplete, just include the missing information in a benign way.
Dumping something that cannot be read by any known application
(AFAIK) does
not make much sense, does it? So I agree with Nicolas here to omit
the code.
I feel the situation has been drastically misunderstood.
The code that was omitted from the patch integration was not ONLY for
screenshots that could not be represented in the NEO format, it was
primarily written to make relevant old-format compliant screenshots work
for ST Hires, as well as TT and Falcon mode screenshots that use
ST-compatible low/medium/high resolutions. These all could be read by
existing tools. NEOChrome itself cannot read medium or high resolution,
but there are several other tools that can read this kind of NEO file.
Because this code to handle those was omitted, these relevant cases will
produce NEO files with empty screens in the current version. This is a
flaw that should be corrected.
[...]