Zandronum Chat @ irc.zandronum.com
#zandronum
Get the latest version: 3.0
Source Code

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002875Zandronum[All Projects] Bugpublic2016-10-09 22:452017-11-12 14:37
Reporterunknownna 
Assigned ToLeonard 
PrioritynormalSeveritytweakReproducibilityalways
Statusneeds testingResolutionopen 
PlatformOSOS Version
Product Version2.1 
Target Version3.1Fixed in Version 
Summary0002875: "smooth_mouse 1" potentially hard-coded into Zandronum's mouse behavior
Description
Quote from unknownna
I recall that Torr told me that smooth_mouse does nothing in Skulltag/Zandronum [...]

Quote from Source comment
smooth_mouse doesn't do anything in Skulltag (Carn deactivated it long ago) [...]

Quote from unknownna
I noticed one difference between ZDoom and Zandronum with cl_capfps 0: The view is jittery in ZDoom and smooth in Zandronum when aiming and moving around at the same time. In ZDoom, you have to use "smooth_mouse 1" to make the view as smooth as Zandronum. Could it be that Carnevil hard-coded "smooth_mouse 1" into Skulltag's default mouse behavior and thus disabled smooth_mouse altogether?
Attached Files

- Relationships

-  Notes
User avatar (0015871)
Edward-san (developer)
2016-10-09 22:54

To be more precise, the deactivated code is in r_utility.cpp line 591, marked with:


// [BC] This makes the mouse incredibly jerky for client games.


LocalKeyboardTurner saves the value of smooth_mouse when needed.
User avatar (0015913)
Ru5tK1ng (updater)
2016-10-12 00:52

So should the menu option continue to toggle m_filter? I don't see anyone actively trying to use no smooth mouse if it causes jitter issues.
User avatar (0015917)
Edward-san (developer)
2016-10-12 08:55

For now yes. This ticket can be kept open in case somebody wants to fix the discrepancy :P
User avatar (0018619)
Leonard (developer)
2017-10-23 14:03

PR.
User avatar (0018626)
Torr Samaho (administrator)
2017-10-26 19:26

I added your pull request.
User avatar (0018819)
Leonard (developer)
2017-11-08 18:09

Just noticed there was a missing check for zandronum demos.
Fix PR here.
User avatar (0018861)
Leonard (developer)
2017-11-12 14:37

The fix was merged too.

Issue Community Support
Only registered users can voice their support. Click here to register, or here to log in.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
2016-10-09 22:45 unknownna New Issue
2016-10-09 22:45 unknownna Severity trivial => feature
2016-10-09 22:46 unknownna Description Updated View Revisions
2016-10-09 22:54 Edward-san Note Added: 0015871
2016-10-12 00:52 Ru5tK1ng Note Added: 0015913
2016-10-12 08:55 Edward-san Note Added: 0015917
2017-10-22 22:15 Leonard Assigned To => Leonard
2017-10-22 22:15 Leonard Priority low => normal
2017-10-22 22:15 Leonard Severity feature => tweak
2017-10-22 22:15 Leonard Status new => assigned
2017-10-22 22:15 Leonard Target Version => 3.1
2017-10-23 14:03 Leonard Note Added: 0018619
2017-10-23 14:03 Leonard Status assigned => needs review
2017-10-26 19:26 Torr Samaho Note Added: 0018626
2017-10-26 19:26 Torr Samaho Status needs review => needs testing
2017-11-08 18:09 Leonard Note Added: 0018819
2017-11-08 18:09 Leonard Status needs testing => needs review
2017-11-12 14:37 Leonard Note Added: 0018861
2017-11-12 14:37 Leonard Status needs review => needs testing






Questions or other issues? Contact Us.

Links


Copyright © 2000 - 2017 MantisBT Team
Powered by Mantis Bugtracker