Validators improvements

src/ripple/validators/impl/Logic.h

@@ -223,6 +223,11 @@ public:
 
     struct State
     {
+        State ()
+        {
+            //sources.reserve (64);
+        }
+
         MapType map;
         SourcesType sources;
     };
@@ -266,20 +271,19 @@ public:
     //
     void addStatic (Source* source)
     {
-        m_journal.info << "Add static Source, " << source->name();
+        m_journal.info << "Addding static source '" << source->name() << "'";
 
         ScopedPointer <Source> object (source);
-
         Source::Result result (object->fetch (m_journal));
 
-        SharedState::Access state (m_state);
         if (result.success)
         {
-            merge (result.list, state);
+            SharedState::Access state (m_state);
+            merge (result.list, source, state);
         }
         else
         {
-            // VFALCO NOTE Maybe log the error and message?
+            // TODO: Report the error
         }
     }
 
@@ -287,18 +291,23 @@ public:
     //
     void add (Source* source)
     {
-        m_journal.info << "Add Source, " << source->name();
+        m_journal.info << "Adding source '" << source->name() << "'";
+
+        {
             SharedState::Access state (m_state);
             SourceDesc& desc (*state->sources.emplace_back ());
             desc.source = source;
             m_store.insert (desc);
         }
+    }
 
     // Add each entry in the list to the map, incrementing the
     // reference count if it already exists, and updating fields.
     //
-    void merge (Array <Source::Info> const& list, SharedState::Access& state)
+    void merge (Array <Source::Info> const& list,
+        Source* source, SharedState::Access& state)
     {
+        std::size_t numAdded (0);
         for (std::size_t i = 0; i < list.size (); ++i)
         {
             Source::Info const& info (list.getReference (i));
@@ -309,16 +318,22 @@ public:
             if (result.second)
             {
                 // This is a new one
+                ++numAdded;
                 dirtyChosen ();
             }
         }
+
+        m_journal.info << "Added " << numAdded
+                       << " trusted validators from '" << source->name() << "'";
     }
 
     // Decrement the reference count of each item in the list
     // in the map.
     //
-    void remove (Array <Source::Info> const& list, SharedState::Access& state)
+    void remove (Array <Source::Info> const& list,
+        Source* source, SharedState::Access& state)
     {
+        std::size_t numRemoved (0);
         for (std::size_t i = 0; i < list.size (); ++i)
         {
             Source::Info const& info (list.getReference (i));
@@ -328,10 +343,14 @@ public:
             if (--validatorInfo.refCount == 0)
             {
                 // Last reference removed
+                ++numRemoved;
                 state->map.erase (iter);
                 dirtyChosen ();
             }
         }
+
+        m_journal.info << "Removed " << numRemoved
+                       << " trusted validators from '" << source->name() << "'";
     }
 
     //----------------------------------------------------------------------
@@ -405,13 +424,13 @@ public:
             SharedState::Access state (m_state);
 
             // Add the new source info to the map
-            merge (result.list, state);
+            merge (result.list, desc.source, state);
 
             // Swap lists
             desc.result.swapWith (result);
 
             // Remove the old source info from the map
-            remove (result.list, state);
+            remove (result.list, desc.source, state);
 
             // See if we need to rebuild
             checkChosen ();
@@ -437,7 +456,7 @@ public:
     void expire (SourceDesc& desc, SharedState::Access& state)
     {
         // Decrement reference count on each validator
-        remove (desc.result.list, state);
+        remove (desc.result.list, desc.source, state);
 
         m_store.update (desc);
     }

src/ripple/validators/impl/Manager.cpp

@@ -140,44 +140,6 @@ public:
         stopThread ();
     }
 
-    //--------------------------------------------------------------------------
-    //
-    // Stoppable
-    //
-
-    void onPrepare (Journal journal)
-    {
-        journal.info << "Preparing";
-
-        addRPCHandlers();
-    }
-
-    void onStart (Journal journal)
-    {
-        journal.info << "Starting";
-
-        // Do this late so the sources have a chance to be added.
-        m_queue.dispatch (bind (&ManagerImp::setCheckSources, this));
-
-        startThread();
-    }
-
-    void onStop (Journal journal)
-    {
-        journal.info << "Stopping";
-
-        if (this->Thread::isThreadRunning())
-        {
-            m_journal.debug << "Signaling thread exit";
-            m_queue.dispatch (bind (&Thread::signalThreadShouldExit, this));
-        }
-        else
-        {
-            m_journal.debug << "Thread was never started";
-            stopped();
-        }
-    }
-
     //--------------------------------------------------------------------------
     //
     // RPC::Service
@@ -232,7 +194,8 @@ public:
 
     void addFile (File const& file)
     {
-        addStaticSource (SourceFile::New (file));
+        //addStaticSource (SourceFile::New (file));
+        addSource (SourceFile::New (file));
     }
 
     void addURL (URL const& url)
@@ -244,12 +207,20 @@ public:
 
     void addSource (Source* source)
     {
+#if RIPPLE_USE_NEW_VALIDATORS
         m_queue.dispatch (bind (&Logic::add, &m_logic, source));
+#else
+        delete source;
+#endif
     }
 
     void addStaticSource (Source* source)
     {
+#if RIPPLE_USE_NEW_VALIDATORS
         m_queue.dispatch (bind (&Logic::addStatic, &m_logic, source));
+#else
+        delete source;
+#endif
     }
 
     // VFALCO NOTE we should just do this on the callers thread?
@@ -274,6 +245,47 @@ public:
 #endif
     }
 
+    //--------------------------------------------------------------------------
+    //
+    // Stoppable
+    //
+
+    void onPrepare (Journal journal)
+    {
+#if RIPPLE_USE_NEW_VALIDATORS
+        journal.info << "Preparing";
+
+        addRPCHandlers();
+#endif
+    }
+
+    void onStart (Journal journal)
+    {
+#if RIPPLE_USE_NEW_VALIDATORS
+        journal.info << "Starting";
+
+        // Do this late so the sources have a chance to be added.
+        m_queue.dispatch (bind (&ManagerImp::setCheckSources, this));
+
+        startThread();
+#endif
+    }
+
+    void onStop (Journal journal)
+    {
+        journal.info << "Stopping";
+
+        if (this->Thread::isThreadRunning())
+        {
+            m_journal.debug << "Signaling thread exit";
+            m_queue.dispatch (bind (&Thread::signalThreadShouldExit, this));
+        }
+        else
+        {
+            stopped();
+        }
+    }
+
     //--------------------------------------------------------------------------
 
     void init ()

src/ripple/validators/impl/SourceFile.cpp

@@ -52,6 +52,33 @@ public:
     {
         Result result;
 
+        int64 const fileSize (m_file.getSize ());
+
+        if (fileSize != 0)
+        {
+            if (fileSize < std::numeric_limits<int32>::max())
+            {
+                MemoryBlock buffer (fileSize);
+                RandomAccessFile f;
+                RandomAccessFile::ByteCount amountRead;
+
+                f.open (m_file, RandomAccessFile::readOnly);
+                f.read (buffer.begin(), fileSize, &amountRead);
+
+                if (amountRead == fileSize)
+                {
+                }
+            }
+            else
+            {
+                // too big!
+            }
+        }
+        else
+        {
+            // file doesn't exist
+        }
+
         return result;
     }
 

src/ripple/validators/impl/Utilities.h

@@ -17,7 +17,6 @@
 */
 //==============================================================================
 
-
 #ifndef RIPPLE_VALIDATORS_UTILITIES_H_INCLUDED
 #define RIPPLE_VALIDATORS_UTILITIES_H_INCLUDED