0004107: don't make BA default to classicgroundmovetype

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0004107

Spring engine

General

public

2013-11-03 11:48

2013-11-03 12:48

Reporter

silentwings

Assigned To

Kloot

Priority

low

Severity

trivial

Reproducibility

N/A

Status

resolved

Resolution

fixed

Product Version

94.1.1+git

Target Version

Fixed in Version

94.1.1+git

Summary

0004107: don't make BA default to classicgroundmovetype

Description

i don't know the history in detail, but there is no need for this anymore

ba hasn't used classicgroundmovetype since ages and won't use it in future

Additional Information

[10:31:01] <[LCC]jK> cgm was just added to prove the fail of it vs. the new one
[10:31:20] <[LCC]jK> that's also the reason why it had a BA detection and auto enables for it

i suggest removing classicgroundmovetype entirely

afaik its only still there for some old reason to do with ba & when i just tested it always crashes anyway (with a math error) in latest spring

i know no one will fix it and i don't care that no one will fix and i did read the comment at top of the file but just for posterity...

C:\\Spring94.1.1-1440\\spring.exe 0x00767C83 rts/Sim/MoveTypes/MoveMath/MoveMath.cpp 176
C:\\Spring94.1.1-1440\\spring.exe 0x007587F7 rts/Sim/MoveTypes/ClassicGroundMoveType.cpp 1073
C:\\Spring94.1.1-1440\\spring.exe 0x007591C8 rts/Sim/MoveTypes/ClassicGroundMoveType.cpp 242

Tags

No tags attached.

Checked infolog.txt for Errors

Attached Files

Relationships

Relationships

Notes
There are no notes attached to this issue.

Notes

Date Modified	Username	Field	Change
Issue History
2013-11-03 11:48	silentwings	New Issue
2013-11-03 12:48	Kloot	Status	new => resolved
2013-11-03 12:48	Kloot	Fixed in Version	=> 94.1.1+git
2013-11-03 12:48	Kloot	Resolution	open => fixed
2013-11-03 12:48	Kloot	Assigned To	=> Kloot

Issue History