diff options
author | Noel Grandin <noel@peralex.com> | 2016-02-02 16:15:51 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2016-02-08 06:08:14 +0000 |
commit | b14224fe97b8a44232c9c1401d3a49771f46582e (patch) | |
tree | 8f9cf31cf4b51a0edbb43022499a6acd17d0945d /framework | |
parent | c474e610e453d0f38f7cc6cb9559ad7e7b5d69ca (diff) |
loplugin:unusedmethods
using an idea from dtardon:
<dtardon> noelgrandin, hi. could you try to run the unusedmethods clang
plugin with "make build-nocheck"? that would catch functions that are
only used in tests. e.g., i just removed the whole o3tl::range class,
which has not been used in many years, but htere was a test for it...
<noelgrandin> dtardon, interesting idea! Sure, I can do that.
Change-Id: I5653953a426a2186a1e43017212d87ffce520387
Reviewed-on: https://gerrit.libreoffice.org/22041
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'framework')
-rw-r--r-- | framework/inc/classes/checkediterator.hxx | 35 |
1 files changed, 0 insertions, 35 deletions
diff --git a/framework/inc/classes/checkediterator.hxx b/framework/inc/classes/checkediterator.hxx index 2f9834e80409..4674e8bf0e97 100644 --- a/framework/inc/classes/checkediterator.hxx +++ b/framework/inc/classes/checkediterator.hxx @@ -68,41 +68,6 @@ class CheckedIterator // interface methods - /*-**************************************************************************************************** - @short step to next element in container. - @descr If end of container is reached we change our internal "m_eEndState". - If end reached for first time; we set it to E_END; - If you step to next element again; we set it to E_AFTEREND. - So you have a chance to differ between "exact end" and "after end"! - - @return A reference to our changed object himself. - *//*-*****************************************************************************************************/ - - inline CheckedIterator& operator++() - { - // Warn programmer if he forget to initialize object! - SAL_WARN_IF( m_pContainer==nullptr, "fwk", "CheckedIterator::operator++(): Object not initialized!" ); - // Step to next element if any exist or set our end states. - switch( m_eEndState ) - { - case E_BEFOREEND: { - ++m_pPosition; - // If iterator reaching end ... set right state! - if( m_pPosition == m_pContainer->end() ) - { - m_eEndState = E_END; - } - } - break; - case E_END : { - // Set state only ... iterator already points to end of container! - m_eEndState = E_AFTEREND; - } - break; - } - return *this; - } - // private member private: |