src/share/vm/compiler/compileLog.cpp

rev 8529 : 8129423: Fix unlink() of LogCompilation tmp files lost in merge of 8007993 and 8060074.

*** 1,7 ****
  /*
!  * Copyright (c) 2002, 2014, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
--- 1,7 ----
  /*
!  * Copyright (c) 2002, 2015, Oracle and/or its affiliates. All rights reserved.
   * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
   *
   * This code is free software; you can redistribute it and/or modify it
   * under the terms of the GNU General Public License version 2 only, as
   * published by the Free Software Foundation.
*** 56,72 ****
  }
  
  CompileLog::~CompileLog() {
    delete _out; // Close fd in fileStream::~fileStream()
    _out = NULL;
    FREE_C_HEAP_ARRAY(char, _identities);
    FREE_C_HEAP_ARRAY(char, _file);
  }
  
  
  // see_tag, pop_tag:  Override the default do-nothing methods on xmlStream.
! // These methods provide a hook for managing the the extra context markup.
  void CompileLog::see_tag(const char* tag, bool push) {
    if (_context.size() > 0 && _out != NULL) {
      _out->write(_context.base(), _context.size());
      _context.reset();
    }
--- 56,74 ----
  }
  
  CompileLog::~CompileLog() {
    delete _out; // Close fd in fileStream::~fileStream()
    _out = NULL;
+   // Remove partial file after merging in CompileLog::finish_log_on_error
+   unlink(_file);
    FREE_C_HEAP_ARRAY(char, _identities);
    FREE_C_HEAP_ARRAY(char, _file);
  }
  
  
  // see_tag, pop_tag:  Override the default do-nothing methods on xmlStream.
! // These methods provide a hook for managing the extra context markup.
  void CompileLog::see_tag(const char* tag, bool push) {
    if (_context.size() > 0 && _out != NULL) {
      _out->write(_context.base(), _context.size());
      _context.reset();
    }