[editorial] Simplify Source Text Module Record's ResolveExport

[nicolo-ribaudo]

[[BindingName]] can either be a String or NAMESPACE (source: https://tc39.es/ecma262/#resolvedbinding-record).

These two checks are basically doing

if (resolution.BindingName === NAMESPACE || starResolution.BindingName === NAMESPACE) {
  if (resolution.BindingName !== starResolution.BindingName) return AMBIGUOUS;
}
if (resolution.BindingName !== NAMESPACE && starResolution.BindingName !== NAMESPACE) {
  if (resolution.BindingName !== starResolution.BindingName) return AMBIGUOUS;
}

Which is just

if (resolution.BindingName !== starResolution.BindingName) return AMBIGUOUS;

Because the two conditions are the complement of each other.

[github-actions]

The rendered spec for this PR is available at https://tc39.es/ecma262/pr/3711.

[michaelficarra]

I love that we're getting rid of that stray "either ... or ... is ..." construction as well.

[bakkot]

Thanks! I believe I introduced this in #2566, although at that time it was not clearly wrong because we were using SameValue, which does not support enum values.