hare: gen: Copy unions with gen_copy_aligned v1 NEEDS REVISION

Lassi Pulkkinen: 1
 gen: Copy unions with gen_copy_aligned

 1 files changed, 1 insertions(+), 3 deletions(-)

[PATCH hare] gen: Copy unions with gen_copy_aligned Export this patch

This appears to be historical oversight.

Signed-off-by: Lassi Pulkkinen <lassi@pulk.fi>
---
 src/gen.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/gen.c b/src/gen.c
index e739500..768ea34 100644
--- a/src/gen.c
+++ b/src/gen.c
@@ -118,14 +118,12 @@ gen_store(struct gen_context *ctx,
 	case STORAGE_SLICE:
 	case STORAGE_STRING:
 	case STORAGE_STRUCT:
+	case STORAGE_UNION:
 	case STORAGE_TAGGED:
 	case STORAGE_TUPLE:
 	case STORAGE_VALIST:
 		gen_copy_aligned(ctx, object, value);
 		return;
-	case STORAGE_UNION:
-		gen_copy_memcpy(ctx, object, value);
-		return;
 	case STORAGE_ENUM:
 		object.type = ty->alias.type;
 		break;
-- 
2.38.1

builds.sr.ht <builds@sr.ht>

a month ago

hare/patches: FAILED in 53s

[gen: Copy unions with gen_copy_aligned][0] from [Lassi Pulkkinen][1]

[0]: https://lists.sr.ht/~sircmpwn/hare-dev/patches/38662
[1]: mailto:lassi@pulk.fi

✗ #933174 FAILED hare/patches/freebsd.yml https://builds.sr.ht/~sircmpwn/job/933174
✗ #933173 FAILED hare/patches/alpine.yml  https://builds.sr.ht/~sircmpwn/job/933173

Drew DeVault <sir@cmpwn.com>

12 days ago

Needs rebase, and the subject prefix is wrong (should be PATCH harec)

Lassi Pulkkinen <lassi@pulk.fi>

11 days ago

Eh, I noticed that immediately after submission and posted v2, which has
already been applied.