diff --git a/src/core/interface.h b/src/core/interface.h
index be5ace7a9..307152a9d 100644
--- a/src/core/interface.h
+++ b/src/core/interface.h
@@ -80,7 +80,7 @@ protected:
   Interface (InterfaceId iid);
   /**
    * \param iid the Interface id of the interface
-   * \param a pointer to the interface object
+   * \param interface a pointer to the interface object
    *
    * If you are not a direct subclass of the ns3::Interface base class,
    * and if you want to register yourself as another accessible interface
diff --git a/src/core/random-variable.h b/src/core/random-variable.h
index b5076ca5f..9901b665a 100644
--- a/src/core/random-variable.h
+++ b/src/core/random-variable.h
@@ -200,6 +200,7 @@ protected:
  * UniformVariable x(0,10);
  * x.GetValue();  //will always return numbers [0,10]
  * UniformVariable::GetSingleValue(100,1000); //returns a value [100,1000]
+ * \endcode
  */
 class UniformVariable : public RandomVariable {
 public:
diff --git a/src/node/net-device.h b/src/node/net-device.h
index 92289d5e0..b5e45860b 100644
--- a/src/node/net-device.h
+++ b/src/node/net-device.h
@@ -111,7 +111,7 @@ public:
   /**
    * \param index ifIndex of the device 
    */
-  void SetIfIndex(const uint32_t);
+  void SetIfIndex(const uint32_t index);
   /**
    * \return index ifIndex of the device 
    */
diff --git a/src/node/socket.h b/src/node/socket.h
index 82769b405..aacafc8f1 100644
--- a/src/node/socket.h
+++ b/src/node/socket.h
@@ -206,24 +206,24 @@ public:
   
   /**
    * \brief Receive data
-   * \param Received data callback. Invoked whenever new data is received.
+   * \param receivedData Invoked whenever new data is received.
    *
    * If you wish to transport only dummy packets, this method is not a very
    * efficient way to receive these dummy packets: it will trigger a memory
    * allocation to hold the dummy memory into a buffer which can be passed
    * to the user. Instead, consider using the RecvDummy method.
    */
-  void Recv(Callback<void, Ptr<Socket>, const uint8_t*, uint32_t,const Ipv4Address&, uint16_t> = 
+  void Recv(Callback<void, Ptr<Socket>, const uint8_t*, uint32_t,const Ipv4Address&, uint16_t> receivedData = 
             MakeCallback (&Socket::DummyCallbackVoidSocketBufferUi32Ipv4AddressUi16));
   
   /**
    * \brief Receive data
-   * \param Received data callback. Invoked whenever new data is received.
+   * \param receivedData Invoked whenever new data is received.
    *
    * This method is included because it is vastly more efficient than the 
    * Recv method when you use dummy payload.
    */
-  void RecvDummy(Callback<void, Ptr<Socket>, uint32_t,const Ipv4Address&, uint16_t> = 
+  void RecvDummy(Callback<void, Ptr<Socket>, uint32_t,const Ipv4Address&, uint16_t> receivedData =
                  MakeCallback (&Socket::DummyCallbackVoidSocketUi32Ipv4AddressUi16));
 
 private: