Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit5f2f99c

Browse files
committed
Remove unnecessary casts
Some code carefully cast all data buffer arguments for data write andread function calls to void *, even though the respective argumentsare already void *. Remove this unnecessary clutter.Discussion:https://www.postgresql.org/message-id/flat/11dda853-bb5b-59ba-a746-e168b1ce4bdb%40enterprisedb.com
1 parent388e801 commit5f2f99c

File tree

5 files changed

+23
-23
lines changed

5 files changed

+23
-23
lines changed

‎src/backend/executor/nodeAgg.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2961,10 +2961,10 @@ hashagg_spill_tuple(AggState *aggstate, HashAggSpill *spill,
29612961

29622962
tape=spill->partitions[partition];
29632963

2964-
LogicalTapeWrite(tape,(void*)&hash,sizeof(uint32));
2964+
LogicalTapeWrite(tape,&hash,sizeof(uint32));
29652965
total_written+=sizeof(uint32);
29662966

2967-
LogicalTapeWrite(tape,(void*)tuple,tuple->t_len);
2967+
LogicalTapeWrite(tape,tuple,tuple->t_len);
29682968
total_written+=tuple->t_len;
29692969

29702970
if (shouldFree)
@@ -3029,7 +3029,7 @@ hashagg_batch_read(HashAggBatch *batch, uint32 *hashp)
30293029
tuple->t_len=t_len;
30303030

30313031
nread=LogicalTapeRead(tape,
3032-
(void*) ((char*)tuple+sizeof(uint32)),
3032+
(char*)tuple+sizeof(uint32),
30333033
t_len-sizeof(uint32));
30343034
if (nread!=t_len-sizeof(uint32))
30353035
ereport(ERROR,

‎src/backend/storage/file/buffile.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -607,7 +607,7 @@ BufFileRead(BufFile *file, void *ptr, size_t size)
607607
memcpy(ptr,file->buffer.data+file->pos,nthistime);
608608

609609
file->pos+=nthistime;
610-
ptr= (void*) ((char*)ptr+nthistime);
610+
ptr= (char*)ptr+nthistime;
611611
size-=nthistime;
612612
nread+=nthistime;
613613
}
@@ -655,7 +655,7 @@ BufFileWrite(BufFile *file, void *ptr, size_t size)
655655
file->pos+=nthistime;
656656
if (file->nbytes<file->pos)
657657
file->nbytes=file->pos;
658-
ptr= (void*) ((char*)ptr+nthistime);
658+
ptr= (char*)ptr+nthistime;
659659
size-=nthistime;
660660
}
661661
}

‎src/backend/utils/sort/logtape.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -319,7 +319,7 @@ ltsReadFillBuffer(LogicalTape *lt)
319319
datablocknum+=lt->offsetBlockNumber;
320320

321321
/* Read the block */
322-
ltsReadBlock(lt->tapeSet,datablocknum,(void*)thisbuf);
322+
ltsReadBlock(lt->tapeSet,datablocknum,thisbuf);
323323
if (!lt->frozen)
324324
ltsReleaseBlock(lt->tapeSet,datablocknum);
325325
lt->curBlockNumber=lt->nextBlockNumber;
@@ -806,7 +806,7 @@ LogicalTapeWrite(LogicalTape *lt, void *ptr, size_t size)
806806

807807
/* set the next-pointer and dump the current block. */
808808
TapeBlockGetTrailer(lt->buffer)->next=nextBlockNumber;
809-
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,(void*)lt->buffer);
809+
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,lt->buffer);
810810

811811
/* initialize the prev-pointer of the next block */
812812
TapeBlockGetTrailer(lt->buffer)->prev=lt->curBlockNumber;
@@ -826,7 +826,7 @@ LogicalTapeWrite(LogicalTape *lt, void *ptr, size_t size)
826826
lt->pos+=nthistime;
827827
if (lt->nbytes<lt->pos)
828828
lt->nbytes=lt->pos;
829-
ptr= (void*) ((char*)ptr+nthistime);
829+
ptr= (char*)ptr+nthistime;
830830
size-=nthistime;
831831
}
832832
}
@@ -888,7 +888,7 @@ LogicalTapeRewindForRead(LogicalTape *lt, size_t buffer_size)
888888
lt->buffer_size-lt->nbytes);
889889

890890
TapeBlockSetNBytes(lt->buffer,lt->nbytes);
891-
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,(void*)lt->buffer);
891+
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,lt->buffer);
892892
}
893893
lt->writing= false;
894894
}
@@ -953,7 +953,7 @@ LogicalTapeRead(LogicalTape *lt, void *ptr, size_t size)
953953
memcpy(ptr,lt->buffer+lt->pos,nthistime);
954954

955955
lt->pos+=nthistime;
956-
ptr= (void*) ((char*)ptr+nthistime);
956+
ptr= (char*)ptr+nthistime;
957957
size-=nthistime;
958958
nread+=nthistime;
959959
}
@@ -1004,7 +1004,7 @@ LogicalTapeFreeze(LogicalTape *lt, TapeShare *share)
10041004
lt->buffer_size-lt->nbytes);
10051005

10061006
TapeBlockSetNBytes(lt->buffer,lt->nbytes);
1007-
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,(void*)lt->buffer);
1007+
ltsWriteBlock(lt->tapeSet,lt->curBlockNumber,lt->buffer);
10081008
}
10091009
lt->writing= false;
10101010
lt->frozen= true;
@@ -1031,7 +1031,7 @@ LogicalTapeFreeze(LogicalTape *lt, TapeShare *share)
10311031

10321032
if (lt->firstBlockNumber==-1L)
10331033
lt->nextBlockNumber=-1L;
1034-
ltsReadBlock(lt->tapeSet,lt->curBlockNumber,(void*)lt->buffer);
1034+
ltsReadBlock(lt->tapeSet,lt->curBlockNumber,lt->buffer);
10351035
if (TapeBlockIsLast(lt->buffer))
10361036
lt->nextBlockNumber=-1L;
10371037
else
@@ -1098,7 +1098,7 @@ LogicalTapeBackspace(LogicalTape *lt, size_t size)
10981098
returnseekpos;
10991099
}
11001100

1101-
ltsReadBlock(lt->tapeSet,prev,(void*)lt->buffer);
1101+
ltsReadBlock(lt->tapeSet,prev,lt->buffer);
11021102

11031103
if (TapeBlockGetTrailer(lt->buffer)->next!=lt->curBlockNumber)
11041104
elog(ERROR,"broken tape, next of block %ld is %ld, expected %ld",
@@ -1142,7 +1142,7 @@ LogicalTapeSeek(LogicalTape *lt, long blocknum, int offset)
11421142

11431143
if (blocknum!=lt->curBlockNumber)
11441144
{
1145-
ltsReadBlock(lt->tapeSet,blocknum,(void*)lt->buffer);
1145+
ltsReadBlock(lt->tapeSet,blocknum,lt->buffer);
11461146
lt->curBlockNumber=blocknum;
11471147
lt->nbytes=TapeBlockPayloadSize;
11481148
lt->nextBlockNumber=TapeBlockGetTrailer(lt->buffer)->next;

‎src/backend/utils/sort/tuplesort.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2908,7 +2908,7 @@ markrunend(LogicalTape *tape)
29082908
{
29092909
unsignedintlen=0;
29102910

2911-
LogicalTapeWrite(tape,(void*)&len,sizeof(len));
2911+
LogicalTapeWrite(tape,&len,sizeof(len));
29122912
}
29132913

29142914
/*

‎src/backend/utils/sort/tuplesortvariants.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1002,10 +1002,10 @@ writetup_heap(Tuplesortstate *state, LogicalTape *tape, SortTuple *stup)
10021002
/* total on-disk footprint: */
10031003
unsignedinttuplen=tupbodylen+sizeof(int);
10041004

1005-
LogicalTapeWrite(tape,(void*)&tuplen,sizeof(tuplen));
1006-
LogicalTapeWrite(tape,(void*)tupbody,tupbodylen);
1005+
LogicalTapeWrite(tape,&tuplen,sizeof(tuplen));
1006+
LogicalTapeWrite(tape,tupbody,tupbodylen);
10071007
if (base->sortopt&TUPLESORT_RANDOMACCESS)/* need trailing length word? */
1008-
LogicalTapeWrite(tape,(void*)&tuplen,sizeof(tuplen));
1008+
LogicalTapeWrite(tape,&tuplen,sizeof(tuplen));
10091009
}
10101010

10111011
staticvoid
@@ -1475,10 +1475,10 @@ writetup_index(Tuplesortstate *state, LogicalTape *tape, SortTuple *stup)
14751475
unsignedinttuplen;
14761476

14771477
tuplen=IndexTupleSize(tuple)+sizeof(tuplen);
1478-
LogicalTapeWrite(tape,(void*)&tuplen,sizeof(tuplen));
1479-
LogicalTapeWrite(tape,(void*)tuple,IndexTupleSize(tuple));
1478+
LogicalTapeWrite(tape,&tuplen,sizeof(tuplen));
1479+
LogicalTapeWrite(tape,tuple,IndexTupleSize(tuple));
14801480
if (base->sortopt&TUPLESORT_RANDOMACCESS)/* need trailing length word? */
1481-
LogicalTapeWrite(tape,(void*)&tuplen,sizeof(tuplen));
1481+
LogicalTapeWrite(tape,&tuplen,sizeof(tuplen));
14821482
}
14831483

14841484
staticvoid
@@ -1564,10 +1564,10 @@ writetup_datum(Tuplesortstate *state, LogicalTape *tape, SortTuple *stup)
15641564

15651565
writtenlen=tuplen+sizeof(unsignedint);
15661566

1567-
LogicalTapeWrite(tape,(void*)&writtenlen,sizeof(writtenlen));
1567+
LogicalTapeWrite(tape,&writtenlen,sizeof(writtenlen));
15681568
LogicalTapeWrite(tape,waddr,tuplen);
15691569
if (base->sortopt&TUPLESORT_RANDOMACCESS)/* need trailing length word? */
1570-
LogicalTapeWrite(tape,(void*)&writtenlen,sizeof(writtenlen));
1570+
LogicalTapeWrite(tape,&writtenlen,sizeof(writtenlen));
15711571
}
15721572

15731573
staticvoid

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp