Right now GetProperty uses an optional boolean parameter to indicate whether the property was present. It would probably be more ergonomic to have it return Maybe<T>, at least for "small" values that are stored directly in the slot of the property table. (Maybe even worth doing for other values as well, which would then effectively return Maybe<T*> or something along those lines?) (Thanks to emilio for the suggestion in https://phabricator.services.mozilla.com/D126031#inline-693151 )
Bug 1731509 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Right now GetProperty uses an optional boolean parameter to indicate whether the property was present. It would probably be more ergonomic to have it return Maybe<T>, at least for "small" values that are stored directly in the slot of the property table. (Maybe even worth doing for other externally-allocated properties as well, which would then effectively return Maybe<T*> or something along those lines?) (Thanks to emilio for the suggestion in https://phabricator.services.mozilla.com/D126031#inline-693151 )