forked from n64decomp/mk64
-
Notifications
You must be signed in to change notification settings - Fork 1
Applid Clang format on the branch #4
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
Open
github-actions
wants to merge
50
commits into
patch-1
Choose a base branch
from
clang-format
base: patch-1
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
66c9c3a to
cda9de7
Compare
* Update clang-pr.yml * Update clang-pr.yml * Update clang-pr.yml --------- Co-authored-by: MegaMech <[email protected]>
Co-authored-by: MegaMech <[email protected]> Co-authored-by: MegaMech <[email protected]>
* Update code_80091750.c * Delete asm/non_matchings/code_80091750/func_800A66A8.s * Update code_80091750.c * Update code_80091750.c * Update code_80091750.c
* Clang Format * Update code_80091750.c --------- Co-authored-by: MegaMech <[email protected]> Co-authored-by: MegaMech <[email protected]>
* Fix color text array and function * seg2 header
* Label menus.c and small documentation * remove test * save.h * BACK_OUT -> GO_BACK
* some rename and allow change the course engine * fix comment * add more comment * rename and change description
Co-authored-by: MegaMech <[email protected]>
* remove useless yaml file * update format --------- Co-authored-by: MegaMech <[email protected]>
cda9de7 to
4182a1a
Compare
* some rename * type menu * continue rename and place const * Renames and doc * more * buffer vars * clang format * continue rename * some rename and get it compile (but don't match for obscure reason) * get it match again and fix some warning * fix all warning * Match 6 functions * continue rename * load img renames and first step or label menu items * MenuItem struct names and some rename * rename around * rename code_80091750 to menu_item * clean up a little * Update menu_item.h * document waypoint and more * Revert "document waypoint and more" This reverts commit 5e6eb4f. * apply review * Update menu_item.c * fix some review * rename func_8009E620 to clear_menus * do a rename * apply review * finish rename normally * more small rename * few fix of name * few more fix * minor fix * Update menus.c * last rename normally * Update render_menus.s * Update menu_items.c * Renames seg2 textures.c * MENU_ITEMtype --> MENU_ITEM_TYPE * remove a useless parameter * func_80099110 --> clear_menu_textures --------- Co-authored-by: MegaMech <[email protected]> Co-authored-by: AloXado320 <[email protected]>
* Document code_80005FD0.c
* match func_80099AEC and fix some warnings
Co-authored-by: Sonic Dreamcaster <[email protected]>
* couple of matches * match func_800C4FE4 * match func_800C284C * match func_800C2A2C * match func_800C3724 * match func_800C4398 * format * format2 * remove asms
4182a1a to
80a991a
Compare
* Commit unpr'd matches * fix compile
* Commit unpr'd matches * fix compile * Commit matching func_80086074
* square is a bad company
80a991a to
9469f6d
Compare
* Fix EU build and eu-final offsets * Fix instrument align and bring EU matches * Fix EU trig table segment * Label audio_init * Simplify float comments * Proper EU name version defines
* Commit unpr'd matches * fix compile * Commit matching func_80086074 * More matches * Fix missed labelling * menus.c formatting * More matches * Match update_player_path_completion * Formatting * Match func_80050E34 * Match EU Audio init * Match func_800C16E8 * Match synthesis_process_note * Match func_80009B60 * Match * Match cpu_use_item_strategy * Match render_menus * match last func
Co-authored-by: MegaMech <[email protected]>
n64decomp#710 changed the environment variables for compiling EU roms
9469f6d to
5bbf0e5
Compare
* fix more name * Update cpu_vehicles_camera_path.c * more fix of name --------- Co-authored-by: MegaMech <[email protected]>
…ndling etc.) (n64decomp#720) * Document various functions and variables * Fix code changes * Rename function based on discussion * Identify global table of player speeds * Improve names of functions * Label further things related to playerIndex * Label further balloon related things * Document further balloon related things * Use STAR_DURATION * Update vars * Fix missing arg
* Document alpha, movement * Fix typo * Add missing ~
* Document projection matrix function Document function for computing a projection matrix. Used for camera and graphics. * Fix loop index bug Fix straightforward loop index bug introduced in last commit * Style guide updates Changes to match style recommendations * Update function variables to camelCase * Use dOxygen tags * Use block comment --------- Co-authored-by: MegaMech <[email protected]>
* Shiftability fix and documentation * Allow setting `AVOID_UB=1` directly while building * Set `AVOID_UB` when building with DEBUG. * Document options for enabling shiftability * Document `make clean` * Delete HTML file * Update compiling.md --------- Co-authored-by: MegaMech <[email protected]>
* Labeling particle related code * More labeling. Naming pass on what was labeled last commit * Stick X clamped with deadzone * Renaming TyreParticle to Particle * Matching coco's names and renaming some already labeled to keep style consistent * Minor extra particle labeling * Small extra labeling. Restoring some removed newlines * One last newline adjustment --------- Co-authored-by: MegaMech <[email protected]>
Co-authored-by: MegaMech <[email protected]>
* document vertex * Update render_player.c
* Document projection matrix function Document function for computing a projection matrix. Used for camera and graphics. * Fix loop index bug Fix straightforward loop index bug introduced in last commit * Style guide updates Changes to match style recommendations * Update function variables to camelCase * Use dOxygen tags * Use block comment * Document math functions * Give all functions in src\racing\math_util.c descriptive names * Improve documentation for math_util Improves a number of functions by giving descriptive argument names and comments * Additional merge * Fix style issues update variables / parameters to use camelCase instead of snakeCase * doxygen and other documentation * Add doxygen documentation * Rename mat -> mtx for consistency among matrices * Theta -> Angle for angles * Give some arguments more descriptive names * Partial documentation for calculate_orientation_matrix Documentation for when cosAxisY is 1 and observations that things break when if it does not equal 1. * Rename functions Renamed multiple functions for accuracy or clarity * doxygen whitespace alignment Improve readability of doxygen info by aligning text * Make format check Formatting updates suggested by make format * Make skyboxes static * Change function name angle_from_coords -> get_angle_between_coords * Math values renamed Functions: render_distance_squared -> distance_if_visible mtxf_rotation_zxy_translate -> mtxf_rotate_zxy_translate Arguments: vecTrans -> translate --------- Co-authored-by: MegaMech <[email protected]>
* Document time trial input replay system Documents how the input replay system works in time trials. This is used for * Course ghosts (on the raceways) * Player ghosts * Replays * Rename ghosts/replays and minor cleanup Refer to staff and player ghosts in time trials as replays. Refer to the basic replay as a "post time trial replay" Change variable / function names to reflect terminology change * Define replay magic values * Rename staff_ghosts files to replays Renames staff_ghosts files to replays, since they also deal with player ghosts and post time trial replays. Also, changes staff_ghosts_loop -> replays_loop * REPLAY_NOT_FRAME_COUNTER -> REPLAY_CLEAR_FRAME_COUNTER * Add replays header and format * Update replays.c * Name load replay functions --------- Co-authored-by: MegaMech <[email protected]>
* Update README.md * Update README.md * Update README.md * Update README.md --------- Co-authored-by: MegaMech <[email protected]>
5bbf0e5 to
5a7c404
Compare
* Update torch * Update torch * Update torch * Update torch
5a7c404 to
79a294a
Compare
79a294a to
ede7dfa
Compare
* Update render_courses.c * More fixes
* Naming sound effects * Update sound effect names and categorize * Add bomb effect and fix bug * HIT_FAKE_ITEM -> EXPLOSION_CRASH * Rename sound_effects to triggers --------- Co-authored-by: MegaMech <[email protected]>
This reverts commit e88f33f.
* Document projection matrix function Document function for computing a projection matrix. Used for camera and graphics. * Fix loop index bug Fix straightforward loop index bug introduced in last commit * Style guide updates Changes to match style recommendations * Update function variables to camelCase * Use dOxygen tags * Use block comment * Document math functions * Give all functions in src\racing\math_util.c descriptive names * Improve documentation for math_util Improves a number of functions by giving descriptive argument names and comments * Additional merge * Fix style issues update variables / parameters to use camelCase instead of snakeCase * doxygen and other documentation * Add doxygen documentation * Rename mat -> mtx for consistency among matrices * Theta -> Angle for angles * Give some arguments more descriptive names * Partial documentation for calculate_orientation_matrix Documentation for when cosAxisY is 1 and observations that things break when if it does not equal 1. * Rename functions Renamed multiple functions for accuracy or clarity * doxygen whitespace alignment Improve readability of doxygen info by aligning text * Make format check Formatting updates suggested by make format * Make skyboxes static * Change function name angle_from_coords -> get_angle_between_coords * Math values renamed Functions: render_distance_squared -> distance_if_visible mtxf_rotation_zxy_translate -> mtxf_rotate_zxy_translate Arguments: vecTrans -> translate * Document many effects Define effects for * Braking * Drifting * Spinning out (driving and bananas) * Mini turbos * Losing at GP * Tumbling on terrain * CPU rubberbanding And substitute some pre-existing defined effect names in place of bit flags * Replace effect magic numbers with defined values * Separate effect bit-flags into components * Document more effects * Rename some effects * Replace effect bit_flags with defined values * Formatting fixes --------- Co-authored-by: MegaMech <[email protected]>
…64decomp#744) This reverts commit 6c60f5d.
* Update render_courses.c * More fixes * Fix hex values
…sic) (n64decomp#740) * Document sequence IDs * Document sequence IDs * Document sequence IDs * Document sequence IDs * Document sequence IDs * Fix function comments * Properly identify the podium ceremony sequence; fix comments --------- Co-authored-by: MegaMech <[email protected]>
* Naming sound effects * Update sound effect names and categorize * Add bomb effect and fix bug * HIT_FAKE_ITEM -> EXPLOSION_CRASH * Rename sound_effects to triggers * Merge branch 'master' into sound_effects --------- Co-authored-by: MegaMech <[email protected]>
* small adjustement * adjust some naming and other just two other in link with time trial --------- Co-authored-by: MegaMech <[email protected]>
ede7dfa to
21de062
Compare
* unk_044 initial documentation * Define player.unk_044 magic values * Turns more than 5 degrees * Lakitu Effects * UNK_0B6 (effect graphics) documentation * UNK_002 documentation * Document UNK_0DE and define magic values * Use defined player types * Rename `player->unk_0CA` to `lakitu_props` * rename `player->unk_0B6` to `kart_graphics` * Rename `player->unk_0DE` to `oobProps` * rename `player->unk_044` to kartProps * Formatting updates * Use camelCase instead of snake_case for player properties * Fix reference to wrong attribute in comment * Rename `PRESS_A` to `THROTTLE` Rename `PRESS_A` to `THROTTLE` to describe the Kart state, which may be slightly different than just pressing A.
21de062 to
4ccce89
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Automated changes by create-pull-request GitHub action