LGTM, but maybe you should mention the issue number for the fixed bug?

On Mon, Jun 29, 2009 at 10:05 AM, <a...@chromium.org> wrote:
> Reviewers: Kasper Lund,
>
> Description:
> Prepare push of version 1.2.10 to trunk.
>
> Please review this at http://codereview.chromium.org/151015
>
> SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/
>
> Affected files:
>  M     ChangeLog
>  M     src/version.cc
>
>
> Index: src/version.cc
> ===================================================================
> --- src/version.cc      (revision 2286)
> +++ src/version.cc      (working copy)
> @@ -34,7 +34,7 @@
>  // cannot be changed without changing the SCons build script.
>  #define MAJOR_VERSION     1
>  #define MINOR_VERSION     2
> -#define BUILD_NUMBER      10
> +#define BUILD_NUMBER      11
>  #define PATCH_LEVEL       0
>  #define CANDIDATE_VERSION true
>
> Index: ChangeLog
> ===================================================================
> --- ChangeLog   (revision 2286)
> +++ ChangeLog   (working copy)
> @@ -1,3 +1,12 @@
> +2009-06-29: Version 1.2.10
> +
> +        Improved debugger support.
> +
> +        Fixed bug in exception message reporting.
> +
> +        Improved overall performance.
> +
> +
>  2009-06-23: Version 1.2.9
>
>         Improved math performance on ARM.
>
>
>

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to