js_global_data and global_alternate violate naming rules.

fpdfsdk/src/javascript/global.cpp

 // 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/IJavaScript.h"
 #include "../../include/javascript/JS_Context.h"
 #include "../../include/javascript/JS_Define.h"
 #include "../../include/javascript/JS_EventHandler.h"
@@ skipping 71 matching lines @@
 BEGIN_JS_STATIC_CONST(CJS_Global)
 END_JS_STATIC_CONST()
 
 BEGIN_JS_STATIC_PROP(CJS_Global)
 END_JS_STATIC_PROP()
 
 BEGIN_JS_STATIC_METHOD(CJS_Global)
 JS_STATIC_METHOD_ENTRY(setPersistent)
 END_JS_STATIC_METHOD()
 
-IMPLEMENT_SPECIAL_JS_CLASS(CJS_Global, global_alternate, global);
+IMPLEMENT_SPECIAL_JS_CLASS(CJS_Global, JSGlobalAlternate, global);
 
 FX_BOOL CJS_Global::InitInstance(IFXJS_Context* cc) {
   CJS_Context* pContext = (CJS_Context*)cc;
   ASSERT(pContext != NULL);
 
-  global_alternate* pGlobal = (global_alternate*)GetEmbedObject();
+  JSGlobalAlternate* pGlobal = (JSGlobalAlternate*)GetEmbedObject();
   ASSERT(pGlobal != NULL);
 
   pGlobal->Initial(pContext->GetReaderApp());
 
   return TRUE;
 };
 
-global_alternate::global_alternate(CJS_Object* pJSObject)
-    : CJS_EmbedObj(pJSObject), m_pApp(NULL) {}
+JSGlobalAlternate::JSGlobalAlternate(CJS_Object* pJSObject)
+    : CJS_EmbedObj(pJSObject), m_pApp(NULL) {
+}
 
-global_alternate::~global_alternate() {
+JSGlobalAlternate::~JSGlobalAlternate() {
   DestroyGlobalPersisitentVariables();
   m_pApp->GetRuntimeFactory()->ReleaseGlobalData();
 }
 
-void global_alternate::Initial(CPDFDoc_Environment* pApp) {
+void JSGlobalAlternate::Initial(CPDFDoc_Environment* pApp) {
   m_pApp = pApp;
   m_pGlobalData = pApp->GetRuntimeFactory()->NewGlobalData(pApp);
   UpdateGlobalPersistentVariables();
 }
 
-FX_BOOL global_alternate::QueryProperty(const FX_WCHAR* propname) {
+FX_BOOL JSGlobalAlternate::QueryProperty(const FX_WCHAR* propname) {
   return CFX_WideString(propname) != L"setPersistent";
 }
 
-FX_BOOL global_alternate::DelProperty(IFXJS_Context* cc,
-                                      const FX_WCHAR* propname,
-                                      CFX_WideString& sError) {
-  js_global_data* pData = NULL;
+FX_BOOL JSGlobalAlternate::DelProperty(IFXJS_Context* cc,
+                                       const FX_WCHAR* propname,
+                                       CFX_WideString& sError) {
+  JSGlobalData* pData = NULL;
   CFX_ByteString sPropName = CFX_ByteString::FromUnicode(propname);
 
   if (m_mapGlobal.Lookup(sPropName, (void*&)pData)) {
     pData->bDeleted = TRUE;
     return TRUE;
   }
 
   return FALSE;
 }
 
-FX_BOOL global_alternate::DoProperty(IFXJS_Context* cc,
-                                     const FX_WCHAR* propname,
-                                     CJS_PropValue& vp,
-                                     CFX_WideString& sError) {
+FX_BOOL JSGlobalAlternate::DoProperty(IFXJS_Context* cc,
+                                      const FX_WCHAR* propname,
+                                      CJS_PropValue& vp,
+                                      CFX_WideString& sError) {
   if (vp.IsSetting()) {
     CFX_ByteString sPropName = CFX_ByteString::FromUnicode(propname);
     switch (vp.GetType()) {
       case VT_number: {
         double dData;
         vp >> dData;
         return SetGlobalVariables(sPropName, JS_GLOBALDATA_TYPE_NUMBER, dData,
                                   false, "", v8::Local<v8::Object>(), FALSE);
       }
       case VT_boolean: {
@@ skipping 28 matching lines @@
   } else {
     void* pVoid = nullptr;
     if (!m_mapGlobal.Lookup(CFX_ByteString::FromUnicode(propname), pVoid)) {
       vp.SetNull();
       return TRUE;
     }
     if (!pVoid) {
       vp.SetNull();
       return TRUE;
     }
-    js_global_data* pData = (js_global_data*)pVoid;
+    JSGlobalData* pData = (JSGlobalData*)pVoid;
     if (pData->bDeleted)
       return TRUE;
 
     switch (pData->nType) {
       case JS_GLOBALDATA_TYPE_NUMBER:
         vp << pData->dData;
         return TRUE;
       case JS_GLOBALDATA_TYPE_BOOLEAN:
         vp << pData->bData;
         return TRUE;
       case JS_GLOBALDATA_TYPE_STRING:
         vp << pData->sData;
         return TRUE;
       case JS_GLOBALDATA_TYPE_OBJECT: {
         v8::Local<v8::Object> obj =
             v8::Local<v8::Object>::New(vp.GetIsolate(), pData->pData);
         vp << obj;
         return TRUE;
       }
       case JS_GLOBALDATA_TYPE_NULL:
         vp.SetNull();
         return TRUE;
       default:
         break;
     }
   }
   return FALSE;
 }
 
-FX_BOOL global_alternate::setPersistent(IFXJS_Context* cc,
-                                        const CJS_Parameters& params,
-                                        CJS_Value& vRet,
-                                        CFX_WideString& sError) {
+FX_BOOL JSGlobalAlternate::setPersistent(IFXJS_Context* cc,
+                                         const CJS_Parameters& params,
+                                         CJS_Value& vRet,
+                                         CFX_WideString& sError) {
   CJS_Context* pContext = static_cast<CJS_Context*>(cc);
   if (params.size() != 2) {
     sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
     return FALSE;
   }
 
   CFX_ByteString sName = params[0].ToCFXByteString();
 
-  js_global_data* pData = NULL;
+  JSGlobalData* pData = NULL;
   if (m_mapGlobal.Lookup(sName, (void*&)pData)) {
     if (pData && !pData->bDeleted) {
       pData->bPersistent = params[1].ToBool();
       return TRUE;
     }
   }
 
   sError = JSGetStringFromID(pContext, IDS_STRING_JSNOGLOBAL);
   return FALSE;
 }
 
-void global_alternate::UpdateGlobalPersistentVariables() {
+void JSGlobalAlternate::UpdateGlobalPersistentVariables() {
   ASSERT(m_pGlobalData != NULL);
 
   for (int i = 0, sz = m_pGlobalData->GetSize(); i < sz; i++) {
     CJS_GlobalData_Element* pData = m_pGlobalData->GetAt(i);
     ASSERT(pData != NULL);
 
     switch (pData->data.nType) {
       case JS_GLOBALDATA_TYPE_NUMBER:
         SetGlobalVariables(pData->data.sKey, JS_GLOBALDATA_TYPE_NUMBER,
                            pData->data.dData, false, "",
@@ skipping 34 matching lines @@
         SetGlobalVariables(pData->data.sKey, JS_GLOBALDATA_TYPE_NULL, 0, false,
                            "", v8::Local<v8::Object>(),
                            pData->bPersistent == 1);
         JS_PutObjectNull(NULL, (JSFXObject)(*m_pJSObject),
                          pData->data.sKey.UTF8Decode().c_str());
         break;
     }
   }
 }
 
-void global_alternate::CommitGlobalPersisitentVariables() {
+void JSGlobalAlternate::CommitGlobalPersisitentVariables() {
   ASSERT(m_pGlobalData != NULL);
 
   FX_POSITION pos = m_mapGlobal.GetStartPosition();
   while (pos) {
     CFX_ByteString name;
-    js_global_data* pData = NULL;
+    JSGlobalData* pData = NULL;
     m_mapGlobal.GetNextAssoc(pos, name, (void*&)pData);
 
     if (pData) {
       if (pData->bDeleted) {
         m_pGlobalData->DeleteGlobalVariable(name);
       } else {
         switch (pData->nType) {
           case JS_GLOBALDATA_TYPE_NUMBER:
             m_pGlobalData->SetGlobalVariableNumber(name, pData->dData);
             m_pGlobalData->SetGlobalVariablePersistent(name,
@@ skipping 25 matching lines @@
             m_pGlobalData->SetGlobalVariableNull(name);
             m_pGlobalData->SetGlobalVariablePersistent(name,
                                                        pData->bPersistent);
             break;
         }
       }
     }
   }
 }
 
-void global_alternate::ObjectToArray(v8::Local<v8::Object> pObj,
-                                     CJS_GlobalVariableArray& array) {
+void JSGlobalAlternate::ObjectToArray(v8::Local<v8::Object> pObj,
+                                      CJS_GlobalVariableArray& array) {
   v8::Local<v8::Context> context = pObj->CreationContext();
   v8::Isolate* isolate = context->GetIsolate();
   v8::Local<v8::Array> pKeyList = JS_GetObjectElementNames(isolate, pObj);
   int nObjElements = pKeyList->Length();
 
   for (int i = 0; i < nObjElements; i++) {
     CFX_WideString ws =
         JS_ToString(isolate, JS_GetArrayElement(isolate, pKeyList, i));
     CFX_ByteString sKey = ws.UTF8Encode();
 
@@ skipping 35 matching lines @@
         pObjElement->nType = JS_GLOBALDATA_TYPE_NULL;
         pObjElement->sKey = sKey;
         array.Add(pObjElement);
       } break;
       default:
         break;
     }
   }
 }
 
-void global_alternate::PutObjectProperty(v8::Local<v8::Object> pObj,
-                                         CJS_KeyValue* pData) {
+void JSGlobalAlternate::PutObjectProperty(v8::Local<v8::Object> pObj,
+                                          CJS_KeyValue* pData) {
   ASSERT(pData != NULL);
 
   for (int i = 0, sz = pData->objData.Count(); i < sz; i++) {
     CJS_KeyValue* pObjData = pData->objData.GetAt(i);
     ASSERT(pObjData != NULL);
 
     switch (pObjData->nType) {
       case JS_GLOBALDATA_TYPE_NUMBER:
         JS_PutObjectNumber(NULL, (JSObject)pObj,
                            pObjData->sKey.UTF8Decode().c_str(),
@@ skipping 18 matching lines @@
                            (JSObject)pNewObj);
       } break;
       case JS_GLOBALDATA_TYPE_NULL:
         JS_PutObjectNull(NULL, (JSObject)pObj,
                          pObjData->sKey.UTF8Decode().c_str());
         break;
     }
   }
 }
 
-void global_alternate::DestroyGlobalPersisitentVariables() {
+void JSGlobalAlternate::DestroyGlobalPersisitentVariables() {
   FX_POSITION pos = m_mapGlobal.GetStartPosition();
   while (pos) {
     CFX_ByteString name;
-    js_global_data* pData = NULL;
+    JSGlobalData* pData = NULL;
     m_mapGlobal.GetNextAssoc(pos, name, (void*&)pData);
     delete pData;
   }
 
   m_mapGlobal.RemoveAll();
 }
 
-FX_BOOL global_alternate::SetGlobalVariables(const FX_CHAR* propname,
-                                             int nType,
-                                             double dData,
-                                             bool bData,
-                                             const CFX_ByteString& sData,
-                                             JSObject pData,
-                                             bool bDefaultPersistent) {
+FX_BOOL JSGlobalAlternate::SetGlobalVariables(const FX_CHAR* propname,
+                                              int nType,
+                                              double dData,
+                                              bool bData,
+                                              const CFX_ByteString& sData,
+                                              JSObject pData,
+                                              bool bDefaultPersistent) {
   if (propname == NULL)
     return FALSE;
 
-  js_global_data* pTemp = NULL;
+  JSGlobalData* pTemp = NULL;
   m_mapGlobal.Lookup(propname, (void*&)pTemp);
 
   if (pTemp) {
     if (pTemp->bDeleted || pTemp->nType != nType) {
       pTemp->dData = 0;
       pTemp->bData = 0;
       pTemp->sData = "";
       pTemp->nType = nType;
     }
 
@@ skipping 14 matching lines @@
       } break;
       case JS_GLOBALDATA_TYPE_NULL:
         break;
       default:
         return FALSE;
     }
 
     return TRUE;
   }
 
-  js_global_data* pNewData = NULL;
+  JSGlobalData* pNewData = NULL;
 
   switch (nType) {
     case JS_GLOBALDATA_TYPE_NUMBER: {
-      pNewData = new js_global_data;
+      pNewData = new JSGlobalData;
       pNewData->nType = JS_GLOBALDATA_TYPE_NUMBER;
       pNewData->dData = dData;
       pNewData->bPersistent = bDefaultPersistent;
     } break;
     case JS_GLOBALDATA_TYPE_BOOLEAN: {
-      pNewData = new js_global_data;
+      pNewData = new JSGlobalData;
       pNewData->nType = JS_GLOBALDATA_TYPE_BOOLEAN;
       pNewData->bData = bData;
       pNewData->bPersistent = bDefaultPersistent;
     } break;
     case JS_GLOBALDATA_TYPE_STRING: {
-      pNewData = new js_global_data;
+      pNewData = new JSGlobalData;
       pNewData->nType = JS_GLOBALDATA_TYPE_STRING;
       pNewData->sData = sData;
       pNewData->bPersistent = bDefaultPersistent;
     } break;
     case JS_GLOBALDATA_TYPE_OBJECT: {
-      pNewData = new js_global_data;
+      pNewData = new JSGlobalData;
       pNewData->nType = JS_GLOBALDATA_TYPE_OBJECT;
       pNewData->pData.Reset(JS_GetRuntime(pData), pData);
       pNewData->bPersistent = bDefaultPersistent;
     } break;
     case JS_GLOBALDATA_TYPE_NULL: {
-      pNewData = new js_global_data;
+      pNewData = new JSGlobalData;
       pNewData->nType = JS_GLOBALDATA_TYPE_NULL;
       pNewData->bPersistent = bDefaultPersistent;
     } break;
     default:
       return FALSE;
   }
 
   m_mapGlobal.SetAt(propname, (void*)pNewData);
 
   return TRUE;
@@ skipping 14 matching lines @@
     return VT_boolean;
   if (nHash == JSCONST_nDateHash)
     return VT_date;
   if (nHash == JSCONST_nObjectHash)
     return VT_object;
   if (nHash == JSCONST_nFXobjHash)
     return VT_fxobject;
 
   return VT_unknown;
 }