Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Usage of mempcpy in core/thumbnalijob.cpp #297

Closed
BabylonAS opened this issue Oct 14, 2018 · 2 comments
Closed

Usage of mempcpy in core/thumbnalijob.cpp #297

BabylonAS opened this issue Oct 14, 2018 · 2 comments

Comments

@BabylonAS
Copy link
Contributor

BabylonAS commented Oct 14, 2018

The file core/thumbnailjob.cpp features the use of mempcpy(3) function on line 99. This particular function is a non-standard extension by GNU, which may cause concerns regarding compatibility and is already the reason why libfm-qt refuses to compile on FreeBSD when built from this very repository by clang (however, there is a FreeBSD package and port available).

Possible Solution
  1. Replace mempcpy with an applicable function from the Standard C Library. I don’t quite know what exactly makes mempcpy special, but right now I see memmove as the closest alternative.
  2. Provide information on how to enable GNU extensions on Clang. If this solution fails, this might mean that I have to use glibc and gcc...
System Information
  • Distribution & Version: FreeBSD 1.12-RELEASE (i386)
  • Compiler: clang (LLVM 6.0)
@BabylonAS
Copy link
Contributor Author

After examining the mempcpy source code at Sourceware, I find that it offers no advantages over basic memcpy, therefore I am submitting a pull request.

@agaida
Copy link
Member

agaida commented Oct 15, 2018

fixed in /pull/298

@agaida agaida closed this as completed Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants