Opened 6 years ago

Closed 6 years ago

## #10683 closed defect (fixed)

# MOHAWK: MYST: Assertion failure due to recent cos/sin table changes

Reported by: | lotharsm | Owned by: | dafioram |
---|---|---|---|

Priority: | high | Component: | Engine: Mohawk |

Version: | Keywords: | ||

Cc: | dafioram | Game: | Myst |

## Description

The recent cos/sin table changes introduced a regression that leads to an assertion failure in MYST when trying to start a new game. The assertion failure happens right after the start of the game, where the intro movie is supposed to be played.

git bisect results:

commit 9cfc70e7fe881cfbd680322b6beeb9a7ab7149c3 Author: David Fioramonti <dafioram@gmail.com> Date: Thu Aug 16 06:37:10 2018 -0700 COMMON: allow cos/sin number of points to be more flexible Previously, the cos/sin table had to be a power of 2, but there are many use cases where the number of points is not necessarily a power of 2 so this change the constructor so it now takes in the number of points rather than the number of points as the exponent of a power of 2. The restriction is that the size must be divisible by 4. :040000 040000 73b87a157764d21e9d3c1a2fad1ddea4b3217862 0e97da067fd291113e353ccf4279da97bc89d680 M common

### Change History (2)

### comment:1 by , 6 years ago

### comment:2 by , 6 years ago

Owner: | set to |
---|---|

Resolution: | → fixed |

Status: | new → closed |

Thanks for your report. This is fixed in cf99bb0.

**Note:**See TracTickets for help on using tickets.

PR1315.