From 88135c7bb27557e214928918ca46ecb86c790a0b Mon Sep 17 00:00:00 2001 From: David Robillard Date: Thu, 1 Mar 2012 06:35:58 +0000 Subject: Rework forge sink interface to support buffer resizing (e.g. realloc). --- lv2/lv2plug.in/ns/ext/atom/atom-test.c | 65 ++++++++----- lv2/lv2plug.in/ns/ext/atom/forge.h | 171 ++++++++++++++++++--------------- 2 files changed, 134 insertions(+), 102 deletions(-) (limited to 'lv2') diff --git a/lv2/lv2plug.in/ns/ext/atom/atom-test.c b/lv2/lv2plug.in/ns/ext/atom/atom-test.c index 7e90c15..addad81 100644 --- a/lv2/lv2plug.in/ns/ext/atom/atom-test.c +++ b/lv2/lv2plug.in/ns/ext/atom/atom-test.c @@ -88,47 +88,53 @@ main() lv2_atom_forge_set_buffer(&forge, buf, BUF_SIZE); LV2_Atom_Forge_Frame obj_frame; - LV2_Atom* obj = (LV2_Atom*)lv2_atom_forge_resource( - &forge, &obj_frame, 0, eg_Object); + LV2_Atom* obj = lv2_atom_forge_deref( + &forge, lv2_atom_forge_resource( &forge, &obj_frame, 0, eg_Object)); // eg_one = (Int32)1 lv2_atom_forge_property_head(&forge, eg_one, 0); - LV2_Atom_Int32* one = lv2_atom_forge_int32(&forge, 1); + LV2_Atom_Int32* one = (LV2_Atom_Int32*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_int32(&forge, 1)); if (one->body != 1) { return test_fail("%d != 1\n", one->body); } // eg_two = (Int64)2 lv2_atom_forge_property_head(&forge, eg_two, 0); - LV2_Atom_Int64* two = lv2_atom_forge_int64(&forge, 2); + LV2_Atom_Int64* two = (LV2_Atom_Int64*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_int64(&forge, 2)); if (two->body != 2) { return test_fail("%ld != 2\n", two->body); } // eg_three = (Float)3.0 lv2_atom_forge_property_head(&forge, eg_three, 0); - LV2_Atom_Float* three = lv2_atom_forge_float(&forge, 3.0f); + LV2_Atom_Float* three = (LV2_Atom_Float*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_float(&forge, 3.0f)); if (three->body != 3) { return test_fail("%f != 3\n", three->body); } // eg_four = (Double)4.0 lv2_atom_forge_property_head(&forge, eg_four, 0); - LV2_Atom_Double* four = lv2_atom_forge_double(&forge, 4.0); + LV2_Atom_Double* four = (LV2_Atom_Double*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_double(&forge, 4.0)); if (four->body != 4) { return test_fail("%ld != 4\n", four->body); } // eg_true = (Bool)1 lv2_atom_forge_property_head(&forge, eg_true, 0); - LV2_Atom_Bool* t = lv2_atom_forge_bool(&forge, true); + LV2_Atom_Bool* t = (LV2_Atom_Bool*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_bool(&forge, true)); if (t->body != 1) { return test_fail("%ld != 1 (true)\n", t->body); } // eg_false = (Bool)0 lv2_atom_forge_property_head(&forge, eg_false, 0); - LV2_Atom_Bool* f = lv2_atom_forge_bool(&forge, false); + LV2_Atom_Bool* f = (LV2_Atom_Bool*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_bool(&forge, false)); if (f->body != 0) { return test_fail("%ld != 0 (false)\n", f->body); } @@ -137,7 +143,8 @@ main() const uint8_t* pstr = (const uint8_t*)"/foo/bar"; const size_t pstr_len = strlen((const char*)pstr); lv2_atom_forge_property_head(&forge, eg_path, 0); - LV2_Atom_String* path = lv2_atom_forge_uri(&forge, pstr, pstr_len); + LV2_Atom_String* path = (LV2_Atom_String*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_uri(&forge, pstr, pstr_len)); uint8_t* pbody = (uint8_t*)LV2_ATOM_BODY(path); if (strcmp((const char*)pbody, (const char*)pstr)) { return test_fail("%s != \"%s\"\n", @@ -148,8 +155,9 @@ main() const uint8_t* ustr = (const uint8_t*)"http://example.org/value"; const size_t ustr_len = strlen((const char*)ustr); lv2_atom_forge_property_head(&forge, eg_uri, 0); - LV2_Atom_String* uri = lv2_atom_forge_uri(&forge, ustr, ustr_len); - uint8_t* ubody = (uint8_t*)LV2_ATOM_BODY(uri); + LV2_Atom_String* uri = (LV2_Atom_String*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_uri(&forge, ustr, ustr_len)); + uint8_t* ubody = (uint8_t*)LV2_ATOM_BODY(uri); if (strcmp((const char*)ubody, (const char*)ustr)) { return test_fail("%s != \"%s\"\n", (const char*)ubody, (const char*)ustr); @@ -158,15 +166,17 @@ main() // eg_urid = (URID)"http://example.org/value" LV2_URID eg_value = urid_map(NULL, "http://example.org/value"); lv2_atom_forge_property_head(&forge, eg_urid, 0); - LV2_Atom_URID* urid = lv2_atom_forge_urid(&forge, eg_value); + LV2_Atom_URID* urid = (LV2_Atom_URID*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_urid(&forge, eg_value)); if (urid->body != eg_value) { return test_fail("%u != %u\n", urid->body, eg_value); } // eg_string = (String)"hello" lv2_atom_forge_property_head(&forge, eg_string, 0); - LV2_Atom_String* string = lv2_atom_forge_string( - &forge, (const uint8_t*)"hello", strlen("hello")); + LV2_Atom_String* string = (LV2_Atom_String*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_string( + &forge, (const uint8_t*)"hello", strlen("hello"))); uint8_t* sbody = (uint8_t*)LV2_ATOM_BODY(string); if (strcmp((const char*)sbody, "hello")) { return test_fail("%s != \"hello\"\n", (const char*)sbody); @@ -174,9 +184,10 @@ main() // eg_literal = (Literal)"hello"@fr lv2_atom_forge_property_head(&forge, eg_literal, 0); - LV2_Atom_Literal* literal = lv2_atom_forge_literal( - &forge, (const uint8_t*)"bonjour", strlen("bonjour"), - 0, urid_map(NULL, "http://lexvo.org/id/term/fr")); + LV2_Atom_Literal* literal = (LV2_Atom_Literal*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_literal( + &forge, (const uint8_t*)"bonjour", strlen("bonjour"), + 0, urid_map(NULL, "http://lexvo.org/id/term/fr"))); uint8_t* lbody = (uint8_t*)LV2_ATOM_CONTENTS(LV2_Atom_Literal, literal); if (strcmp((const char*)lbody, "bonjour")) { return test_fail("%s != \"bonjour\"\n", (const char*)lbody); @@ -185,11 +196,13 @@ main() // eg_tuple = "foo",true lv2_atom_forge_property_head(&forge, eg_tuple, 0); LV2_Atom_Forge_Frame tuple_frame; - LV2_Atom_Tuple* tuple = (LV2_Atom_Tuple*)lv2_atom_forge_tuple( - &forge, &tuple_frame); - LV2_Atom_String* tup0 = lv2_atom_forge_string( - &forge, (const uint8_t*)"foo", strlen("foo")); - LV2_Atom_Bool* tup1 = lv2_atom_forge_bool(&forge, true); + LV2_Atom_Tuple* tuple = (LV2_Atom_Tuple*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_tuple(&forge, &tuple_frame)); + LV2_Atom_String* tup0 = (LV2_Atom_String*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_string( + &forge, (const uint8_t*)"foo", strlen("foo"))); + LV2_Atom_Bool* tup1 = (LV2_Atom_Bool*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_bool(&forge, true)); lv2_atom_forge_pop(&forge, &tuple_frame); LV2_Atom_Tuple_Iter i = lv2_tuple_begin(tuple); if (lv2_tuple_is_end(tuple, i)) { @@ -215,8 +228,9 @@ main() // eg_vector = (Vector)1,2,3,4 lv2_atom_forge_property_head(&forge, eg_vector, 0); int32_t elems[] = { 1, 2, 3, 4 }; - LV2_Atom_Vector* vector = lv2_atom_forge_vector( - &forge, 4, forge.Int32, sizeof(int32_t), elems); + LV2_Atom_Vector* vector = (LV2_Atom_Vector*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_vector( + &forge, 4, forge.Int32, sizeof(int32_t), elems));; void* vec_body = LV2_ATOM_CONTENTS(LV2_Atom_Vector, vector); if (memcmp(elems, vec_body, sizeof(elems))) { return test_fail("Corrupt vector\n"); @@ -225,7 +239,8 @@ main() // eg_seq = (Sequence)1, 2 lv2_atom_forge_property_head(&forge, eg_seq, 0); LV2_Atom_Forge_Frame seq_frame; - LV2_Atom_Sequence* seq = (LV2_Atom_Sequence*)lv2_atom_forge_sequence_head(&forge, &seq_frame, 0); + LV2_Atom_Sequence* seq = (LV2_Atom_Sequence*)lv2_atom_forge_deref( + &forge, lv2_atom_forge_sequence_head(&forge, &seq_frame, 0)); lv2_atom_forge_frame_time(&forge, 0); lv2_atom_forge_int32(&forge, 1); lv2_atom_forge_frame_time(&forge, 1); diff --git a/lv2/lv2plug.in/ns/ext/atom/forge.h b/lv2/lv2plug.in/ns/ext/atom/forge.h index f55cb08..b3a5e4d 100644 --- a/lv2/lv2plug.in/ns/ext/atom/forge.h +++ b/lv2/lv2plug.in/ns/ext/atom/forge.h @@ -57,15 +57,21 @@ extern "C" { /** Handle for LV2_Atom_Forge_Sink. */ typedef void* LV2_Atom_Forge_Sink_Handle; +/** A reference to a chunk of written output. */ +typedef intptr_t LV2_Atom_Forge_Ref; + /** Sink function for writing output. See lv2_atom_forge_set_sink(). */ -typedef void* (*LV2_Atom_Forge_Sink)(LV2_Atom_Forge_Sink_Handle handle, - const void* buf, - uint32_t size); +typedef LV2_Atom_Forge_Ref (*LV2_Atom_Forge_Sink)( + LV2_Atom_Forge_Sink_Handle handle, const void* buf, uint32_t size); + +/** Function for resolving a reference. See lv2_atom_forge_set_sink(). */ +typedef void* (*LV2_Atom_Forge_Deref_Func)( + LV2_Atom_Forge_Sink_Handle handle, LV2_Atom_Forge_Ref ref); /** A stack frame used for keeping track of nested Atom containers. */ typedef struct _LV2_Atom_Forge_Frame { struct _LV2_Atom_Forge_Frame* parent; - LV2_Atom* atom; + LV2_Atom_Forge_Ref ref; } LV2_Atom_Forge_Frame; /** A "forge" for creating atoms by appending to a buffer. */ @@ -75,6 +81,7 @@ typedef struct { uint32_t size; LV2_Atom_Forge_Sink sink; + LV2_Atom_Forge_Deref_Func deref; LV2_Atom_Forge_Sink_Handle handle; LV2_Atom_Forge_Frame* stack; @@ -103,15 +110,15 @@ typedef struct { pointer), but may be called by the user to push the top level container when writing to an existing Atom. */ -static inline LV2_Atom* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_push(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame, - LV2_Atom* atom) + LV2_Atom_Forge_Ref ref) { frame->parent = forge->stack; - frame->atom = atom; + frame->ref = ref; forge->stack = frame; - return atom; + return ref; } /** Pop a stack frame. This must be called when a container is finished. */ @@ -129,6 +136,7 @@ lv2_atom_forge_set_buffer(LV2_Atom_Forge* forge, uint8_t* buf, size_t size) forge->buf = buf; forge->size = size; forge->offset = 0; + forge->deref = NULL; forge->sink = NULL; forge->handle = NULL; forge->stack = NULL; @@ -149,10 +157,12 @@ lv2_atom_forge_set_buffer(LV2_Atom_Forge* forge, uint8_t* buf, size_t size) static inline void lv2_atom_forge_set_sink(LV2_Atom_Forge* forge, LV2_Atom_Forge_Sink sink, + LV2_Atom_Forge_Deref_Func deref, LV2_Atom_Forge_Sink_Handle handle) { forge->buf = NULL; forge->size = forge->offset = 0; + forge->deref = deref; forge->sink = sink; forge->handle = handle; forge->stack = NULL; @@ -187,29 +197,38 @@ lv2_atom_forge_init(LV2_Atom_Forge* forge, LV2_URID_Map* map) forge->Vector = map->map(map->handle, LV2_ATOM_URI "#Vector"); } +static inline LV2_Atom* +lv2_atom_forge_deref(LV2_Atom_Forge* forge, LV2_Atom_Forge_Ref ref) +{ + if (forge->buf) { + return (LV2_Atom*)(forge->buf + ref); + } else { + return forge->deref(forge->handle, ref); + } +} + /** Write raw output. This is used internally, but is also useful for writing atom types not explicitly supported by the forge API. Note the caller is responsible for ensuring the output is approriately padded. */ -static inline void* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_raw(LV2_Atom_Forge* forge, const void* data, uint32_t size) { - uint8_t* out = NULL; + LV2_Atom_Forge_Ref out = 0; if (forge->sink) { - out = (uint8_t*)forge->sink(forge->handle, data, size); + out = forge->sink(forge->handle, data, size); } else { - out = forge->buf + forge->offset; + out = forge->offset; + uint8_t* mem = forge->buf + forge->offset; if (forge->offset + size > forge->size) { - return NULL; + return 0; } forge->offset += size; - memcpy(out, data, size); + memcpy(mem, data, size); } - if (out) { - for (LV2_Atom_Forge_Frame* f = forge->stack; f; f = f->parent) { - f->atom->size += size; - } + for (LV2_Atom_Forge_Frame* f = forge->stack; f; f = f->parent) { + lv2_atom_forge_deref(forge, f->ref)->size += size; } return out; } @@ -224,10 +243,10 @@ lv2_atom_forge_pad(LV2_Atom_Forge* forge, uint32_t written) } /** Write raw output, padding to 64-bits as necessary. */ -static inline void* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_write(LV2_Atom_Forge* forge, const void* data, uint32_t size) { - void* out = lv2_atom_forge_raw(forge, data, size); + LV2_Atom_Forge_Ref out = lv2_atom_forge_raw(forge, data, size); if (out) { lv2_atom_forge_pad(forge, size); } @@ -235,97 +254,96 @@ lv2_atom_forge_write(LV2_Atom_Forge* forge, const void* data, uint32_t size) } /** Write an atom:Atom header. */ -static inline LV2_Atom* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_atom(LV2_Atom_Forge* forge, uint32_t size, uint32_t type) { const LV2_Atom a = { size, type }; - return (LV2_Atom*)lv2_atom_forge_raw(forge, &a, sizeof(a)); + return lv2_atom_forge_raw(forge, &a, sizeof(a)); } /** Write an atom:Int32. */ -static inline LV2_Atom_Int32* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_int32(LV2_Atom_Forge* forge, int32_t val) { const LV2_Atom_Int32 a = { { sizeof(val), forge->Int32 }, val }; - return (LV2_Atom_Int32*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write an atom:Int64. */ -static inline LV2_Atom_Int64* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_int64(LV2_Atom_Forge* forge, int64_t val) { const LV2_Atom_Int64 a = { { sizeof(val), forge->Int64 }, val }; - return (LV2_Atom_Int64*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write an atom:Float. */ -static inline LV2_Atom_Float* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_float(LV2_Atom_Forge* forge, float val) { const LV2_Atom_Float a = { { sizeof(val), forge->Float }, val }; - return (LV2_Atom_Float*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write an atom:Double. */ -static inline LV2_Atom_Double* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_double(LV2_Atom_Forge* forge, double val) { const LV2_Atom_Double a = { { sizeof(val), forge->Double }, val }; - return (LV2_Atom_Double*)lv2_atom_forge_write( - forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write an atom:Bool. */ -static inline LV2_Atom_Bool* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_bool(LV2_Atom_Forge* forge, bool val) { const LV2_Atom_Bool a = { { sizeof(val), forge->Bool }, val }; - return (LV2_Atom_Bool*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write an atom:URID. */ -static inline LV2_Atom_URID* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_urid(LV2_Atom_Forge* forge, LV2_URID id) { const LV2_Atom_URID a = { { sizeof(id), forge->URID }, id }; - return (LV2_Atom_URID*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write a string body. Used internally. */ -static inline uint8_t* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_string_body(LV2_Atom_Forge* forge, const uint8_t* str, uint32_t len) { - void* out = NULL; + LV2_Atom_Forge_Ref out = 0; if ( (out = lv2_atom_forge_raw(forge, str, len)) && (out = lv2_atom_forge_raw(forge, "", 1))) { lv2_atom_forge_pad(forge, len + 1); } - return (uint8_t*)out; + return out; } /** Write an atom compatible with atom:String. Used internally. */ -static inline LV2_Atom_String* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_typed_string(LV2_Atom_Forge* forge, uint32_t type, const uint8_t* str, uint32_t len) { - const LV2_Atom_String a = { { len + 1, type } }; - LV2_Atom_String* out = (LV2_Atom_String*) - lv2_atom_forge_raw(forge, &a, sizeof(a)); + const LV2_Atom_String a = { { len + 1, type } }; + LV2_Atom_Forge_Ref out = lv2_atom_forge_raw(forge, &a, sizeof(a)); if (out) { if (!lv2_atom_forge_string_body(forge, str, len)) { - out->atom.size = out->atom.type = 0; - out = NULL; + LV2_Atom* atom = lv2_atom_forge_deref(forge, out); + atom->size = atom->type = 0; + out = 0; } } return out; } /** Write an atom:String. Note that @p str need not be NULL terminated. */ -static inline LV2_Atom_String* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_string(LV2_Atom_Forge* forge, const uint8_t* str, uint32_t len) { return lv2_atom_forge_typed_string(forge, forge->String, str, len); @@ -336,21 +354,21 @@ lv2_atom_forge_string(LV2_Atom_Forge* forge, const uint8_t* str, uint32_t len) This does not map the URI, but writes the complete URI string. To write a mapped URI, use lv2_atom_forge_urid(). */ -static inline LV2_Atom_String* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_uri(LV2_Atom_Forge* forge, const uint8_t* uri, uint32_t len) { return lv2_atom_forge_typed_string(forge, forge->URI, uri, len); } /** Write an atom:Path. Note that @p path need not be NULL terminated. */ -static inline LV2_Atom_String* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_path(LV2_Atom_Forge* forge, const uint8_t* path, uint32_t len) { return lv2_atom_forge_typed_string(forge, forge->Path, path, len); } /** Write an atom:Literal. */ -static inline LV2_Atom_Literal* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_literal(LV2_Atom_Forge* forge, const uint8_t* str, uint32_t len, @@ -363,19 +381,19 @@ lv2_atom_forge_literal(LV2_Atom_Forge* forge, { datatype, lang } }; - LV2_Atom_Literal* out = (LV2_Atom_Literal*) - lv2_atom_forge_raw(forge, &a, sizeof(a)); + LV2_Atom_Forge_Ref out = lv2_atom_forge_raw(forge, &a, sizeof(a)); if (out) { if (!lv2_atom_forge_string_body(forge, str, len)) { - out->atom.size = out->atom.type = 0; - out = NULL; + LV2_Atom* atom = lv2_atom_forge_deref(forge, out); + atom->size = atom->type = 0; + out = 0; } } return out; } /** Write an atom:Vector header, but not the vector body. */ -static inline LV2_Atom_Vector* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_vector_head(LV2_Atom_Forge* forge, uint32_t elem_count, uint32_t elem_type, @@ -386,18 +404,18 @@ lv2_atom_forge_vector_head(LV2_Atom_Forge* forge, { size - sizeof(LV2_Atom), forge->Vector }, { elem_count, elem_type } }; - return (LV2_Atom_Vector*)lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_write(forge, &a, sizeof(a)); } /** Write a complete atom:Vector. */ -static inline LV2_Atom_Vector* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_vector(LV2_Atom_Forge* forge, uint32_t elem_count, uint32_t elem_type, uint32_t elem_size, - void* elems) + const void* elems) { - LV2_Atom_Vector* out = lv2_atom_forge_vector_head( + LV2_Atom_Forge_Ref out = lv2_atom_forge_vector_head( forge, elem_count, elem_type, elem_size); if (out) { lv2_atom_forge_write(forge, elems, elem_size * elem_count); @@ -422,12 +440,12 @@ lv2_atom_forge_vector(LV2_Atom_Forge* forge, lv2_atom_forge_pop(forge, &frame); @endcode */ -static inline LV2_Atom_Tuple* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_tuple(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame) { - const LV2_Atom_Tuple a = { { 0, forge->Tuple } }; - LV2_Atom* atom = (LV2_Atom*)lv2_atom_forge_write(forge, &a, sizeof(a)); - return (LV2_Atom_Tuple*)lv2_atom_forge_push(forge, frame, atom); + const LV2_Atom_Tuple a = { { 0, forge->Tuple } }; + LV2_Atom_Forge_Ref out = lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_push(forge, frame, out); } /** @@ -454,7 +472,7 @@ lv2_atom_forge_tuple(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame) lv2_atom_forge_pop(forge, &frame); @endcode */ -static inline LV2_Atom_Object* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_resource(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame, LV2_URID id, @@ -464,14 +482,14 @@ lv2_atom_forge_resource(LV2_Atom_Forge* forge, { sizeof(LV2_Atom_Object) - sizeof(LV2_Atom), forge->Resource }, { id, otype } }; - LV2_Atom* atom = (LV2_Atom*)lv2_atom_forge_write(forge, &a, sizeof(a)); - return (LV2_Atom_Object*)lv2_atom_forge_push(forge, frame, atom); + LV2_Atom_Forge_Ref out = lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_push(forge, frame, out); } /** The same as lv2_atom_forge_resource(), but for object:Blank. */ -static inline LV2_Atom_Object* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_blank(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame, uint32_t id, @@ -481,22 +499,21 @@ lv2_atom_forge_blank(LV2_Atom_Forge* forge, { sizeof(LV2_Atom_Object) - sizeof(LV2_Atom), forge->Blank }, { id, otype } }; - LV2_Atom* atom = (LV2_Atom*)lv2_atom_forge_write(forge, &a, sizeof(a)); - return (LV2_Atom_Object*)lv2_atom_forge_push(forge, frame, atom); + LV2_Atom_Forge_Ref out = lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_push(forge, frame, out); } /** Write the header for a property body (likely in an Object). See lv2_atom_forge_object() documentation for an example. */ -static inline LV2_Atom_Property_Body* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_property_head(LV2_Atom_Forge* forge, LV2_URID key, LV2_URID context) { const LV2_Atom_Property_Body a = { key, context, { 0, 0 } }; - return (LV2_Atom_Property_Body*)lv2_atom_forge_write( - forge, &a, 2 * sizeof(uint32_t)); + return lv2_atom_forge_write(forge, &a, 2 * sizeof(uint32_t)); } /** @@ -504,7 +521,7 @@ lv2_atom_forge_property_head(LV2_Atom_Forge* forge, The size of the returned sequence will be 0, so passing it as the parent parameter to other forge methods will do the right thing. */ -static inline LV2_Atom_Sequence* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_sequence_head(LV2_Atom_Forge* forge, LV2_Atom_Forge_Frame* frame, uint32_t unit) @@ -513,8 +530,8 @@ lv2_atom_forge_sequence_head(LV2_Atom_Forge* forge, { sizeof(LV2_Atom_Sequence) - sizeof(LV2_Atom), forge->Sequence }, { unit, 0 } }; - LV2_Atom* atom = (LV2_Atom*)lv2_atom_forge_write(forge, &a, sizeof(a)); - return (LV2_Atom_Sequence*)lv2_atom_forge_push(forge, frame, atom); + LV2_Atom_Forge_Ref out = lv2_atom_forge_write(forge, &a, sizeof(a)); + return lv2_atom_forge_push(forge, frame, out); } /** @@ -522,10 +539,10 @@ lv2_atom_forge_sequence_head(LV2_Atom_Forge* forge, After this, call the appropriate forge method(s) to write the body, passing the same @p parent parameter. Note the returned LV2_Event is NOT an Atom. */ -static inline int64_t* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_frame_time(LV2_Atom_Forge* forge, int64_t frames) { - return (int64_t*)lv2_atom_forge_write(forge, &frames, sizeof(frames)); + return lv2_atom_forge_write(forge, &frames, sizeof(frames)); } /** @@ -533,10 +550,10 @@ lv2_atom_forge_frame_time(LV2_Atom_Forge* forge, int64_t frames) After this, call the appropriate forge method(s) to write the body, passing the same @p parent parameter. Note the returned LV2_Event is NOT an Atom. */ -static inline double* +static inline LV2_Atom_Forge_Ref lv2_atom_forge_beat_time(LV2_Atom_Forge* forge, double beats) { - return (double*)lv2_atom_forge_write(forge, &beats, sizeof(beats)); + return lv2_atom_forge_write(forge, &beats, sizeof(beats)); } #ifdef __cplusplus -- cgit v1.2.1