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

Setting default renderer for Windows to Direct3D for 0.D #27915

Merged
merged 13 commits into from
Jan 30, 2019
Merged

Setting default renderer for Windows to Direct3D for 0.D #27915

merged 13 commits into from
Jan 30, 2019

Conversation

Firestorm01X2
Copy link
Contributor

@Firestorm01X2 Firestorm01X2 commented Jan 27, 2019

Setting default renderer for Windows to Direct3D for 0.D
Player reports that it should help to avoid this:
#27731

#27731 can be worked around by setting the default renderer to something besides software mode, or at least you can set it to direct3D on windows. I don't know if that resolves the issue on OSX or Linux, though.

Aka black lines around tiles.

It is workaround, but not bugfix.

Summary

SUMMARY: Content "Setting default renderer for Windows to Direct3D for 0.D"
I dont' want to call it "Bugfix", because it is not.

Purpose of change

#27731
But there is reports that setting default renderer to Direct3D allows to bypass the issue.

Linux and Android will still use default as "Software".

Describe the solution

Setting default rendered fot Windows to Direct3D for 0.D

Describe alternatives you've considered

This should be fixed. Somehow.
#27731

Also there is addtional workaround:
#27731 (comment)

Additional context

None.

Setting default rendered fot Windows to Direct3D for 0.D
@Firestorm01X2 Firestorm01X2 changed the title [WIP] Setting default rendered fot Windows to Direct3D for 0.D [WIP] Setting default renderer fot Windows to Direct3D for 0.D Jan 27, 2019
@Firestorm01X2 Firestorm01X2 changed the title [WIP] Setting default renderer fot Windows to Direct3D for 0.D [WIP] Setting default renderer for Windows to Direct3D for 0.D Jan 27, 2019
if tiles
@ZhilkinSerg ZhilkinSerg added SDL: Tiles / Sound Tiles visual interface and sounds. [C++] Changes (can be) made in C++. Previously named `Code` labels Jan 27, 2019
src/options.cpp Outdated Show resolved Hide resolved
Firestorm01X2 added 5 commits January 27, 2019 13:28
revert
Refactoring
styling
styling
styling
@Firestorm01X2
Copy link
Contributor Author

Jenkins, rebuild.

Firestorm01X2 added 2 commits January 27, 2019 15:42
move renderer_list in #ifndef __ANDROID__
refactoring
}


Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

src/cata_tiles.cpp Outdated Show resolved Hide resolved
Firestorm01X2 added 2 commits January 27, 2019 16:34
Astyling
Styling
Firestorm01X2 added 2 commits January 27, 2019 16:37
comment
Astyling
@Firestorm01X2
Copy link
Contributor Author

Finally it is builds.
Tiny refactoring may be overkill but it will make this #ifndef slightly more readable.

@Firestorm01X2
Copy link
Contributor Author

Warning! Before merging change must checked on non Windows environment. Just in case.

@Firestorm01X2
Copy link
Contributor Author

It works for me on Windows 10 and Ubuntu.
So it is ready.

@Firestorm01X2
Copy link
Contributor Author

Firestorm01X2 commented Jan 27, 2019

But for some reason I am still getting black lines graphical bug. But it is on VS build. Probably MXE build will be OK as some other players report.

@Firestorm01X2
Copy link
Contributor Author

Ready.

@Firestorm01X2 Firestorm01X2 changed the title [WIP] Setting default renderer for Windows to Direct3D for 0.D Setting default renderer for Windows to Direct3D for 0.D Jan 27, 2019
@ZhilkinSerg ZhilkinSerg added this to the 0.D milestone Jan 28, 2019
@ZhilkinSerg ZhilkinSerg self-assigned this Jan 28, 2019
@ZhilkinSerg ZhilkinSerg merged commit 366c62d into CleverRaven:0.D-branch Jan 30, 2019
@Firestorm01X2
Copy link
Contributor Author

Are you going to merge it to master also? Or should I create adtional PR?

@ZhilkinSerg
Copy link
Contributor

You can create new PR against master or wait until we merge commits from this PR to master. Either way changes should end up in master.

kevingranade added a commit that referenced this pull request Jan 31, 2019
* 0.D-branch:
  Make dollchat items display their speech lines again
  Fixing position of morale and power on skinny siderbar
  Default text color for popups
  Overhauled default zombie drops
  Fix for Fish traits
  Add check against deploying a furniture item into same tile as a vehicle
  Add CSC_*_HIDDEN recipe subtab
  Fix items rotting while smoking outside of reality bubble.
  Setting default renderer for Windows to Direct3D for 0.D (#27915)
  Updating help for 0.D (#27900)
@ZhilkinSerg ZhilkinSerg removed their assignment Feb 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` SDL: Tiles / Sound Tiles visual interface and sounds.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants