Commit 6c9f651
committed
Clean up actor code listing.
The value returned by `endIndex` is an invalid, past-the-end index --
saving it and assuming it will become valid when adding an element isn't
really good style. Explicitly removing the last element is clearer and
simpler anyhow.
Marking `contents` as private is a little more realistic -- that gives
the actor some internal state whose concurrency is being managed.1 parent 570c9aa commit 6c9f651
1 file changed
+4
-14
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2296 | 2296 | | |
2297 | 2297 | | |
2298 | 2298 | | |
2299 | | - | |
| 2299 | + | |
2300 | 2300 | | |
2301 | 2301 | | |
2302 | 2302 | | |
| |||
2322 | 2322 | | |
2323 | 2323 | | |
2324 | 2324 | | |
2325 | | - | |
| 2325 | + | |
2326 | 2326 | | |
2327 | | - | |
2328 | 2327 | | |
2329 | 2328 | | |
2330 | | - | |
2331 | | - | |
| 2329 | + | |
2332 | 2330 | | |
2333 | 2331 | | |
2334 | 2332 | | |
| |||
2340 | 2338 | | |
2341 | 2339 | | |
2342 | 2340 | | |
2343 | | - | |
2344 | 2341 | | |
2345 | 2342 | | |
2346 | | - | |
2347 | | - | |
| 2343 | + | |
2348 | 2344 | | |
2349 | 2345 | | |
2350 | 2346 | | |
| |||
2358 | 2354 | | |
2359 | 2355 | | |
2360 | 2356 | | |
2361 | | - | |
2362 | | - | |
2363 | | - | |
2364 | 2357 | | |
2365 | 2358 | | |
2366 | 2359 | | |
| |||
2369 | 2362 | | |
2370 | 2363 | | |
2371 | 2364 | | |
2372 | | - | |
2373 | | - | |
2374 | | - | |
2375 | 2365 | | |
2376 | 2366 | | |
2377 | 2367 | | |
| |||
0 commit comments