Skip to content
This repository was archived by the owner on Apr 16, 2018. It is now read-only.

Develop #21

Open
wants to merge 111 commits into
base: master
Choose a base branch
from
Open

Develop #21

wants to merge 111 commits into from

Conversation

agauniyal
Copy link
Member

[WIP] - Do Not Merge

agauniyal and others added 30 commits February 9, 2017 16:03
@agauniyal
Add .vscode directory to .gitignore
e310a70
@agauniyal
Add partial socket class
04f61cf
@agauniyal
Make net::methods inline for static lib builds
ea7ab39
@agauniyal
Fix bug - message size < buffer size in read method
522ed7c
@agauniyal
Add safeguards to socket.read() method
6fb7e2b
@agauniyal
Add send and recv methods to Socket
4591d38
@agauniyal
Add sendto/recvfrom overloads for send/recv methods
f033e23
@agauniyal
Refactoring - constexpr functions and pass by ref
13ce468
@agauniyal
Change sockaddr member to unique_ptr<sockaddr>
55099c7
@tkhurana96
Add getErrorMsg() function to net::method
3e1c339
@agauniyal
Add -Wstrict-aliasing global flag
f7d39e4
@tkhurana96
Make lockguard and use it to manage lockToErrorString
ccfce2c
@tkhurana96
Merge branch 'feature/socket' of https://github.com/c10k/net into fea…
5234896 
…ture/socket
@tkhurana96
Make getErrorMsg() inline
5ae9114
@tkhurana96 @agauniyal
Refactor error handling, add initial support for AF_UNIX (#2)
f3d9d33 
* Try to complete all TODO's

* Remove static_cast from errno comparisons

* Add code for capturing current errno value in currentErrNo variable

* Try to handle error in non-blocking sockets using bool *_isErrorInNonBlocking in read and recv

* Make getErrorMsg func arg const

* Rename isErrorInNonBlocking to isSpecialErrorInNonBlocking

* Refactor error handling, add initial support for AF_UNIX
@agauniyal
Refactor bind, connect and send methods to take callables
b92d623
@agauniyal
Make most methods protocol independent
586997d
@agauniyal
Fix wrong return type inside decltype()
da38118
@agauniyal
Add corresponding methods for getsockopt/setsockopt
38a3c3e
@agauniyal
Merge pull request #3 from c10k/feature/socket-options
0e6cfa3 
Feature/socket options
@tkhurana96
Try to add documentation in socket.hpp
5bc5d98
@tkhurana96
Add clang formatted doc and remove all 'Improve This' comments
433a7dd
@tkhurana96
Add documentation of functions in net::methods inside socket_family.hpp
39e3e5b
@agauniyal
Organize documentation comments, change addrIpv* to addrIPv*
3c70d97
@agauniyal
Merge pull request #4 from c10k/documentation
d7a8a4b 
Add documentation to the project
@tkhurana96
Try to add some constructor testing of Socket
365cdd3
@tkhurana96
Add test for UDP unix socket
fcc9220
@tkhurana96
Add some basic bind tests which needs revision
eda83a6
@tkhurana96
Try to complete the testing of Socket constructor and bind
813d36c
@tkhurana96
Rename test files for constructor and bind tests
8341dc7
@agauniyal agauniyal self-assigned this Apr 1, 2017
@codecov
Copy link

codecov bot commented Apr 1, 2017 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@f5f216a). Click here to learn what that means.
The diff coverage is 87.14%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #21   +/-   ##
=========================================
  Coverage          ?   87.14%           
=========================================
  Files             ?       12           
  Lines             ?     1198           
  Branches          ?      103           
=========================================
  Hits              ?     1044           
  Misses            ?      154           
  Partials          ?        0
Impacted Files Coverage Δ
test/socket_bind_test.cpp 100% <100%> (ø)
include/socket_options.hpp 100% <100%> (ø)
test/socket_constructor_test.cpp 100% <100%> (ø)
include/socket.hpp 74.87% <74.87%> (ø)
src/socket.cpp 77.07% <77.07%> (ø)
test/socket_connect_test.cpp 87.5% <87.5%> (ø)
test/socket_recv_test.cpp 87.67% <87.67%> (ø)
test/socket_getSocket_test.cpp 89.47% <89.47%> (ø)
test/socket_read_write_test.cpp 91.89% <91.89%> (ø)
test/socket_send_test.cpp 93.02% <93.02%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5f216a...8f74a5d. Read the comment docs.

agauniyal and others added 17 commits April 2, 2017 23:38
@agauniyal
Fix move constructor & private constructor bugs
a232689
@agauniyal
Merge pull request #22 from c10k/fix/minor-bugs
6b2d18a 
Fix move constructor & private constructor bugs
@agauniyal
Reap Zombie child processes in tcp_fork server example
393fe47
@agauniyal
Merge pull request #23 from c10k/fix/zombie-processes
afb25d1 
Reap Zombie child processes in tcp_fork server example
@agauniyal
Update README with examples
73540fb
@agauniyal
Fix wercker builds
19a866b
@agauniyal
Revert "Fix wercker builds"
84d319f 
This reverts commit 19a866b.
@agauniyal
Install locales before building
a178d2f
@agauniyal
Fix builds - gcc6 broken
87aeaaf
@agauniyal
Add gcc7 compiler to builds
50abf24
@agauniyal
Merge pull request #24 from c10k/add/documentation
8ded159 
Update README with examples
@agauniyal
Add TCP_NODELAY and TCP_MAXSEG options
8a8e8cf
@agauniyal
Fix socket_options_test from failing due to TCP_MAX_SEG
aa7af58
@agauniyal
Merge pull request #25 from c10k/feature/tcp-socket-options
f6c074e 
Add TCP_NODELAY and TCP_MAXSEG options
@agauniyal
Add autogenerated docs
fe6c381
@agauniyal
Remove concurrent threaded example from README
f9c8c6f
@agauniyal
Reformat source files with new clang-format
8f74a5d
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@agauniyal agauniyal

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@agauniyal @tkhurana96