uqbar-project / wollok-language

Wollok language definition
GNU General Public License v3.0
7 stars 8 forks source link

Eliminar los operadores ++, -- #31

Open fdodino opened 4 years ago

fdodino commented 4 years ago

Los operadores ++, --, -= y += en wollok-ts no están implementados adrede. Teníamos pendiente cargar este issue, y luego definir si no lo queremos más.

npasserini commented 4 years ago

++ y -- definitivamente tienen que volar porque rompen nuestra regla de separar acciones de consultas => tienen efecto y devuelven un valor.

Los operadores de asignación (que son varios más que += y -= si no me equivoco), no tienen ese conflicto. La discusión podría ser que se ve un poco menos que hay un mensaje ahí, pero no creo que sea mucho más problemático que el + o - sin asignación.

En este caso mi voto es dejarlos. Se usan mucho y es mucho más cómodo escribir energia += 10 * gramos que la variante más extensa desugared. Definitivamente dejaría los dos mencionados += y -=, se usan muchísimo.

Los demás operadores de asignación no se usan mucho y podrían ser más o menos supérfluos, pero tampoco veo un daño en dejarlos, mantiene una cierta consistencia que cada operador binario tenga su versión asignatrónica.

El jue., 3 de oct. de 2019 a la(s) 23:38, Fernando Dodino ( notifications@github.com) escribió:

Los operadores ++, --, -= y += en wollok-ts no están implementados adrede. Teníamos pendiente cargar este issue, y luego definir si no lo queremos más.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1829?email_source=notifications&email_token=ABDLKOOWMVXAZZ7R5O3PDHDQM2UDNA5CNFSM4I5KNW62YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HPR5IJQ, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDLKOMXS7BHGQQ5CXYCQUDQM2UDNANCNFSM4I5KNW6Q .

fdodino commented 4 years ago

Ah, perdón, me confundí:

var a
a += 2
a -= 5

se puede hacer tanto en wollok-ts como en wollok-xtext:

image

Entonces eliminar los postfix operators ++ y -- solamente es lo que estamos de acuerdo en volar.

matifreyre commented 4 years ago

Votos++ para volar ++

El vie., 4 de oct. de 2019 07:47, Fernando Dodino notifications@github.com escribió:

Ah, perdón, me confundí:

var a

a += 2

a -= 5

se puede hacer tanto en wollok-ts como en wollok-xtext.

Entonces eliminar los postfix operators ++ y -- solamente es lo que estamos de acuerdo en volar.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1829?email_source=notifications&email_token=ACPYIZXZALWF37GYPASBMDDQM4NLFA5CNFSM4I5KNW62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEALIVOY#issuecomment-538348219, or mute the thread https://github.com/notifications/unsubscribe-auth/ACPYIZTU4WC2OD76MPAS6JLQM4NLFANCNFSM4I5KNW6Q .

npasserini commented 4 years ago

votos += 1 por favor

El vie., 4 de oct. de 2019 a la(s) 08:36, Matías Freyre ( notifications@github.com) escribió:

Votos++ para volar ++

El vie., 4 de oct. de 2019 07:47, Fernando Dodino < notifications@github.com> escribió:

Ah, perdón, me confundí:

var a

a += 2

a -= 5

se puede hacer tanto en wollok-ts como en wollok-xtext.

Entonces eliminar los postfix operators ++ y -- solamente es lo que estamos de acuerdo en volar.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub < https://github.com/uqbar-project/wollok/issues/1829?email_source=notifications&email_token=ACPYIZXZALWF37GYPASBMDDQM4NLFA5CNFSM4I5KNW62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEALIVOY#issuecomment-538348219 , or mute the thread < https://github.com/notifications/unsubscribe-auth/ACPYIZTU4WC2OD76MPAS6JLQM4NLFANCNFSM4I5KNW6Q

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok/issues/1829?email_source=notifications&email_token=ABDLKOLAI2RBAKWBRP5HMD3QM4TEVA5CNFSM4I5KNW62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEALL5BQ#issuecomment-538361478, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDLKOMDUMAVWRCDIHOSJLTQM4TEVANCNFSM4I5KNW6Q .

npasserini commented 4 years ago

Hhabíamos dicho sólo ++ / -- Los de asignación quedan.

El mar., 12 de nov. de 2019 a la(s) 20:48, Fernando Dodino ( notifications@github.com) escribió:

Los operadores ++, --, -= y += en wollok-ts no están implementados adrede. Teníamos pendiente cargar este issue, y luego definir si no lo queremos más.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/uqbar-project/wollok-language/issues/31?email_source=notifications&email_token=ABDLKONPB2GZJNXMGNSHATDQTM6FHA5CNFSM4JMLQOF2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HY3CZJA, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABDLKONBORCUSXNUSZXVZCLQTM6FHANCNFSM4JMLQOFQ .