Fixed some descrepancies in sector information for newly connected clients

• Rapporteur: akmdm
• Propriétaire: (Aucun)
• Type: Patches
• État: Atteints
• Composant: (Aucun)
• Jalon: (Aucun)
• Priorité: 5 - moyen
• Sévérité: 5 - moyen
• Résolution: Accepted
• Fichier: 2

Détails Répondre

With this patch, the server now only sends information about a sector to newly connected clients if their current values are different from the saved values, not when they're different from the default values (e.g. 0.0 for texture offsets, or 1.0 for texture scale). The fixes the floor alignment issues reported in: https://zandronum.com/tracker/view.php?id=3582

Ticket History (3/6 Histories)

2021-01-04 05:01 Updated by: akmdm

• New Ticket "Fixed some descrepancies in sector information for newly connected clients" created

2021-01-04 05:01 Updated by: akmdm

• File 10240.patch (File ID: 5793) is attached

2021-01-04 06:09 Updated by: torr_samaho

Commentaire Répondre

Thanks for the patch! Looks very good, I just noticed a related problem due to your static_cast<signed> casts: I think instead of casting the results of GetAngle, we should change SavedCeilingAngle and SavedFloorAngle in sector_t to angle_t.

2021-01-11 04:30 Updated by: akmdm

• File 10240_r2.patch (File ID: 5805) is attached

2021-01-11 04:32 Updated by: akmdm

Commentaire Répondre

I made revisions to the previous patch. The data types of SavedCeilingAngle and SavedFloorAngle have been changed to angle_t, so now those static_cast<signed> calls are no longer needed.

2021-01-11 05:36 Updated by: torr_samaho

• Résolution Update from Aucun to Accepted
• État Update from Ouvert to Atteints

Commentaire Répondre

Thanks! I added your updated patch.