Fix cppcheck warnings, part 2
This commit is contained in:
dokutan
parent
107e90a2c7
commit
98bb1fb58c
11 changed files with 19 additions and 21 deletions
|
|
@ -117,7 +117,7 @@ int mouse_generic::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m709::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m711::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m715::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m719::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m908::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m913::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m990::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -117,7 +117,7 @@ int mouse_m990chroma::get_macro( int number, std::string& macro ){
|
|||
return 0;
|
||||
|
||||
std::vector< uint8_t > macro_bytes;
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), macro_bytes.begin() );
|
||||
std::copy( _s_macro_data[number-1].begin(), _s_macro_data[number-1].end(), std::back_inserter(macro_bytes) );
|
||||
|
||||
_i_decode_macro( macro_bytes, output, "", 8 );
|
||||
macro = output.str();
|
||||
|
|
|
|||
|
|
@ -323,10 +323,6 @@ int rd_mouse::_i_decode_macro( const std::vector< uint8_t >& macro_bytes, std::o
|
|||
|
||||
bool unknown_code = false;
|
||||
|
||||
// failsafe, i sometimes gets incremented by >1 which could lead to it being out of bounds
|
||||
if( i >= macro_bytes.size() )
|
||||
break;
|
||||
|
||||
// mouse buttons ( 0x81 = down, 0x01 = up )
|
||||
if( macro_bytes[i] == 0x81 && macro_bytes[i+1] == 0x01 )
|
||||
output << prefix << "down\tmouse_left\n";
|
||||
|
|
|
|||
|
|
@ -21,19 +21,21 @@
|
|||
#define RD_MOUSE
|
||||
|
||||
#include <libusb.h>
|
||||
#include <map>
|
||||
#include <array>
|
||||
#include <string>
|
||||
|
||||
#include <algorithm>
|
||||
#include <array>
|
||||
#include <exception>
|
||||
#include <regex>
|
||||
#include <fstream>
|
||||
#include <sstream>
|
||||
#include <utility>
|
||||
#include <iostream>
|
||||
#include <iomanip>
|
||||
#include <string>
|
||||
#include <iostream>
|
||||
#include <iterator>
|
||||
#include <map>
|
||||
#include <regex>
|
||||
#include <set>
|
||||
#include <sstream>
|
||||
#include <string>
|
||||
#include <string>
|
||||
#include <utility>
|
||||
#include <variant>
|
||||
|
||||
/* These declarations exist to make it possible for mouse_variant
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue