regcomp.c: Rmv C undefined behavior

One analyzer said that what this commit changes was C undefined
behavior, in casting void* pointers.  Right now, the only actual type it
is called with is SV*, but I made it void*, because I thought it might
be used more generally.  But, it turns out that Unicode is planning on
changing its regular expression processing requirements to where what I
have no longer will make sense.  And, since only SV* is actually used,
this commit changes the void* to SV*, removing any undefined behavior,
with no changes to program logic.

The changes for the new Unicode direction will come in probably 5.34;
their document is still in draft, but I anticipate it will soon be
finalized.

1 files changed, 3 insertions, 4 deletions

diff --git a/regcomp.h b/regcomp.h
index 2eb4a4f73a..9634e94e9a 100644
--- a/regcomp.h
+++ b/regcomp.h
@@ -151,14 +151,13 @@ struct regnode_1 {
     U32 arg1;
 };
 
-/* Node whose argument is 'void *', a pointer to void.  This needs to be used
- * very carefully in situations where pointers won't become invalid because of,
- * say re-mallocs */
+/* Node whose argument is 'SV *'.  This needs to be used very carefully in
+ * situations where pointers won't become invalid because of, say re-mallocs */
 struct regnode_p {
     U8	flags;
     U8  type;
     U16 next_off;
-    void * arg1;
+    SV * arg1;
 };
 
 /* Similar to a regnode_1 but with an extra signed argument */