search

SollmoStudio / beyond

Beyond: The Scalable Game Server Framework
http://www.beyondframework.com
Apache License 2.0
25 stars 9 forks source link

Support the array of optional value. #184

Closed sgkim126 closed 9 years ago

sgkim126 commented 9 years ago

Currently, we don't put the value for optional, but there is need to save something for the value of array. (e.g. type: 'array', elementType: { type: 'int', optional: true })

We determined to put null for optional value. And this patch also changes the option name to nullable, because optional can be mistaken. But keeping optional option for a while to support backward compatibility.

sgkim126 commented 9 years ago

I fixed @noraesae 's comment on https://github.com/sgkim126/beyond/commit/de1d781ab1242643618c216f77e3719aca2091ec#commitcomment-9146819

And I answerd to the other comment

hatashiro commented 9 years ago

LGTM.