Project

General

Profile

Actions
Copy link

Bug #97

closed

[PATCH] Move video postprocess option from GUI settings to video settings

Added by arnova over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Other (un-categorized)
Target version:
3.0
Start date:
Due date:
% Done:

0%

Resolution:
fixed
Affected Version:

Description

I've created a patch which moves the GUI setting for postprocess to the video settings dialog. The problem is: it doesn't work yet. I can't seem to figure out why but after applying this patch all video settings are screwed up.

Files

postprocess_video_settings.patch (14.4 KB) postprocess_video_settings.patch arnova, 22/08/2010 11:57 AM
Actions
Copy link
 #1

Updated by arnova over 13 years ago

@Exobuzz: Do you have any idea what I possibly did wrong?

Actions
Copy link
 #2

Updated by buzz over 13 years ago

Heh, I had some similar code on my local copy. I was just looking at having both a setting here, but also set a global "default", (I was thinking to make a patch for xbmc mainline where a global default could be useful too, it is less important/useful on xbox).

as for your error, just had a brief look over the patch. You have a GetBoolean which should be a SetBoolean I believe. Don't know if that though is enough to cause the problem.

Actions
Copy link
 #3

Updated by arnova over 13 years ago

  • Status changed from New to Closed
  • Resolution set to fixed

Fixed in svn

Actions
Copy link

Also available in: Atom PDF