Skip to content

Conversation

@gefarion
Copy link
Collaborator

Me pareció mucho más simple duplicar los handlers, de esta forma no es más necesario tener el método readOnlyEnvInObj y tampoco tener benchs diferentes.
De esta forma solo hace falta cambiar el path al momento de correr los benchs.

Si te parece bien aplico la misma idea a los viejos benchs de readonly.

@gefarion gefarion requested a review from charig March 24, 2020 01:30
Copy link
Owner

@charig charig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No me queda claro porque necesitás duplicar la verdad, pero si te simplifica la corrida en RTruffleMate ok.

@gefarion gefarion changed the title Add MO In Object handlers Separate ReadOnly EnvInObj handlers Mar 24, 2020
@gefarion
Copy link
Collaborator Author

No me queda claro porque necesitás duplicar la verdad, pero si te simplifica la corrida en RTruffleMate ok.

Sino lo separo hay que cambiar todos los benchs para que en lugar de usar readOnly llamen al método readOnlyEnvInObj. Se simplifica mucho la corrida separando los handlers y cargando la version que se quiera correr.

@charig
Copy link
Owner

charig commented Mar 24, 2020

No me queda claro porque necesitás duplicar la verdad, pero si te simplifica la corrida en RTruffleMate ok.

Sino lo separo hay que cambiar todos los benchs para que en lugar de usar readOnly llamen al método readOnlyEnvInObj. Se simplifica mucho la corrida separando los handlers y cargando la version que se quiera correr.

Ah ok. Manana reviso cómo lo estoy haciendo en trufflemate y vemos cómo mergiar.

Copy link
Owner

@charig charig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 cosas:

  1. No habría que cambiar el readonly de object.som también (o sea, borrar el readonlyEnvInObject).
  2. Las clases que están duplicadas 100% creo se puede mejorar. Ponerlas en un path Immutabillity/Base que sea común y después solo Handle en dos dir separados no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants