Skip to content

Add missing static keywords.#434

Merged
flobernd merged 1 commit intozyantific:masterfrom
jpidancet:fix-compile-warnings
Apr 18, 2023
Merged

Add missing static keywords.#434
flobernd merged 1 commit intozyantific:masterfrom
jpidancet:fix-compile-warnings

Conversation

@jpidancet
Copy link
Copy Markdown
Contributor

Fix some warnings output by GCC when compiling amalgamated source:

Zydis.c:45353:12: warning: no previous declaration for ‘ZydisDecodeOperands’ [-Wmissing-declarations] 45353 | ZyanStatus ZydisDecodeOperands(const ZydisDecoder* decoder, const ZydisDecoderContext* context,
| ^~~~~~~~~~~~~~~~~~~
Zydis.c:52494:6: warning: no previous declaration for ‘ZydisBuildRegisterOperand’ [-Wmissing-declarations]
52494 | void ZydisBuildRegisterOperand(const ZydisEncoderOperand user_op,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:53639:6: warning: no previous declaration for ‘ZydisFormatterBufferInit’ [-Wmissing-declarations]
53639 | void ZydisFormatterBufferInit(ZydisFormatterBuffer
buffer, char* user_buffer,
| ^~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:53661:6: warning: no previous declaration for ‘ZydisFormatterBufferInitTokenized’ [-Wmissing-declarations]
53661 | void ZydisFormatterBufferInitTokenized(ZydisFormatterBuffer* buffer,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:54354:12: warning: no previous declaration for ‘ZydisStringAppendDecU64’ [-Wmissing-declarations]
54354 | ZyanStatus ZydisStringAppendDecU64(ZyanString* string, ZyanU64 value, ZyanU8 padding_length)
| ^~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:54475:12: warning: no previous declaration for ‘ZydisStringAppendHexU64’ [-Wmissing-declarations]
54475 | ZyanStatus ZydisStringAppendHexU64(ZyanString* string, ZyanU64 value, ZyanU8 padding_length,
| ^~~~~~~~~~~~~~~~~~~~~~~

Fix some warnings output by GCC when compiling amalgamated source:

Zydis.c:45353:12: warning: no previous declaration for ‘ZydisDecodeOperands’ [-Wmissing-declarations]
45353 | ZyanStatus ZydisDecodeOperands(const ZydisDecoder* decoder, const ZydisDecoderContext* context,
      |            ^~~~~~~~~~~~~~~~~~~
Zydis.c:52494:6: warning: no previous declaration for ‘ZydisBuildRegisterOperand’ [-Wmissing-declarations]
52494 | void ZydisBuildRegisterOperand(const ZydisEncoderOperand *user_op,
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:53639:6: warning: no previous declaration for ‘ZydisFormatterBufferInit’ [-Wmissing-declarations]
53639 | void ZydisFormatterBufferInit(ZydisFormatterBuffer* buffer, char* user_buffer,
      |      ^~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:53661:6: warning: no previous declaration for ‘ZydisFormatterBufferInitTokenized’ [-Wmissing-declarations]
53661 | void ZydisFormatterBufferInitTokenized(ZydisFormatterBuffer* buffer,
      |      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:54354:12: warning: no previous declaration for ‘ZydisStringAppendDecU64’ [-Wmissing-declarations]
54354 | ZyanStatus ZydisStringAppendDecU64(ZyanString* string, ZyanU64 value, ZyanU8 padding_length)
      |            ^~~~~~~~~~~~~~~~~~~~~~~
Zydis.c:54475:12: warning: no previous declaration for ‘ZydisStringAppendHexU64’ [-Wmissing-declarations]
54475 | ZyanStatus ZydisStringAppendHexU64(ZyanString* string, ZyanU64 value, ZyanU8 padding_length,
      |            ^~~~~~~~~~~~~~~~~~~~~~~
Copy link
Copy Markdown
Member

@flobernd flobernd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks.

@flobernd flobernd merged commit 80ccb8e into zyantific:master Apr 18, 2023
athre0z pushed a commit that referenced this pull request Feb 4, 2024
Fix some warnings output by GCC when compiling amalgamated source
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants