FX Bool considered harmful, part 3
[pdfium.git] / fpdfsdk / src / javascript / event.cpp
index 196ab32..ee73e42 100644 (file)
@@ -1,7 +1,7 @@
 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
+
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../../include/javascript/JavaScript.h"
@@ -43,12 +43,12 @@ BEGIN_JS_STATIC_PROP(CJS_Event)
        JS_STATIC_PROP_ENTRY(willCommit)
 END_JS_STATIC_PROP()
 
-BEGIN_JS_STATIC_METHOD(CJS_Event)  
+BEGIN_JS_STATIC_METHOD(CJS_Event)
 END_JS_STATIC_METHOD()
 
 IMPLEMENT_JS_CLASS(CJS_Event,event)
 
-event::event(CJS_Object * pJsObject) : CJS_EmbedObj(pJsObject)                           
+event::event(CJS_Object * pJsObject) : CJS_EmbedObj(pJsObject)
 {
 }
 
@@ -56,7 +56,7 @@ event::~event(void)
 {
 }
 
-FX_BOOL event::change(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::change(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -73,12 +73,12 @@ FX_BOOL event::change(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErr
        {
                vp << wChange;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::changeEx(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::changeEx(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -86,12 +86,12 @@ FX_BOOL event::changeEx(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sE
        ASSERT(pEvent != NULL);
 
        vp << pEvent->ChangeEx();
-       return TRUE;    
+       return true;
 }
 
-FX_BOOL event::commitKey(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::commitKey(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -99,29 +99,29 @@ FX_BOOL event::commitKey(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& s
        ASSERT(pEvent != NULL);
 
        vp << pEvent->CommitKey();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::fieldFull(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
-{      
+bool event::fieldFull(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
+{
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
        CJS_EventHandler* pEvent = pContext->GetEventHandler();
        ASSERT(pEvent != NULL);
 
        if (!vp.IsGetting() && wcscmp((const wchar_t*)pEvent->Name(),L"Keystroke") != 0)
-               return FALSE;
+               return false;
 
        if (pEvent->FieldFull())
-               vp << TRUE;
+               vp << true;
        else
-               vp << FALSE;
-       return TRUE;
+               vp << false;
+       return true;
 }
 
-FX_BOOL event::keyDown(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::keyDown(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -129,15 +129,15 @@ FX_BOOL event::keyDown(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sEr
        ASSERT(pEvent != NULL);
 
        if (pEvent->KeyDown())
-               vp << TRUE;
+               vp << true;
        else
-               vp << FALSE;
-       return TRUE;
+               vp << false;
+       return true;
 }
 
-FX_BOOL event::modifier(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::modifier(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -145,15 +145,15 @@ FX_BOOL event::modifier(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sE
        ASSERT(pEvent != NULL);
 
        if (pEvent->Modifier())
-               vp << TRUE;
+               vp << true;
        else
-               vp << FALSE;
-       return TRUE;
+               vp << false;
+       return true;
 }
 
-FX_BOOL event::name(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::name(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -161,17 +161,17 @@ FX_BOOL event::name(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError
        ASSERT(pEvent != NULL);
 
        vp << pEvent->Name();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::rc(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::rc(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
        CJS_EventHandler* pEvent = pContext->GetEventHandler();
        ASSERT(pEvent != NULL);
 
-    FX_BOOL &bRc = pEvent->Rc();
+    bool &bRc = pEvent->Rc();
        if (vp.IsSetting())
        {
                vp>>bRc;
@@ -180,12 +180,12 @@ FX_BOOL event::rc(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
        {
                vp<<bRc;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::richChange(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::richChange(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       return TRUE;
+       return true;
        if (vp.IsSetting())
        {
        }
@@ -193,12 +193,12 @@ FX_BOOL event::richChange(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString&
        {
                ;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::richChangeEx(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::richChangeEx(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       return TRUE;
+       return true;
        if (vp.IsSetting())
        {
        }
@@ -206,13 +206,13 @@ FX_BOOL event::richChangeEx(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString
        {
                ;
        }
-       return TRUE;
+       return true;
 }
 
 
-FX_BOOL event::richValue(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::richValue(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       return TRUE;
+       return true;
        if (vp.IsSetting())
        {
        }
@@ -220,10 +220,10 @@ FX_BOOL event::richValue(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& s
        {
                ;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::selEnd(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::selEnd(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -232,7 +232,7 @@ FX_BOOL event::selEnd(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErr
 
        if (wcscmp((const wchar_t*)pEvent->Name(),L"Keystroke") != 0)
        {
-               return TRUE;
+               return true;
        }
 
        int &iSelEnd = pEvent->SelEnd();
@@ -244,11 +244,11 @@ FX_BOOL event::selEnd(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErr
        {
                vp << iSelEnd;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::selStart(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
-{      
+bool event::selStart(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
+{
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
        CJS_EventHandler* pEvent = pContext->GetEventHandler();
@@ -256,7 +256,7 @@ FX_BOOL event::selStart(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sE
 
        if (wcscmp((const wchar_t*)pEvent->Name(),L"Keystroke") != 0)
        {
-               return TRUE;
+               return true;
        }
        int &iSelStart = pEvent->SelStart();
        if (vp.IsSetting())
@@ -267,12 +267,12 @@ FX_BOOL event::selStart(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sE
        {
                vp << iSelStart;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::shift(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::shift(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -280,15 +280,15 @@ FX_BOOL event::shift(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErro
        ASSERT(pEvent != NULL);
 
        if (pEvent->Shift())
-               vp << TRUE;
+               vp << true;
        else
-               vp << FALSE;
-       return TRUE;
+               vp << false;
+       return true;
 }
 
-FX_BOOL event::source(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::source(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;       
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -296,12 +296,12 @@ FX_BOOL event::source(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErr
        ASSERT(pEvent != NULL);
 
        vp << pEvent->Source()->GetJSObject();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::target(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::target(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -309,12 +309,12 @@ FX_BOOL event::target(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErr
        ASSERT(pEvent != NULL);
 
        vp<<pEvent->Target_Field()->GetJSObject();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::targetName(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::targetName(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -322,12 +322,12 @@ FX_BOOL event::targetName(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString&
        ASSERT(pEvent != NULL);
 
        vp << pEvent->TargetName();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::type(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::type(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -335,10 +335,10 @@ FX_BOOL event::type(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError
        ASSERT(pEvent != NULL);
 
        vp << pEvent->Type();
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::value(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::value(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -346,24 +346,24 @@ FX_BOOL event::value(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sErro
        ASSERT(pEvent != NULL);
 
        if (wcscmp((const wchar_t*)pEvent->Type(),L"Field") != 0)
-               return FALSE;
+               return false;
        if(!pEvent->m_pValue)
-               return FALSE;
+               return false;
        CFX_WideString & val = pEvent->Value();
        if (vp.IsSetting())
        {
-               val = vp;
+               vp >> val;
        }
        else
        {
                vp << val;
        }
-       return TRUE;
+       return true;
 }
 
-FX_BOOL event::willCommit(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString& sError)
+bool event::willCommit(IFXJS_Context* cc, CJS_PropValue& vp, CFX_WideString& sError)
 {
-       if (!vp.IsGetting())return FALSE;
+       if (!vp.IsGetting())return false;
 
        CJS_Context* pContext = (CJS_Context*)cc;
        ASSERT(pContext != NULL);
@@ -371,9 +371,9 @@ FX_BOOL event::willCommit(IFXJS_Context* cc, CJS_PropValue& vp, JS_ErrorString&
        ASSERT(pEvent != NULL);
 
        if (pEvent->WillCommit())
-               vp << TRUE;
+               vp << true;
        else
-               vp << FALSE;
-       return TRUE;
+               vp << false;
+       return true;
 }