# HG changeset patch # User sla # Date 1408688018 -7200 # Node ID fd78b4574c4ac7ad011063e41692b064ea58a987 # Parent f3c619e4afb11e6abd2f882138c7a9c50b3b6c65 8055754: filemap.cpp does not compile with clang Reviewed-by: kvn, iklam, coleenp diff -r f3c619e4afb1 -r fd78b4574c4a src/share/vm/memory/filemap.cpp --- a/src/share/vm/memory/filemap.cpp Mon Apr 09 20:36:04 2018 -0400 +++ b/src/share/vm/memory/filemap.cpp Fri Aug 22 08:13:38 2014 +0200 @@ -452,7 +452,7 @@ // close and remove the file. See bug 6372906. close(); remove(_full_path); - fail_stop("Unable to write to shared archive file.", NULL); + fail_stop("Unable to write to shared archive file."); } } _file_offset += nbytes; @@ -471,7 +471,7 @@ // that the written file is the correct length. _file_offset -= 1; if (lseek(_fd, (long)_file_offset, SEEK_SET) < 0) { - fail_stop("Unable to seek.", NULL); + fail_stop("Unable to seek."); } char zero = 0; write_bytes(&zero, 1); @@ -542,7 +542,7 @@ // other reserved memory (like the code cache). ReservedSpace rs(size, os::vm_allocation_granularity(), false, requested_addr); if (!rs.is_reserved()) { - fail_continue(err_msg("Unable to reserve shared space at required address " INTPTR_FORMAT, requested_addr)); + fail_continue("Unable to reserve shared space at required address " INTPTR_FORMAT, requested_addr); return rs; } // the reserved virtual memory is for mapping class data sharing archive @@ -566,7 +566,7 @@ requested_addr, size, si->_read_only, si->_allow_exec); if (base == NULL || base != si->_base) { - fail_continue(err_msg("Unable to map %s shared space at required address.", shared_region_name[i])); + fail_continue("Unable to map %s shared space at required address.", shared_region_name[i]); return NULL; } #ifdef _WINDOWS @@ -605,7 +605,7 @@ void FileMapInfo::assert_mark(bool check) { if (!check) { - fail_stop("Mark mismatch while restoring from shared file.", NULL); + fail_stop("Mark mismatch while restoring from shared file."); } } @@ -748,7 +748,7 @@ void FileMapInfo::stop_sharing_and_unmap(const char* msg) { FileMapInfo *map_info = FileMapInfo::current_info(); if (map_info) { - map_info->fail_continue(msg); + map_info->fail_continue("%s", msg); for (int i = 0; i < MetaspaceShared::n_regions; i++) { if (map_info->_header->_space[i]._base != NULL) { map_info->unmap_region(i); @@ -756,6 +756,6 @@ } } } else if (DumpSharedSpaces) { - fail_stop(msg, NULL); + fail_stop("%s", msg); } } diff -r f3c619e4afb1 -r fd78b4574c4a src/share/vm/memory/filemap.hpp --- a/src/share/vm/memory/filemap.hpp Mon Apr 09 20:36:04 2018 -0400 +++ b/src/share/vm/memory/filemap.hpp Fri Aug 22 08:13:38 2014 +0200 @@ -196,8 +196,8 @@ bool remap_shared_readonly_as_readwrite(); // Errors. - static void fail_stop(const char *msg, ...); - static void fail_continue(const char *msg, ...); + static void fail_stop(const char *msg, ...) ATTRIBUTE_PRINTF(1, 2); + static void fail_continue(const char *msg, ...) ATTRIBUTE_PRINTF(1, 2); // Return true if given address is in the mapped shared space. bool is_in_shared_space(const void* p) NOT_CDS_RETURN_(false);