Opened 12 years ago

Closed 11 years ago

#4991 closed Patches (fixed)

uuid compile errors and warnings

Reported by: olim@… Owned by: Andy Tompkins
Milestone: Boost 1.46.0 Component: uuid
Version: Boost 1.45.0 Severity: Problem
Keywords: Cc:

Description

Can someone please apply this patch to boost uuid? This patch fixes a compile error and warning when using a Green Hills compiler.

Attachments (1)

uuid.patch​ (1.3 KB ) - added by olim@… 12 years ago.
Compile error fix for GHS compiler

Download all attachments as: .zip

Change History (9)

by olim@…, 12 years ago

Attachment: uuid.patch​ added

Compile error fix for GHS compiler

comment:1 by Marshall Clow, 12 years ago

What is the call that is in <memory.h> ?

comment:2 by Andy Tompkins, 12 years ago

I agree with the fix in string_generator.hpp. In seed_rng.hpp ::memcpy is used from <memory.h>

comment:3 by anonymous, 12 years ago

So the green hills library doesn't have memcpy in <cstring>, which is what the standard requires?

comment:4 by Marshall Clow, 12 years ago

Anonymous ==> marshall in the last comment.

comment:5 by Andy Tompkins, 12 years ago

Ah, I see. I was reading things wrong. Right, memcpy should be in <cstring>. I am no longer sure why <memory.h> is included. Maybe I was reading things wrong much earlier in this file's life. If no tests on any platform fail after removing the #include line, I see no reason to not remove it completely.

comment:6 by olim@…, 12 years ago

The green hills compiler does have memcpy in <cstring>, but it does not have <memory.h>. Well, it looks like you guys figured it out already.

comment:7 by Andy Tompkins, 12 years ago

Removed semicolon from string_generator.hpp. Removed #include <memory.h> from seed_rng.hpp. Committed to trunk.

comment:8 by Andy Tompkins, 11 years ago

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.