undef/git-commit-vandalism
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

git-tar-tree: no more void pointer arithmetic

Browse Source 
Noticed by Florian Forster: Use a char pointer when adding offsets,
because void pointer arithmetic is a GNU extension.   Const'ify the
function arguments while we're at it.

Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx>
Signed-off-by: Junio C Hamano <junkio@cox.net>
This commit is contained in:
Rene Scharfe 2006-06-18 12:58:01 +02:00 committed by Junio C Hamano
parent 9236cdd488
commit 6698060c15
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
builtin-tar-tree.c
View File

@ -22,8 +22,10 @@ static unsigned long offset;
static time_t archive_time; static time_t archive_time;
/* tries hard to write, either succeeds or dies in the attempt */ /* tries hard to write, either succeeds or dies in the attempt */
static void reliable_write(void *buf, unsigned long size) static void reliable_write(const void *data, unsigned long size)
{ {
const char *buf = data;
while (size > 0) { while (size > 0) {
long ret = xwrite(1, buf, size); long ret = xwrite(1, buf, size);
if (ret < 0) { if (ret < 0) {
@ -51,8 +53,9 @@ static void write_if_needed(void)
* queues up writes, so that all our write(2) calls write exactly one * queues up writes, so that all our write(2) calls write exactly one
* full block; pads writes to RECORDSIZE * full block; pads writes to RECORDSIZE
*/ */
static void write_blocked(void *buf, unsigned long size) static void write_blocked(const void *data, unsigned long size)
{ {
const char *buf = data;
unsigned long tail; unsigned long tail;
if (offset) { if (offset) {