Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(31)

Issue 3484041: Ns-3 SO_TYPE (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 4 months ago by Josh Pelkey
Modified:
13 years, 2 months ago
Reviewers:
Tom Henderson
CC:
ns-3-reviews_googlegroups.com
Visibility:
Public.

Description

Implemented Socket::GetSockType() (analogous to getsockopt(SO_TYPE) in ns-3. ipv4-raw-socket, ipv6-raw-socket, and packet-socket return SOCK_RAW tcp-socket and nsc-tcp-socket return SOCK_STREAM udp-socket returns SOCK_DGRAM

Patch Set 1 #

Patch Set 2 : Remove logging from getters #

Unified diffs Side-by-side diffs Delta from patch set Stats (+58 lines, -0 lines) Patch
M src/internet-stack/ipv4-raw-socket-impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/internet-stack/ipv4-raw-socket-impl.cc View 1 1 chunk +7 lines, -0 lines 0 comments Download
M src/internet-stack/ipv6-raw-socket-impl.h View 1 chunk +6 lines, -0 lines 0 comments Download
M src/internet-stack/ipv6-raw-socket-impl.cc View 1 1 chunk +5 lines, -0 lines 0 comments Download
M src/internet-stack/nsc-tcp-socket-impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/internet-stack/nsc-tcp-socket-impl.cc View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/internet-stack/tcp-socket-impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/internet-stack/tcp-socket-impl.cc View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/internet-stack/udp-socket-impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/internet-stack/udp-socket-impl.cc View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/node/packet-socket.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/node/packet-socket.cc View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/node/socket.h View 2 chunks +11 lines, -0 lines 0 comments Download

Messages

Total messages: 2
Tom Henderson
This looks fine to me; just one minor comment, to replace NS_LOG_FUNCTION_NOARGS() with NS_LOG_FUNCTION (this). ...
13 years, 4 months ago (2010-12-07 05:53:01 UTC) #1
Josh Pelkey
13 years, 4 months ago (2010-12-07 21:04:33 UTC) #2
I removed logging from the getters.

On 2010/12/07 05:53:01, Tom Henderson wrote:
> This looks fine to me; just one minor comment, to replace
> NS_LOG_FUNCTION_NOARGS() with NS_LOG_FUNCTION (this).  The former is usually
> reserved for static functions.  I would be also fine with not having these
> getters return function log calls but I see that the rest of the
setters/getters
> return them.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b