Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor warning fixes #332

Merged
merged 1 commit into from
Nov 11, 2023
Merged

Minor warning fixes #332

merged 1 commit into from
Nov 11, 2023

Conversation

tuffnatty
Copy link
Member

@tuffnatty tuffnatty commented Nov 8, 2023

2023-11-11 18:38 UTC+0100 Phil Krylov (phil a t krylov.eu)
  * include/hbwinuni.h
  * src/vm/maindllp/dllpcode.c
    * In `HB_WINAPI_GETPROCADDRESS[T]`, cast the return value to `void *`
      to pacify further cast warnings, as the exact function signature
      is almost never a `FARPROC` anyway.
  * src/compiler/harbour.yyc
  * src/macro/macro.yyc
    * Added `HB_SYMBOL_UNUSED( yynerrs )` to pacify compiler warnings.
  * src/rtl/replic.c
    * Added a cast to a comparison to pacify compiler warning.
  * src/vm/classes.c
    ! Fixed outdated format specifier in a trace message.

@tuffnatty tuffnatty changed the title WIP: Minor fixes [WIP] Minor fixes Nov 8, 2023
@tuffnatty
Copy link
Member Author

tuffnatty commented Nov 8, 2023

ChangeLog entry: done.

  * include/hbwinuni.h
  * src/vm/maindllp/dllpcode.c
    * In `HB_WINAPI_GETPROCADDRESS[T]`, cast the return value to `void *`
      to pacify further cast warnings, as the exact function signature is almost
      never a `FARPROC` anyway.
  * src/compiler/harbour.yyc
  * src/macro/macro.yyc
    * Added `HB_SYMBOL_UNUSED( yynerrs )` to pacify compiler warnings.
  * src/rtl/replic.c
    * Added a cast to a comparison to pacify compiler warning.
  * src/vm/classes.c
    ! Fixed outdated format specifier in a trace message.
@tuffnatty tuffnatty changed the title [WIP] Minor fixes Minor warning fixes Nov 11, 2023
@tuffnatty tuffnatty merged commit d54bb0e into master Nov 11, 2023
7 of 44 checks passed
@tuffnatty tuffnatty deleted the minor-warnings branch November 11, 2023 18:39
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.

1 participant