I got that non-working version of
apl-1.7.tar.gz from ftp.gnu.org/pub/gnu/apl,
dated 17 March 2017, so I guess it's a bit out-of-date.
I just now got a chance to pull down the
savannah svn version, which ./configures fine, but there a few
compile errors with g++ (GCC) 8.1.1 20180502 (Red Hat
8.1.1-1). They're all the same nit-picking bug
complaining about memcpy:
error: ‘void* memcpy(void*, const void*, size_t)’ writing to an
object of type ‘class Token’ with no trivial copy-assignment;
use copy-initialization instead [-Werror=class-memaccess]
Here's the relevant portion of an svn diff:
Index: src/Executable.cc
===================================================================
--- src/Executable.cc (revision 1054)
+++ src/Executable.cc (working copy)
@@ -794,9 +794,9 @@
for (;t1 < t2; ++t1, --t2)
{
char tt[sizeof(Token)];
- memcpy(tt, t1, sizeof(Token));
- memcpy(t1, t2, sizeof(Token));
- memcpy(t2, tt, sizeof(Token));
+ memcpy(static_cast <void *>(tt), t1,
sizeof(Token));
+ memcpy(static_cast <void *>(t1), t2,
sizeof(Token));
+ memcpy(static_cast <void *>(t2), tt,
sizeof(Token));
}
}
//-----------------------------------------------------------------------------
Index: src/InputFile.cc
===================================================================
--- src/InputFile.cc (revision 1054)
+++ src/InputFile.cc (working copy)
@@ -40,7 +40,7 @@
//-----------------------------------------------------------------------------
InputFile & InputFile::operator =(const InputFile
& other)
{
- memcpy(this, &other, sizeof(InputFile)); //
illegally duplicates strings!
+ memcpy(static_cast <void *>(this), &other,
sizeof(InputFile)); // illegally duplicates strings!
::new (&object_filter)
UCS_string_vector(other.object_filter);
new (&filename)
UTF8_string(&other.filename[0], other.filename.size());
return *this;
Index: src/Token.cc
===================================================================
--- src/Token.cc (revision 1054)
+++ src/Token.cc (working copy)
@@ -840,9 +840,9 @@
for (;t1 < t2; ++t1, --t2)
{
char tt[sizeof(Token)];
- memcpy(tt, t1, sizeof(Token));
- memcpy(t1, t2, sizeof(Token));
- memcpy(t2, tt, sizeof(Token));
+ memcpy(static_cast <void *>(tt), t1,
sizeof(Token));
+ memcpy(static_cast <void *>(t1), t2,
sizeof(Token));
+ memcpy(static_cast <void *>(t2), tt,
sizeof(Token));
}
}
(If you just do the obvious and (void *) cast the target
argument, it complains that you're using an "old style" cast.)
I don't have any older g++ versions around, so I have no idea
if this patch will break anything.
Chris
On 26/06/18 13:43, Juergen Sauermann
wrote:
Hi Chris,
the svn E155007 happens if you unpack a GNU APL tar
file (as opposed to
checking out GNU APL from SVN, which is the proposed method).
See
https://savannah.gnu.org/svn/?group=apl.
The other error was fixed already in SVN version 1052
(see
http://svn.savannah.gnu.org/viewvc/apl/trunk/src/Svar_record.hh?r1=1051&r2=1052
for details.
Fro the first error message I can see that you have SVN
installed. So checking out
the latest version of GNU APL is simply runnung this command:
svn co http://svn.savannah.gnu.org/svn/apl
That should give you the latest SVN version (currently 1053)
of GNU APL where
both errors above should disappear.
File apl-1.7.tar.gz (whereever you found that) is too
old to compile under
the most recent gcc versions.
/// Jürgen
signature.asc