Move PRISM_NODISCARD to the correct position

Author: kddnewton

include/prism/arena.h

@@ -25,7 +25,7 @@ typedef struct pm_arena_t pm_arena_t;
  *     the caller to free the arena using pm_arena_free when it is no longer
  *     needed.
  */
-PRISM_EXPORTED_FUNCTION pm_arena_t * pm_arena_new(void) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_arena_t * pm_arena_new(void);
 
 /**
  * Frees both the held memory and the arena itself.

include/prism/buffer.h

@@ -24,7 +24,7 @@ typedef struct pm_buffer_t pm_buffer_t;
  * @returns A pointer to the initialized buffer. The caller is responsible for
  *     freeing the buffer with pm_buffer_free.
  */
-PRISM_EXPORTED_FUNCTION pm_buffer_t * pm_buffer_new(void) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_buffer_t * pm_buffer_new(void);
 
 /**
  * Free both the memory held by the buffer and the buffer itself.

include/prism/options.h

@@ -114,7 +114,7 @@ static const uint8_t PM_OPTIONS_COMMAND_LINE_X = 0x20;
  * @returns A new options struct with default values. It is the responsibility
  *     of the caller to free this struct using pm_options_free().
  */
-PRISM_EXPORTED_FUNCTION pm_options_t * pm_options_new(void) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_options_t * pm_options_new(void);
 
 /**
  * Free both the held memory of the given options struct and the struct itself.

include/prism/parser.h

@@ -33,7 +33,7 @@ typedef struct pm_parser_t pm_parser_t;
  * @returns The initialized parser. It is the responsibility of the caller to
  *     free the parser with `pm_parser_free()`.
  */
-PRISM_EXPORTED_FUNCTION pm_parser_t * pm_parser_new(pm_arena_t *arena, const uint8_t *source, size_t size, const pm_options_t *options) PRISM_NODISCARD PRISM_NONNULL(1);
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_parser_t * pm_parser_new(pm_arena_t *arena, const uint8_t *source, size_t size, const pm_options_t *options) PRISM_NONNULL(1);
 
 /**
  * Free both the memory held by the given parser and the parser itself.

include/prism/source.h

@@ -66,7 +66,7 @@ typedef enum {
  * @param length The length of the source data in bytes.
  * @returns A new source. Aborts on allocation failure.
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_constant_new(const uint8_t *data, size_t length) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_constant_new(const uint8_t *data, size_t length);
 
 /**
  * Create a new source that wraps existing shared memory. The memory is not
@@ -76,7 +76,7 @@ PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_constant_new(const uint8_t *data
  * @param length The length of the source data in bytes.
  * @returns A new source. Aborts on allocation failure.
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_shared_new(const uint8_t *data, size_t length) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_shared_new(const uint8_t *data, size_t length);
 
 /**
  * Create a new source that owns its memory. The memory will be freed with
@@ -86,7 +86,7 @@ PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_shared_new(const uint8_t *data,
  * @param length The length of the source data in bytes.
  * @returns A new source. Aborts on allocation failure.
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_owned_new(uint8_t *data, size_t length) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_owned_new(uint8_t *data, size_t length);
 
 /**
  * Create a new source by reading a file into a heap-allocated buffer.
@@ -95,7 +95,7 @@ PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_owned_new(uint8_t *data, size_t
  * @param result Out parameter for the result of the initialization.
  * @returns A new source, or NULL on error (with result written to out param).
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_file_new(const char *filepath, pm_source_init_result_t *result) PRISM_NODISCARD PRISM_NONNULL(1, 2);
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_file_new(const char *filepath, pm_source_init_result_t *result) PRISM_NONNULL(1, 2);
 
 /**
  * Create a new source by memory-mapping a file. Falls back to file reading on
@@ -110,7 +110,7 @@ PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_file_new(const char *filepath, p
  * @param result Out parameter for the result of the initialization.
  * @returns A new source, or NULL on error (with result written to out param).
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_mapped_new(const char *filepath, int open_flags, pm_source_init_result_t *result) PRISM_NODISCARD PRISM_NONNULL(1, 3);
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_mapped_new(const char *filepath, int open_flags, pm_source_init_result_t *result) PRISM_NONNULL(1, 3);
 
 /**
  * Create a new source by reading from a stream using the provided callbacks.
@@ -120,7 +120,7 @@ PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_mapped_new(const char *filepath,
  * @param feof The function to use to check if the stream is at EOF.
  * @returns A new source. Aborts on allocation failure.
  */
-PRISM_EXPORTED_FUNCTION pm_source_t * pm_source_stream_new(void *stream, pm_source_stream_fgets_t *fgets, pm_source_stream_feof_t *feof) PRISM_NODISCARD;
+PRISM_EXPORTED_FUNCTION PRISM_NODISCARD pm_source_t * pm_source_stream_new(void *stream, pm_source_stream_fgets_t *fgets, pm_source_stream_feof_t *feof);
 
 /**
  * Free the given source and any memory it owns.