Thread: Patch for a bug in handling of the KEVNM header field variable

Started: 2012-12-06 02:52:03
Last activity: 2013-01-06 02:15:30
Topics: SAC Developers
(no body)
  • I did your test and reproduced the error. I then included your pathces,
    and not only did it fix the errors you found but also errors I had fround
    when I used a Fortran call like

    call setkhv('kevnm', 'co 400 blt zp', nerr)

    followed with a wsac3, and looked at the KEVNM using lh within SAC.

    Thanks!

    We will include these patches in the next release of SAC.

    On Wed, 5 Dec 2012, A. Kasahara wrote:

    Dear all,

    I wrote a patch to fix a bug related to handling of the KEVNM header field
    variable.
    The last character of KEVNM header field value is deleted by following
    successive operations:
    sac> write sac tmp.sac
    sac> read tmp.sac
    sac> write alpha tmp.sac_ascii
    sac> read alpha tmp.sac_ascii

    My environment is
    $ sac
    SEISMIC ANALYSIS CODE [12/04/2012 (Version 101.5c)]
    Copyright 1995 Regents of the University of California

    You can test the bug by executing an attached script, test_kevnm.sh.
    $ chmod u+x test_kevnm.bash
    $ ./test_kevnm.bash SOME_SAC_FILE_TO_BE_USED_BY_TEST

    To apply the patch:$ patch --strip=0 < kevnm.diff

    Sincerely,
    Amato

    --
    Amato Kasahara,
    Master's Program in Geoscience,
    Graduate School of Life and Environmental Sciences,
    University of Tsukuba




20:13:50 v.e73c6799