src: remove unnecessary cast in crypto_sig.cc

tniessen · ruyadorno · commit 460397709be0 · 2022-08-22T19:37:09.000-04:00
ByteSource::Allocated accepts a void pointer now, so we do not need to cast the argument to a char pointer. Refs: #43202 PR-URL: #44001 Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Feng Yu <F3n67u@outlook.com>
diff --git a/src/crypto/crypto_sig.cc b/src/crypto/crypto_sig.cc
@@ -216,7 +216,7 @@ ByteSource ConvertSignatureToDER(
 
   CHECK_NOT_NULL(data);
 
-  return ByteSource::Allocated(reinterpret_cast<char*>(data), len);
+  return ByteSource::Allocated(data, len);
 }
 
 void CheckThrow(Environment* env, SignBase::Error error) {

Original file line number	Diff line number	Diff line change
`@@ -216,7 +216,7 @@ ByteSource ConvertSignatureToDER(`
`216`	`216`
`217`	`217`	`CHECK_NOT_NULL(data);`
`218`	`218`
`219`		`- return ByteSource::Allocated(reinterpret_cast<char*>(data), len);`
	`219`	`+ return ByteSource::Allocated(data, len);`
`220`	`220`	`}`
`221`	`221`
`222`	`222`	`void CheckThrow(Environment* env, SignBase::Error error) {`