-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
F9 toggle message log. #531
Conversation
We decided to change the log message for the F9 toggle since, the previous log message doesn't suit well for those who use D3D9/D3D11/OGL(software mode). http://forums.pcsx2.net/Thread-Simple-bug-with-the-console-reporting?pid=455457#pid455457 |
Please change the commit message to |
Sorry, to |
What does that mean? please say what you want to say instead of sending me read a full thread and have me guessing what you want me to understand ;) It's fine to add a link for more info, but as a supplementary. |
Oh sorry I meant to link to my post on that thread's second page. I want to change the console message to remove the words Hardware/Software. Seems a bit redundant. If you're toggling, you're toggling between the options available and those are the only two available options. Well now that I think about it, technically that toggle could switch between different renderers (D3D9, D3D11, OGL) instead of different renderer modes (software, hardware) so I guess it is needed. Although the software renderers are not really renderer "modes", are they? They are full on renderers. So if we say "toggling GSdx renderers", that's still correct. Maybe the way to do it would be "Toggling GSdx %s renderer" &rendererVariable so it prints "Toggling GSdx D3D9 renderer" or "Toggling GSdx OpenGL renderer", etc, etc. |
the render variable values are only numbers, it couldn't be implemented that way. I think the current definition is pretty much understandable. no one would expect the change of a video renderer when we are just toggling the modes. |
@avih |
Oh that's not meant to be actual code, I have no idea what the variable name is or what it contains. I was just trying to explain it in a way that kinda looked like code so you guys would understand what I was suggesting more clearly. I figured they wouldn't be strings since you mentioned a conditional statement was needed. Thanks for clarifying though. :) "no one would expect the change of a video renderer when we are just toggling the modes." But you do have a point though, most people look at them as modes, so maybe we can just have the console message read "Toggling GSdx renderer modes" if you don't want to implement the variable in there. What do you guys think? |
I think the current "toggling gsdx hardware/software render" is pretty fine. We have mentioned in the console message itself that only the hardware/software mode gets toggled. IDK why would anyone consider the change of the video backend. If you feel any reasons on why users' would confuse it with a different video backend then, please mention why. |
What about "Toggling renderer mode"? Or finding a way to include which renderer they're being switch to (maybe with a variable)? That way there's no doubts about what renderer they just switched to since it'll say hardware or software. Plus since we are referring to them as modes in the console message, no user will think we're switching to a renderer with a different backend. |
There is still the null mode, that might confuse the user's. As I had said earlier, I really think that the current commited log message is fine. That would prevent confusion since, it is mentioned that only the hardware/software mode of the specific renderer is getting toggled. |
So you don't like the idea of using the variable to print to the console a message with exactly which mode is being used? (hardware/software) |
I like the idea of mentioning the renderer used. I was mentioning my opinion on idea of log message on "toggling the render mode". It is almost midnight at my region, I can't make the renderer variable changes now but, I'll look into it tomorrow morning. |
Sounds good. |
console: use correct message for hw/sw rendering switch
Thanks. |
No description provided.