You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by nielsbasjes <gi...@git.apache.org> on 2017/12/14 10:55:19 UTC

[GitHub] avro pull request #266: Avro 2117 code cleanup

GitHub user nielsbasjes opened a pull request:

    https://github.com/apache/avro/pull/266

    Avro 2117 code cleanup

    This is a massive set of changes to reduce the number of needless warnings in the (mostly Java) code base.
    I have kept each type of change in a separate commit so we can cherry-pick if one or more are unwanted.
    
    Please review.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/nielsbasjes/avro AVRO-2117-CodeCleanup

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/avro/pull/266.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #266
    
----
commit 6a2adc9403c8c70332d7fd988d203232be165792
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T13:56:00Z

    AVRO-2117: Cleanup Java - Fix spelling errors

commit 8f9ff6cdd39019621912ebba085227a86f870660
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T14:24:49Z

    AVRO-2117: Cleanup Java - Remove trailing spaces

commit 40df4ebee2d98a3f83276a5b22ce84b46d2c1843
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T14:25:01Z

    AVRO-2117: Cleanup C# - Remove trailing spaces

commit 723eb369485d3377b9da32a448fd0828fae47289
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T14:48:24Z

    AVRO-2117: Cleanup Java - Explicit types replaced with <>

commit 501e6f21ec2db7a975ed9ca243a2253d72d5085a
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T14:54:13Z

    AVRO-2117: Cleanup Java - Copyright block is reported as 'Dangling Javadoc'

commit 3d8791143dd4a0d2fa2a0ddd17138cc2711f01b5
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T15:01:18Z

    AVRO-2117: Cleanup Java - Remove unused imports

commit 2204c502997e836f1cea065ebe373e98197a200b
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-13T16:03:34Z

    AVRO-2117: Cleanup Java - Cleanup tests

commit fd43dafe106b1858b70973770fa42acba905d670
Author: Niels Basjes <nb...@bol.com>
Date:   2017-12-14T09:56:21Z

    AVRO-2117: Cleanup Java - Remove redundant type arguments

----


---

[GitHub] avro pull request #266: AVRO-2117: Code cleanup

Posted by nielsbasjes <gi...@git.apache.org>.
Github user nielsbasjes closed the pull request at:

    https://github.com/apache/avro/pull/266


---