Skip to content

Fixes spanish typos and translation errors #443

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 29 commits into from
Aug 24, 2015
Merged

Fixes spanish typos and translation errors #443

merged 29 commits into from
Aug 24, 2015

Conversation

fagossa
Copy link
Contributor

@fagossa fagossa commented Aug 3, 2015

No description provided.

@fagossa fagossa changed the title Fixes spanish typos and translation erros Fixes spanish typos and translation errors Aug 3, 2015
@SethTisue
Copy link
Member

@heathermiller @dickwall do we know any Spanish speakers who could review this?

@SethTisue
Copy link
Member

I mentioned this on scala/scala/scala-user on gitter, hoping to attract a reviewer or two.


Más problemático es que los valores usados para ser por defecto están tanto en la documentación (Javadoc) y en el código. Mantener esto actualizado es fácil de olvidar. Un típico patrón utilizado para no cometer estos errores es agregar constantes públicas cuyo valor será mostrado en el Javadoc:
Más problemático es que los valores usados para ser por defecto están tanto en la documentación (Javadoc) como en el código. Mantener ambos actualizado es dificil. Un patrón típico utilizado para no cometer estos errores es agregar constantes públicas cuyo valor será mostrado en el Javadoc:

Choose a reason for hiding this comment

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

"Mantener ambos actualizado" must be changed to "Mantener ambos actualizados"

Choose a reason for hiding this comment

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

dificil must be changed to difícil

@dickwall
Copy link
Contributor

Thanks Miguel, how about we give this a week to try and attract a reviewer, and then merge it in either way. I do have someone in mind who might be able to give it a once over for correctness.

@@ -30,7 +30,7 @@ Ahora suponga que queremos escribir una función `cloneAndReset` la cual recibe
cloned
}

La pregunta que surge es cuál es el tipo del parámetro `obj`. Si este fuera `Cloneable` entonces el objeto puede ser clonado mediante el método `clone`, pero no puede usarse el método `reset`; Si fuera `Resetable` podríamos resetearlo mediante el método `reset`, pero no sería posible clonarlo. Para evitar casteos (refundiciones) de tipos en situaciones como la descripta, podemos especificar que el tipo del objeto `obj` sea tanto `Clonable` como `Resetable`. En tal caso estaríamos creando un tipo compuesto; de la siguiente manera:
La pregunta que surge es cuál es el tipo del parámetro `obj`. Si este fuera `Cloneable` entonces el objeto puede ser clonado mediante el método `clone`, pero no puede usarse el método `reset`; Si fuera `Resetable` podríamos resetearlo mediante el método `reset`, pero no sería posible clonarlo. Para evitar casteos (refundiciones, en inglés `casting`) de tipos en situaciones como la descrita, podemos especificar que el tipo del objeto `obj` sea tanto `Clonable` como `Resetable`. En tal caso estaríamos creando un tipo compuesto; de la siguiente manera:
Copy link

Choose a reason for hiding this comment

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

What about restablecerlo rather than resetearlo?

@sattvik
Copy link

sattvik commented Aug 24, 2015

The patch with all of Miguel's suggestions LGTM.

@heathermiller
Copy link
Member

Cool, thanks @sattvik!

heathermiller added a commit that referenced this pull request Aug 24, 2015
Fixes spanish typos and translation errors
@heathermiller heathermiller merged commit 64c8efb into scala:master Aug 24, 2015
@heathermiller
Copy link
Member

I'll let @fagossa or anyone else willing to take @miguel-vila's & @sattvik's feedback and make a new PR out of it. (Anyone can fix these typos btw! Doesn't have to be only @fagossa)

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.

7 participants