[Bug 421] Implement a DIB engine

wine-bugs at winehq.org wine-bugs at winehq.org
Sat Jan 24 17:13:54 CST 2009


http://bugs.winehq.org/show_bug.cgi?id=421





--- Comment #101 from max at veneto.com  2009-01-24 17:13:47 ---
(In reply to comment #100)
> Hi Max!  Thanks for posting that.  
> You've preserved Jesse and Huw's copyright notices, which is great,
> but it'd be nice if you also credited Jesse and Huw whenever you post this.
> Can you give a rough percentage of who wrote what
> (e.g. is it 20% Jesse, 50% Huw, and 30% you)?
> 

Eh, the percentage.... quite difficult.
I started with Jesse's engine, then I added the complete Huw engine, changing
about all off Jesse's one besides parts on blitting (which were missing in
Huw's, but patched by me) and the big amount of stubs (quite comfortable to
have for testing, BTW), then I changed code again, moving driver loading from
SelectBitmap to CreateDC, CreateCompatibleDC and DeleteDC, added registry and
environment entries for enabling driver, corrected some stuffs which made it
crash if driver is not present/disabled, added the missing ExtTextOut() stuffs,
displaced color table grabbing hack to RealizeDefaultPalette() which is called
earlier.... Who knows ? :-)

If you look to code lines, I'd say 33 33 33, if looking on implemented stuffs
I'd say 25 30 45.... If looking on correctness, it's really a mystery :-)

BTW, I think the most important would be having it working :-)
I'm somehow in touch with Jesse, which (AFAIK) is building a GIT repository
with the new engine, even if I think the best would be to have it in main tree,
if possible.
Now that it is disabled by default it should do no harm at all to existing
applications (GDI with disabled dib engine behaves exactly as before), so it
should make no regressions at all.

I'd like to know Alexandre's opinion about it.

Ciao

Max


-- 
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
Do not reply to this email, post in Bugzilla using the
above URL to reply.
------- You are receiving this mail because: -------
You are watching all bug changes.



More information about the wine-bugs mailing list