Hi,

> FYI : I've committed based mostly on Jacek's input.

Sorry guys, I was away for a week or so. I had no way to check your
patches (Danny's and Pedro's). I've looked at them now. They both seem to
be looking fine (I have no possibility to test them in a compiled code).
Let me, however, say one thing. With your patches, the structure becomes
really awful. You have to unscramble it in order to see what's inside.
In my solution, one could easily see how the structure is defined in
WinCE. It's a matter of taste, of course (I prefer simplicity, even if it
costs some additional lines in the source code -> it is easier to read
and maintain).

There is just one thing missing in your patches, that is present in the
MS documents that I've given you some time ago: *NPDEVMODEW
-------------------------------------------------
typedef struct _devicemodeW {
...
} DEVMODEW, *PDEVMODEW, *NPDEVMODEW, *LPDEVMODEW;
-------------------------------------------------

Best regards,
Jacek.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Cegcc-devel mailing list
Cegcc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cegcc-devel

Reply via email to