LGTM

On Mon, Mar 9, 2009 at 8:47 PM, <[email protected]> wrote:

> Reviewers: Søren Gjesse,
>
> Description:
> Fix test for building on 64-bit Linux.
>
> TBR=sgjesse
>
> Please review this at http://codereview.chromium.org/41007
>
> SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
>
> Affected files:
>   M     test/cctest/test-api.cc
>
>
> Index: test/cctest/test-api.cc
> ===================================================================
> --- test/cctest/test-api.cc     (revision 1462)
> +++ test/cctest/test-api.cc     (working copy)
> @@ -5321,7 +5321,7 @@
>
>
>  static void CheckTryCatchSourceInfo(v8::Handle<v8::Script> script,
> -                                    char* resource_name,
> +                                    const char* resource_name,
>                                      int line_offset) {
>    v8::HandleScope scope;
>    v8::TryCatch try_catch;
> @@ -5360,7 +5360,7 @@
>        "\n"
>        "Foo();\n");
>
> -  char* resource_name;
> +  const char* resource_name;
>    v8::Handle<v8::Script> script;
>    resource_name = "test.js";
>    script = v8::Script::Compile(source, v8::String::New(resource_name));
>
>
>
> >
>

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to