There's no good reason (IMHO) they shouldn't also accept kIntptrType atoms, since they aggressively use those as keys where possible.
Created attachment 447832 [details] [diff] [review] Patch Obvious fix, also consolidates duplicate code. (Also, drive-by change to getAtomPropertyFromProtoChain, which was unnecessarily virtual)
any test results?
Nope -- I should probably withdraw this as it's non-urgent, but it's one of those weird things I wanted to note down somewhere...
Comment on attachment 447832 [details] [diff] [review] Patch Nothing obviously wrong with the patch, but cancelling review until testing is further along.
Assignee: nobody → stejohns
Priority: -- → P4
Target Milestone: --- → flash10.2
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → WONTFIX
bulk verifying resolved !fixed issues
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.