search

PalladioSimulator / Palladio-Analyzer-Slingshot

0 stars 1 forks source link

Refactor Result #5

Closed stiesssh closed 1 year ago

stiesssh commented 1 year ago

Got some ideas to improving Result while adding the Type arguments from that other (already merged) PR.

it's mostly about:

  1. make all methods of, it's odd to have them all called differently. (marked the former empty and from as depecated, did not delete anything.)
  2. add one of to create Results from Optionals
  3. add upperbound on the Type parameter : this is up for discussion as i played around with the dependencies and relocated an interface. if you do not approve of this, i'll revert it.
klinakuf commented 1 year ago

reasonable changes, they look good to me

kjuli commented 1 year ago

Remove the upper bound type.