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

Make the process name configurable. #45

Closed
Ted-Wang opened this issue Jul 23, 2019 · 9 comments · Fixed by #100
Closed

Make the process name configurable. #45

Ted-Wang opened this issue Jul 23, 2019 · 9 comments · Fixed by #100

Comments

@Ted-Wang
Copy link

Now I see the process name is fixed defined in file D2Reader/D2DataReader.cs.
Can you please make it configurable? I want use it in case run D2 with a different process name.

@Zutatensuppe
Copy link
Collaborator

@Ted-Wang sure! :)

@Zutatensuppe
Copy link
Collaborator

Zutatensuppe commented Jul 23, 2019

@Ted-Wang https://github.com/Zutatensuppe/DiabloInterface/releases/tag/v0.4.8

Added configurable process name. but you have to edit the settings file for that. There is no edit via GUI yet, so you will have to edit the settings file itself. After saving once, you will see something like this, you can edit it as you wish (DiabloInterface will try all of the configured processes and use the first one that is valid):


  "ProcessDescriptions": [
    {
      "ProcessName": "game",
      "ModuleName": "Game.exe",
      "SubModules": [
        "D2Common.dll",
        "D2Launch.dll",
        "D2Lang.dll",
        "D2Net.dll",
        "D2Game.dll",
        "D2Client.dll"
      ]
    },
    {
      "ProcessName": "d2se",
      "ModuleName": "D2SE.exe",
      "SubModules": [
        "D2Common.dll",
        "D2Launch.dll",
        "D2Lang.dll",
        "D2Net.dll",
        "D2Game.dll",
        "D2Client.dll"
      ]
    }
  ]

@Ted-Wang
Copy link
Author

@Zutatensuppe Thank you. Actually I want to use this tool in a high screen resolution mode. I patched D2 with D2loader-high.exe. Unfortunately, it seems doesn't work when I setting the process name with "D2loader-high.exe" or "D2loader-high". Anyway, thank you all the same!

@Zutatensuppe
Copy link
Collaborator

oh, ok, if you can provide more information about the D2loader-high.exe i can look why it isnt working

@Zutatensuppe Zutatensuppe reopened this Jul 25, 2019
@Ted-Wang
Copy link
Author

Ted-Wang commented Jul 25, 2019

Actually, I don't know mutch about the patch. I guess it's a wrap program that help to load 3rd party plugins and then call game.exe.
FYI, I upload the patch to Dropbox, you can get it from this link:
https://www.dropbox.com/s/4lzxsep2pkxy4hn/D2_HD_Patch.zip?dl=0

To use this patch, just extract the 3 files to D2 root folder and double click the D2Loader-high.exe, then in the game settings, you can set the screen resolution higher than 800*600, up to 1920*1080.
If your OS is win 7 or higher, you can create a shortcut and set compatible mode to winXP sp3.

The only thing I know is, the process name will be changed to D2Loader-high, I checked this in Powershell(by running command Get-Process), and watching from task manger, the process name is D2Loader-high.exe.

I running D2X 1.13c, not sure if it works for other version.

@Zutatensuppe
Copy link
Collaborator

ok, thank you :) i ll try it out when i have a moment 👍

@Ted-Wang
Copy link
Author

Thank you for you god job!

@Zutatensuppe
Copy link
Collaborator

probably not very interesting anymore as this issue was open for >1 year, but added D2Loader-high support by default.
it will be part of next release

(note: process name still configurable by editing the config files, so not very convenient. config still mixes application settings + character settings :( )

@Ted-Wang
Copy link
Author

Thank you so much for your amazing work!
I can see my character's status together with high screen resolution right now!

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 a pull request may close this issue.

2 participants