As in ignore process[env] and ignore process['env']. We should ignore process[env] because that's just wrong (this bug would not manifest itself if there aren't any process.env because it wouldn't match the fast-path regexp test in custom.js). Also, we should ignore process['env'], although correct, it's just too weird to support.
yoshuawuyts
commented
9 years ago
As in ignore
process[env]and ignoreprocess['env']. We should ignoreprocess[env]because that's just wrong (this bug would not manifest itself if there aren't anyprocess.envbecause it wouldn't match the fast-path regexp test incustom.js). Also, we should ignoreprocess['env'], although correct, it's just too weird to support.cc: @yoshuawuyts @hughsk