[
https://issues.apache.org/jira/browse/CODEC-86?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12740122#action_12740122
]
Gary Gregory commented on CODEC-86:
-----------------------------------
If we do something like this, we should to check all of our APIs for naming
consistency in the case where the return type is included in the method name.
Personally, it seems fine to me the way it is now but if someone wants to
change it, that's fine as long as it is done consistently.
> Rename method to Base64.encodeBase64ToString()
> ----------------------------------------------
>
> Key: CODEC-86
> URL: https://issues.apache.org/jira/browse/CODEC-86
> Project: Commons Codec
> Issue Type: Improvement
> Affects Versions: 1.4
> Reporter: Julius Davies
> Priority: Trivial
> Fix For: 1.4
>
> Attachments: codec86.patch
>
>
> I'm having second thoughts on two method names for CODEC-59. Here's what it
> currently looks like:
> Base64.encodeBase64String(byte[] binaryData)
> Base64.encodeBase64URLSafeString(byte[] binaryData)
> http://juliusdavies.ca/codec/base64String.png
> (screenshot from IntelliJ 7, 15KB, 760x187)
> I'm wondering if the word "To" should be inserted there? As in "ToString":
> Base64.encodeBase64ToString(byte[] binaryData)
> Base64.encodeBase64URLSafeToString(byte[] binaryData)
> http://juliusdavies.ca/codec/base64ToString.png
> (screenshot from IntelliJ 7, 14KB, 736x162)
> Sorry about this inane niggling. I just thought it's either now or
> never! Any one else have opinions about this extremely minor cosmetic
> point?
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
