fix some integer conversions in the Table class #60

Merged

cremno merged 2 commits from mri-table-fix-conversions into master

2014-08-31 09:20:09 +00:00

cremno commented

(Migrated from github.com)

Sizes (x, y, z) are clamped to a non-negative int value.
Use NUM2INT instead of FIX2INT as the argument might not be a Fixnum.

Sizes (x, y, z) are clamped to a non-negative int value. Use NUM2INT instead of FIX2INT as the argument might not be a Fixnum.

Ancurio commented

(Migrated from github.com)

Small nitpick: Can you use std::max instead of the ternary op?

Small nitpick: Can you use `std::max` instead of the ternary op?

cremno commented

(Migrated from github.com)

I've changed it.

Ancurio commented

(Migrated from github.com)

Thank you!~

No reviewers

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MapleShrine/mkxp#60

No description provided.