From 24e6d48d9b8872009901b45249ee1689c5856d78 Mon Sep 17 00:00:00 2001 From: "cira@chromium.org" Date: Mon, 15 Aug 2011 22:29:03 +0000 Subject: [PATCH] Return error not null for date time format. BUG=v8:1602 TEST=new (new v8Locale().createDateTimeFormat()).getWeekdays should return Error not a Null. Review URL: http://codereview.chromium.org/7647027 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@8935 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/extensions/experimental/datetime-format.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/extensions/experimental/datetime-format.cc b/src/extensions/experimental/datetime-format.cc index 7f46302..94a29ac 100644 --- a/src/extensions/experimental/datetime-format.cc +++ b/src/extensions/experimental/datetime-format.cc @@ -135,7 +135,7 @@ v8::Handle DateTimeFormat::GetMonths(const v8::Arguments& args) { v8::Handle DateTimeFormat::GetWeekdays(const v8::Arguments& args) { icu::SimpleDateFormat* date_format = UnpackDateTimeFormat(args.Holder()); if (!date_format) { - ThrowUnexpectedObjectError(); + return ThrowUnexpectedObjectError(); } const icu::DateFormatSymbols* symbols = date_format->getDateFormatSymbols(); -- 2.7.4