Skip to content
This repository has been archived by the owner on Sep 19, 2018. It is now read-only.

Issue #177: Update assert_readonly implementation to use ES5 features #178

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Issue #177: Update assert_readonly implementation to use ES5 features #178

wants to merge 2 commits into from

Conversation

DLehenbauer
Copy link
Contributor

@DLehenbauer DLehenbauer commented Jan 22, 2016

Determines if the property is readonly by inspecting it's descriptor and verifying that attempts to set throw under strict mode.


This change is Reviewable

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/6136

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@gsnedders
Copy link
Member

Because mentions in the title aren't cross-referenced (thanks GitHub!): #177.

zqzhang pushed a commit to zqzhang/realsense-extensions-crosswalk that referenced this pull request Apr 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants