joewalker/gcli — comment on commit 61c3f58bb70052f52ed59ebd6b857fc8bcbb9574
File: lib/gcli/types/javascript.js line 222
By joewalker:

Just for reference, these notes are markdown, so:

matches.forEach(function(m) {
m.sort = ((m1.ownProperty ? 1 : 0) << 1) + (isVendorPrefixed(m) ? 1 :0);
});

matches.sort(function(m1, m2) {
if (m1.sort !== m2.sort) {
return m2.sort - m1.sort;
}
return m1.prop.localeCompare(m2.prop);
});

However since ownProperty is now a real integer, we've got an obvious bug.
I didn't write the nested ternary to start with, and wrote it out as a nested if. I think this way is slightly more readable. But I'll admit its in the eye of the beholder.
I think I'm leaning to leaving it as it is here.