Renaming cryptohashes.c to cryptohashfuncs.c

Started by Michael Paquierover 5 years ago3 messageshackers

michael@paquier.xyz

over 5 years ago

Hi all,

Now that the tree has a new set of files for cryptohash functions as
of src/common/cryptohash*.c, it is a bit weird to have another file
called cryptohashes.c for the set of SQL functions.

Any objections to rename that to cryptohashfuncs.c? That would be
much more consistent with the surroundings (cleaning up anythig
related to MD5 is on my TODO list). Patch is attached.

Thoughts?
--
Michael

Daniel Gustafsson

daniel@yesql.se

over 5 years ago

In reply to: Michael Paquier (#1)

Re: Renaming cryptohashes.c to cryptohashfuncs.c

On 3 Dec 2020, at 03:03, Michael Paquier <michael@paquier.xyz> wrote:

Any objections to rename that to cryptohashfuncs.c? That would be
much more consistent with the surroundings (cleaning up anythig
related to MD5 is on my TODO list). Patch is attached.

+1 on this proposed rename.

cheers ./daniel

Michael Paquier

michael@paquier.xyz

over 5 years ago

In reply to: Daniel Gustafsson (#2)

Re: Renaming cryptohashes.c to cryptohashfuncs.c

On Thu, Dec 03, 2020 at 10:10:44PM +0100, Daniel Gustafsson wrote:

+1 on this proposed rename.

Thanks. I have been able to get that done as of bd94a9c.
--
Michael

Renaming cryptohashes.c to cryptohashfuncs.c

Attachments: