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

IDebugLogs to Game/Logs folder #45

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

JayLCypher
Copy link

@JayLCypher JayLCypher commented Jul 10, 2021

Changes to IDebugLogs to update it and move the logs created by xNVSE to be located in Logs folder in Game folder (C:\Steam\SteamApps\Common\Fallout New Vegas) for easier access over installed plugins, if such a feature is desired.

Tested with debug/release compile and with MO2 modded setup.

Copy link
Contributor

@lStewieAl lStewieAl left a comment

Choose a reason for hiding this comment

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

You haven't included the ID in case creating the log fails (e.g. if the file is already open)

@JayLCypher
Copy link
Author

You haven't included the ID in case creating the log fails (e.g. if the file is already open)

Updated with file ID handle.
Checked Debug that it spawns another log file when nvse.log and log0 exists, creating nvse.log1.
Checked Release that it spawns another log file when other logs have id 1.
Also opened it in MO2 with a modded setup using executeable.

@JayLCypher JayLCypher closed this Jul 10, 2021
@JayLCypher JayLCypher reopened this Jul 10, 2021
@JayLCypher JayLCypher requested a review from lStewieAl July 10, 2021 23:54
@korri123
Copy link
Member

If you could make this change optional through INI then that'd be great. I myself am not keen on having the file in a Logs folder since I frequently look at logs and it's handy for me to have the base game folder open at all times and I can just use my keyboard to find it quickly to open it.

JayLCypher and others added 5 commits December 9, 2021 15:51
Moved a lot to cppLatest

Also fixed some offbyone errors in enum type checks where enum MAX was included, meaning arrays were accessed out-of-bounds.
Moved lots to CppLatest.
Fixed a few off-by-one bugs in enum evaluation (MAX was included where array sizes were MAX-1)
Some other misc changes.
@AVeryUncreativeUsername
Copy link
Contributor

So this thing has been open for two years. Do you think it is time to close it?

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.

4 participants