Named accessor ops

[littledivy]

Implement get/set accessors for object wrap ops.

#[op2]
impl DOMPoint {
  #[getter]
  fn x(&self) -> f64 {
    self.x
  }
  
  #[setter]
  fn x(&self, x: f64) {}
}

point.x;
point.x = 10; 

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 89.91597% with 12 lines in your changes missing coverage. Please review.

Project coverage is 81.53%. Comparing base (0c7f83e) to head (1c2d9c8).
Report is 164 commits behind head on main.

Files with missing lines	Patch %	Lines
core/runtime/bindings.rs	89.47%	8 Missing ⚠️
core/extensions.rs	60.00%	2 Missing ⚠️
ops/op2/dispatch_slow.rs	88.88%	1 Missing ⚠️
testing/checkin/runner/ops.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #965      +/-   ##
==========================================
+ Coverage   81.43%   81.53%   +0.09%     
==========================================
  Files          97       98       +1     
  Lines       23877    25361    +1484     
==========================================
+ Hits        19445    20678    +1233     
- Misses       4432     4683     +251     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bartlomieju]

Is there a particular reason for this? Why don't getters require to start with __get_?

[littledivy]

This is to allow for getter and setters to be defined with the same name. Only one needs to be changed to avoid name collision

#[getter]
fn x() {}

#[setter]
fn x() {} // op2 automatically converts this to __set_x to avoid name collision

[bartlomieju]

Very nice, LGTM!