A few old cosmetic changes

libowlps/libowlps.c

@@ -83,6 +83,7 @@ void owl_mac_bytes_to_string_r(const uint8_t *const mac_binary,
 bool owl_mac_equals(const uint8_t *const mac1,
                     const uint8_t *const mac2)
 {
+  // TODO: we could use memcmp()
   int i ;
   for (i = ETHER_ADDR_LEN - 1 ; i >= 0 ; --i)
     if(mac1[i] != mac2[i])

owlps-positioner/request.cc

@@ -257,7 +257,7 @@ ostream& operator<<(ostream &os, const Request &r)
      << " (received at " << r.time_received << "; " ;
 
   if (r.real_position)
-    os << " Real coordinates : " << *r.real_position << "; " ;
+    os << "Real coordinates: " << *r.real_position << "; " ;
 
   // MAC address
   os << "Type: " << static_cast<uint_fast16_t>(r.type)

owlps-positioner/stock.hh

@@ -86,7 +86,7 @@ public:
   //@{
   /// Returns the number of buildings
   static unsigned int nb_buildings(void) ;
-  /// Reads the Building corresponding to a given name
+  /// Returns the Building corresponding to a given name
   static const Building& get_building(const std::string &name) ;
   /// Searches for a Building and creates it if it does not exist
   static const Building& find_create_building(const std::string &name) ;
@@ -117,7 +117,7 @@ public:
   static unsigned int nb_mobiles(void) ;
   /// Verify the existence of a mobile
   static bool mobile_exists(const std::string &mac) ;
-  /// Reads the Mobile corresponding to a given MAC address
+  /// Returns the Mobile corresponding to a given MAC address
   static const Mobile& get_mobile(const std::string &mac) ;
   /// Searches for a Mobile given its MAC address and creates it if it
   /// does not exist
@@ -138,7 +138,7 @@ public:
   std::unordered_map<std::string, CapturePoint>& get_cps(void) ;
   /// Verify the existence of a CP
   static bool cp_exists(const std::string &mac) ;
-  /// Reads the CapturePoint corresponding to a given MAC address
+  /// Returns the CapturePoint corresponding to a given MAC address
   static const CapturePoint& get_cp(const std::string &mac) ;
   /// Searches for a CapturePoint given its MAC address and creates it
   /// if it does not exist
@@ -166,7 +166,7 @@ public:
   static unsigned int nb_reference_points(void) ;
   /// Verify the existence of a reference point at the given coordinates
   static bool reference_point_exists(const ReferencePoint &point) ;
-  /// Reads the ReferencePoint at the given coordinates
+  /// Returns the ReferencePoint at the given coordinates
   static const ReferencePoint&
   get_reference_point(const ReferencePoint &point) ;
   /// Searches for a ReferencePoint and adds it if it does not exist

scripts/owlps-aggsetcoord.pl

@@ -33,8 +33,8 @@ By default, B<owlps-aggsetcoord> uses the coordinates already present in the
 input file (manually set by the user, most of the time) to interpolate the
 coordinates of lines containing default values. A given coordinate is
 interpolated according to the previous and next known coordinates and their
-timestamps, assuming the mobile moves at a constant speed between two
-timestamps. The interpolation is done in 3 dimensions, each dimension being
+timestamps, assuming the mobile moves at a constant speed between two known
+coordinates. The interpolation is done in 3 dimensions, each dimension being
 interpolated independently.
 
 Alternatively, using the B<-c> option will disable interpolation but apply an