Ares

Sight > 10 has no extra effect, or crashes

Bug #895418 reported by WoRmINaToR
22
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Ares
Fix Released
Low
AlexB
Ares 0.6

Bug Description

This has been mentioned in other small discussions throughout the tracker, but there was never an official issue dedicated to fixing this...

If you set a unit with Sight > 10 in rulesmd.ini, the unit will get the 10 cells of vision radius, however it will not get any extra vision after that... so Sight=14 pretty much is the same as Sight=10.

Also, as you are all aware, having a multiplier take Sight= of a unit above 10, "is a hardcode vegas crash," as described by the programmers in rulesmd.ini (this just drives home how lazy they were... they knew it was a problem and they ignored it).

Revision history for this message
Professor_Tesla (professor-tesla) wrote :

H'm. Weird. I tested a Sight= value above 10 and it didn't crash; it just didn't provide any extra sight.

Revision history for this message
DCoder DCoder (dcoder1337) wrote :

There are plenty of different places where sight/cellspread are used, some of them do range checking, some do not.

Revision history for this message
WoRmINaToR (worminator) wrote :

-"H'm. Weird. I tested a Sight= value above 10 and it didn't crash; it just didn't provide any extra sight."

I dunno if you realized this, but that's exactly what I said.

Revision history for this message
MT1337 (mt1337) wrote :

This would be nice but not overly useful as some major things imo. So in favour but not priority.

Revision history for this message
reaperrr (reaperrr) wrote :

Just for clarification since this is filed as bug, do you only intend to fix the crash, or do you intend to increase the maximum sight, too?

Revision history for this message
DCoder DCoder (dcoder1337) wrote :

The latter.

Revision history for this message
reaperrr (reaperrr) wrote :

Alright, since I can't vote for it on the ICS, I'll formally declare "strong support" then, my mod will have several weapons with ranges far above 10, so increased max. sight would really help me, so I hope it won't be scheduled for a too far away release when the roadmap receives its major schedule update.

Renegade (renegade)
tags: added: happens-in-stock-game
removed: 0
Changed in ares:
status: New → Confirmed
importance: Medium → Low
AlexB (alexander-b)
Changed in ares:
assignee: nobody → AlexB (alexander-b)
milestone: none → 0.6
status: Confirmed → Won't Fix
status: Won't Fix → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.