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

DiscoHawk meta-issue #2535

Closed
4 tasks done
YoshiRulz opened this issue Dec 26, 2020 · 3 comments
Closed
4 tasks done

DiscoHawk meta-issue #2535

YoshiRulz opened this issue Dec 26, 2020 · 3 comments
Labels
App: DiscoHawk Relating to DiscoHawk tool Enhancement For feature requests or possible improvements Meta Relating to code organisation or to things that aren't code Repro: Affects 2.6 Repro: Fixed/added in 2.7

Comments

@YoshiRulz
Copy link
Member

YoshiRulz commented Dec 26, 2020

  • System.Web dependency is superfluous (see Problematic dependencies #2261). 681b564, f56a981
  • MainDiscoForm's icon is loaded from its .resx file, but non-string resources don't work anymore. It's easy enough to use an embedded image instead. a0c7e9c
  • Needs assembly load logic copied from EmuHawk's Program, or immediately crashes due to missing assembly BizHawk.Common on Linux. 1a21765
  • Remake the Form layouts using *Ex controls and FLPs. not worth it IMO
  • Move logic to library projects. f431faf

@adelikat You said you could replace the HttpUtility.UrlDecode call with our path helpers?

@YoshiRulz YoshiRulz added Enhancement For feature requests or possible improvements App: DiscoHawk Relating to DiscoHawk tool Reproducible bug Should only be added to issues with a `Repro: Affects` label. Repro: Affects 2.5.3 dev labels Dec 26, 2020
@YoshiRulz YoshiRulz added this to the 2.5.3 milestone Dec 26, 2020
@YoshiRulz YoshiRulz added Meta Relating to code organisation or to things that aren't code and removed Reproducible bug Should only be added to issues with a `Repro: Affects` label. labels Dec 28, 2020
@YoshiRulz YoshiRulz removed their assignment Dec 28, 2020
@YoshiRulz YoshiRulz modified the milestones: 2.6, 2.6.1 Jan 17, 2021
@YoshiRulz YoshiRulz modified the milestones: 2.6.2, 2.6.3 May 26, 2021
@YoshiRulz YoshiRulz modified the milestones: 2.6.3, 2.6.4 Oct 1, 2021
@adelikat
Copy link
Contributor

@YoshiRulz : the problem is that discoHawk can not load in the BizHawk.Common library for some reason. THerefore the OSTailoredCode can not be referenced which is required to do this correctly. Better is to also import Client.Common so that we can just use PathUtils.ExeDirectoryPath

Can you look into this dll problem?

If you can look into that problem, we ca

@adelikat adelikat assigned YoshiRulz and unassigned adelikat Oct 18, 2021
@YoshiRulz
Copy link
Member Author

OSTailoredCode is compiled into the DiscoHawk assembly under a different namespace so it can be used before assembly loading is set up:

using OSTC = EXE_PROJECT.OSTailoredCode;

I can move PathUtils to the same file. EmuHawk's AssemblyResolve handler doesn't use that, though, it just uses Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location).

@adelikat
Copy link
Contributor

All the todos got done, so closing this issue

@YoshiRulz YoshiRulz removed their assignment Oct 18, 2021
YoshiRulz added a commit that referenced this issue Oct 19, 2021
@YoshiRulz YoshiRulz removed this from the 2.6.4 milestone Nov 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: DiscoHawk Relating to DiscoHawk tool Enhancement For feature requests or possible improvements Meta Relating to code organisation or to things that aren't code Repro: Affects 2.6 Repro: Fixed/added in 2.7
Projects
None yet
Development

No branches or pull requests

2 participants