diff options
| author | Nick Mathewson <nickm@torproject.org> | 2017-08-08 15:54:15 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-09-05 13:57:51 -0400 |
| commit | 6ec50597231c8d72fbf796eee88e02dd9a4d0b78 (patch) | |
| tree | 24001fe0eb99beb6cbaa0c2768f1df1fd4595949 /src/test/test_socks.c | |
| parent | d61da9e61fa0ea15789464c7c3754a9af30fcbb4 (diff) | |
| download | tor-6ec50597231c8d72fbf796eee88e02dd9a4d0b78.tar.gz tor-6ec50597231c8d72fbf796eee88e02dd9a4d0b78.zip | |
Refactor buffer APIs to put a buf_t first.
By convention, a function that frobs a foo_t should be called
foo_frob, and it should have a foo_t * as its first argument. But
for many of the buf_t functions, the buf_t was the final argument,
which is silly.
Diffstat (limited to 'src/test/test_socks.c')
| -rw-r--r-- | src/test/test_socks.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/test_socks.c b/src/test/test_socks.c index 0791db9a18..6dc8215a95 100644 --- a/src/test/test_socks.c +++ b/src/test/test_socks.c @@ -44,7 +44,7 @@ static const struct testcase_setup_t socks_setup = { buf_t *buf = testdata->buf; \ socks_request_t *socks = testdata->req; #define ADD_DATA(buf, s) \ - buf_add(s, sizeof(s)-1, buf) + buf_add(buf, s, sizeof(s)-1) static void socks_request_clear(socks_request_t *socks) |