Skip to content

OpenHoldem solution restructure#53

Open
rub3r0id wants to merge 11 commits into
OpenHoldem:masterfrom
rub3r0id:R-OpenHoldem-restructure
Open

OpenHoldem solution restructure#53
rub3r0id wants to merge 11 commits into
OpenHoldem:masterfrom
rub3r0id:R-OpenHoldem-restructure

Conversation

@rub3r0id
Copy link
Copy Markdown

@rub3r0id rub3r0id commented May 31, 2024

  • Dramatic changes in the project root structure
    • "Common" folder with reusable classes or generic headers
    • "Libraries" folder with External and Internal libraries (static linking, except Keyboard, Mouse and ~User dll)
    • "OpenHoldem" folder with main project
    • "Tools" folder with additional projects (Replay/ManualMode/OpenScrape)
    • "Others" folder with old stuff or specific files (don't need to build main project)
  • Headers is reviewed and optimized
  • Zero warnings (no disabled warning for the Internal/Tools/OpenHoldem projects, only _CRT_SECURE_NO_DEPRECATE for PokerEval and SciLexer)
  • Completely cleanup and optimization for the *.vcxproj files
  • All solution building stuff will be into _int (intermediate objects) and _out (builded projects) folders into root
  • "Publication" project was added to collect builded files and to prepare .zip archive for Release to _pub folder into root
  • "OpenPPL" project was added just to copy libraries *.ohf files to _out directory
  • License and Readme documentation is updated (old is removed), Assets logo is added
  • Tested in Manual mode on Windows XP Professional VmWare

NB

@rub3r0id rub3r0id changed the title R open holdem restructure OpenHoldem solution restructure Jun 1, 2024
@rub3r0id rub3r0id force-pushed the R-OpenHoldem-restructure branch 2 times, most recently from f1b5604 to a72f5ee Compare June 3, 2024 08:17
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