Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(108587)

Issue 1540: WIP fix for higher rd_collect.sh CPU utilization.

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 months, 2 weeks ago by negge
Modified:
4 months, 2 weeks ago
Reviewers:
luc.trudeau
Visibility:
Public.

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M tools/rd_collect.sh View 1 chunk +2 lines, -1 line 1 comment Download
M tools/rd_collect_aom.sh View 2 chunks +15 lines, -18 lines 0 comments Download

Messages

Total messages: 2
negge
4 months, 2 weeks ago #1
luc.trudeau
4 months, 2 weeks ago #2
http://review.xiph.org/1540/diff/2620/tools/rd_collect.sh
File tools/rd_collect.sh (right):

http://review.xiph.org/1540/diff/2620/tools/rd_collect.sh#newcode388
tools/rd_collect.sh:388: RANGE="20 32 43 55 63"
How about making the RANGE configurable

if [ -z ${RANGE+x} ]; then
  RANGE="20 32 43 55 63"
fi

This is based on
https://stackoverflow.com/questions/3601515/how-to-check-if-a-variable-is-set...
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld