Skip to content

Commit

Permalink
Further changes to address #67 and #68.
Browse files Browse the repository at this point in the history
  • Loading branch information
joeberkovitz committed Oct 8, 2015
1 parent 1d9b25e commit 1c07de1
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions index.html
Original file line number Diff line number Diff line change
Expand Up @@ -1256,7 +1256,7 @@ <h2 id="AudioContext">
disabled, and the time-domain waveform has the amplitudes as given
by the Fourier coefficients.
<p>
As <a>PeriodicWave</a> objects maintain their own representation,
As <a>PeriodicWave</a> objects maintain their own copies of these arrays,
any modification of the arrays uses as the <code>real</code> and
<code>imag</code> parameters after the call to <a href=
"#widl-AudioContext-createPeriodicWave-PeriodicWave-Float32Array-real-Float32Array-imag">
Expand Down Expand Up @@ -2473,9 +2473,10 @@ <h2 id="AudioParam">
<dd>
A Float32Array representing a parameter value curve. These
values will apply starting at the given time and lasting for
the given duration. Any modification to the the array used as
<a id="#dfn-values"><code>values</code></a> argument after the
call won't have any effect on the <a>AudioParam</a>.
the given duration.
When this method is called, an internal copy of the curve is created
for automation purposes. Subsequent modifications of the
contents of the passed-in array therefore have no effect on the the <a>AudioParam</a>.

This comment has been minimized.

Copy link
@rtoy

rtoy Oct 8, 2015

Member

I find the original text equally clear as the new text. What's wrong?

</dd>
<dt>
double startTime
Expand Down

2 comments on commit 1c07de1

@joeberkovitz
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing was wrong with the original description per se, I just thought we should use identical language constructs to describe the same thing in different places.

@rtoy
Copy link
Member

@rtoy rtoy commented on 1c07de1 Oct 9, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Please sign in to comment.