Hi Trent, thanks for the patch. Generally cc'ing wine-devel isn't necessary unless you have a specific question.
+// functions for decoding animated cursors
No C++-style comments, please.
+ //DWORD form = *(DWORD*)chunk->ckData;
Dead code should be omitted rather than commented out.
+ANIHEADER *anih = NULL; +LPBYTE *ani_frames = NULL; +DWORD ani_frame_idx = 0;
These should be static (and initializing them to 0/NULL is unnecessary.)
+ switch(chunk->ckID) { + case 0x46464952: // RIFF + {
This indenting style doesn't match the rest of the file.
+ if (anih) {
Nor does this.
+void decodeAnimatedCursor(LPBYTE data, DWORD size, LPBYTE **frames, DWORD *nFrames)
This should be static, as should:
+BOOL isAnimatedCursor(LPBYTE bits)
--Juan