Skip to content

Commit

Permalink
Fixes bin/trace so it doesn't dirty the repo when autogen runs (#295)
Browse files Browse the repository at this point in the history
* Fixes bin/trace so it doesn't dirty the repo when autogen runs

* Put the trace max back to 110 and comment off H5O.c line
  • Loading branch information
derobins authored Feb 3, 2021
1 parent ab2c31b commit eac0599
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 7 deletions.
2 changes: 1 addition & 1 deletion bin/trace
Original file line number Diff line number Diff line change
Expand Up @@ -470,7 +470,7 @@ for $file (@ARGV) {
$file_args = 0;

# Ignore some files that do not need tracing macros
unless ($file eq "H5FDmulti.c" or $file eq "src/H5FDmulti.c" or $file eq "H5FDstdio.c" or $file eq "src/H5FDstdio.c") {
unless ($file eq "H5FDmulti.c" or $file eq "src/H5FDmulti.c" or $file eq "H5FDstdio.c" or $file eq "src/H5FDstdio.c" or $file eq "src/H5TS.c") {

# Snarf up the entire file
open SOURCE, $file or die "$file: $!\n";
Expand Down
5 changes: 3 additions & 2 deletions src/H5O.c
Original file line number Diff line number Diff line change
Expand Up @@ -1188,8 +1188,9 @@ H5Oget_info_by_name_async(const char *app_file, const char *app_func, unsigned a
herr_t ret_value = SUCCEED; /* Return value */

FUNC_ENTER_API(FAIL)
H5TRACE9("e", "*s*sIui*sxIuii", app_file, app_func, app_line, loc_id, name, oinfo, fields, lapl_id,
es_id);
/* clang-format off */
H5TRACE9("e", "*s*sIui*sxIuii", app_file, app_func, app_line, loc_id, name, oinfo, fields, lapl_id, es_id);
/* clang-format on */

/* Set up request token pointer for asynchronous operation */
if (H5ES_NONE != es_id)
Expand Down
4 changes: 0 additions & 4 deletions src/H5TS.c
Original file line number Diff line number Diff line change
Expand Up @@ -408,8 +408,6 @@ herr_t
H5TSmutex_acquire(unsigned int lock_count, hbool_t *acquired){
FUNC_ENTER_API_NAMECHECK_ONLY

/*NO TRACE*/

FUNC_LEAVE_API_NAMECHECK_ONLY(H5TS__mutex_acquire(&H5_g.init_lock, lock_count, acquired))}
/* end H5TSmutex_acquire() */

Expand Down Expand Up @@ -606,7 +604,6 @@ H5TSmutex_get_attempt_count(unsigned int *count)
herr_t ret_value = SUCCEED;

FUNC_ENTER_API_NAMECHECK_ONLY
/*NO TRACE*/

#ifdef H5_HAVE_WIN_THREADS
/* Add Win32 equivalent here when async is supported */
Expand Down Expand Up @@ -644,7 +641,6 @@ H5TSmutex_release(unsigned int *lock_count)
herr_t ret_value = SUCCEED;

FUNC_ENTER_API_NAMECHECK_ONLY
/*NO TRACE*/

*lock_count = 0;
if (0 != H5TS__mutex_unlock(&H5_g.init_lock, lock_count))
Expand Down

0 comments on commit eac0599

Please sign in to comment.