We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 2676

Summary: xdnd_version check issue in case ClientMessage:
Product: SDL Reporter: Nitz <nitin.j4>
Component: videoAssignee: Sam Lantinga <slouken>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: major    
Priority: P2    
Version: 2.0.3   
Hardware: x86   
OS: Linux   
Attachments: patch
patch

Description Nitz 2014-08-08 10:16:18 UTC 
Created attachment 1805 [details]
patch

In X11_DispatchEvent(_THIS) function:
case ClientMessage:

In that, value of 32 bit data shifted to 24 bits which is assigned to xdnd_version

But xdnd_version has no effect on 
XdndEnter,
XdndPosition,
XdndDrop,
neither of the cases

So I think it would be better if we removed the check of xdnd_version
Comment 1 Sam Lantinga 2014-08-17 22:17:55 UTC 
Should the check be added to the other cases? What's the impact of removing that check?
Comment 2 Nitz 2014-09-05 06:10:58 UTC 
I added xdnd_version check to XdndPosition case also
under DEBUG_XEVENTS macro.
by this we can get the action requested by user.

I analysed further and found out that removing xdnd_version check at XdndDrop case is a bad idea because 
in XConvertSelection API timestamp should be passed if(xdnd_version >= 1)
otherwise CurrentTime should be passed
So xdnd_version check is important at XdndDrop case

I made xdnd_version as a static so that it can store the version in other cases also.

Patch is attached.

Thanks!!!
Comment 3 Nitz 2014-09-05 06:12:27 UTC 
Created attachment 1860 [details]
patch
Comment 4 Nitz 2014-10-27 11:50:23 UTC 
A gentle reminder.

Thanks!!!
Comment 5 Nitz 2014-11-14 03:57:23 UTC 
A gentle reminder
Thanks!!!
Comment 6 Sam Lantinga 2014-11-28 12:43:17 UTC 
Fixed, thanks!
https://hg.libsdl.org/SDL/rev/104f3bcfbf45