Differential D31180 Diff 92190 lib/libc/string/mempcpy.c

Changeset View

Standalone View

lib/libc/string/mempcpy.c

This file was added.

				/*-
				* SPDX-License-Identifier: BSD-2-Clause-FreeBSD
				*
				* Copyright (c) 2021 The FreeBSD Foundation
				* All rights reserved.
				*
				* This software was developed by Konstantin Belousov <kib@FreeBSD.org>
				* under sponsorship from the FreeBSD Foundation.
				*
				* Redistribution and use in source and binary forms, with or without
				* modification, are permitted provided that the following conditions
				* are met:
				* 1. Redistributions of source code must retain the above copyright
				* notice, this list of conditions and the following disclaimer.
				* 2. Redistributions in binary form must reproduce the above copyright
				* notice, this list of conditions and the following disclaimer in the
				* documentation and/or other materials provided with the distribution.
				*
				* THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
				* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
				* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
				* ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
				* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
				* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
				* OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
				* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
				* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
				* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
				* SUCH DAMAGE.
				*/

				#include <sys/cdefs.h>
				__FBSDID("$FreeBSD$");

				#include <string.h>

				void *
				mempcpy(void dst, const void src, size_t len)
				markjUnsubmitted Done Inline Actions Shouldn't the restrict qualifier be used in the definition as well? markj: Shouldn't the restrict qualifier be used in the definition as well?
				kibAuthorUnsubmitted Done Inline Actions I always thought that not. I was not able to find an answer in the standard from the cursory look, and think that it is not worth the time. kib: I always thought that not. I was not able to find an answer in the standard from the cursory…
				markjUnsubmitted Not Done Inline Actions I am not sure either. The diff now uses __restrict in the wmempcpy() definition but not in the mempcpy() definition. markj: I am not sure either. The diff now uses __restrict in the wmempcpy() definition but not in the…
				kibAuthorUnsubmitted Done Inline Actions Oh, I think that adding __restrict to mempcpy is a mistake. Look at the man page text for memcpy(3) about overlapping. On the other hand, memcpy() does has restrict on args. kib: Oh, I think that adding __restrict to mempcpy is a mistake. Look at the man page text for…
				markjUnsubmitted Not Done Inline Actions Indeed, though the memcpy() definition in bcopy.c does not use `restrict`. markj: Indeed, though the memcpy() definition in bcopy.c does not use `restrict`.
				{
				return ((char *)memcpy(dst, src, len) + len);
				}