Add one more Valgrind header (memcheck.h) and update the one we already have.

Add one more Valgrind header (memcheck.h) and update the one we already have.

[Alexander Potapenko, 2011-01-25 09:57:57]

http://codereview.chromium.org/6327018/diff/1/base/third_party/valgrind/READM...
File base/third_party/valgrind/README.chromium (right):

http://codereview.chromium.org/6327018/diff/1/base/third_party/valgrind/READM...
base/third_party/valgrind/README.chromium:5: current process is running under
Valgrind, tell Memcheck tool about custom
Mb just "determine whether the current process is running under Valgrind and
tell Memcheck tool about custom memory allocators" ?

[Evgeniy Stepanov, 2011-01-25 10:22:17]

http://codereview.chromium.org/6327018/diff/1/base/third_party/valgrind/READM...
File base/third_party/valgrind/README.chromium (right):

http://codereview.chromium.org/6327018/diff/1/base/third_party/valgrind/READM...
base/third_party/valgrind/README.chromium:5: current process is running under
Valgrind, tell Memcheck tool about custom
On 2011/01/25 09:57:58, Alexander Potapenko wrote:
> Mb just "determine whether the current process is running under Valgrind and
> tell Memcheck tool about custom memory allocators" ?

Fine. We are actually using it for the "and so on" part, but your version sounds
better :)

[Timur Iskhodzhanov, 2011-01-25 15:55:46]

Am I right that your files are 100%-identical to those in Valgrind SVN?
If so - LGTM.

Please wait for Evan to review.

[Timur Iskhodzhanov, 2011-01-25 15:56:19]

Also, consider sending a tryjob to linux_valgrind to make sure it compiles fine.

On 2011/01/25 15:55:46, Timur Iskhodzhanov wrote:
> Am I right that your files are 100%-identical to those in Valgrind SVN?
> If so - LGTM.
> 
> Please wait for Evan to review.

[evanm, 2011-01-25 16:57:56]

LGTM

[eugenis, 2011-01-25 18:14:51]

They are 100% identical. I doubt that I can send a try job, but I'll do
something about it tomorrow.
 On Jan 25, 2011 7:57 PM, <evanm@google.com> wrote:
> LGTM
>
>
> http://codereview.chromium.org/6327018/