Arturo Bernal

Results 48 comments of Arturo Bernal

> The utility/class method focus should be more about synchronizing than locking (hence the names). > > Hi @arturobernalg ! > > > However, this approach has limitations when it...

HI @juanpablo-santos I fix all the comment. Please do another pass. TY

> Hi @arturobernalg ! > > went ahead and pushed remaining requested changes, however I've got tests errors on `XMLUserDatabaseTest`. They aren't happening on the master branch, and were happening...

> @arturobernalg > Hi, > This component is now on Java 8. Hey @garydgregory I think its already done. TY

> @arturobernalg > Do you even run local builds before pushing out a PR? > Some of these tests will obviously fail, for example, see Exec65Test where the OS checks...

> > > @arturobernalg > > > Do you even run local builds before pushing out a PR? > > > Some of these tests will obviously fail, for example,...

Hi @juanpablo-santos, I've made changes to avoid reliance on default platform encoding, I think this would be a nice addition to the 2.12.2 release. Please take a look. Best, Arturo