Revision: 8935
Author: [email protected]
Date: Mon Aug 15 15:29:03 2011
Log: 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
http://code.google.com/p/v8/source/detail?r=8935
Modified:
/branches/bleeding_edge/src/extensions/experimental/datetime-format.cc
=======================================
--- /branches/bleeding_edge/src/extensions/experimental/datetime-format.cc
Fri Jun 24 09:58:28 2011
+++ /branches/bleeding_edge/src/extensions/experimental/datetime-format.cc
Mon Aug 15 15:29:03 2011
@@ -135,7 +135,7 @@
v8::Handle<v8::Value> 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();
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
