diff options
-rw-r--r-- | macosx/Controller.m | 2 | ||||
-rw-r--r-- | macosx/HBPresetsManager.m | 12 | ||||
-rw-r--r-- | macosx/HBQueueController.mm | 8 | ||||
-rw-r--r-- | macosx/HBVideoController.m | 36 |
4 files changed, 27 insertions, 31 deletions
diff --git a/macosx/Controller.m b/macosx/Controller.m index d9327b0a1..97e5f08db 100644 --- a/macosx/Controller.m +++ b/macosx/Controller.m @@ -3418,7 +3418,7 @@ fWorkingCount = 0; else if (job->vcodec & HB_VCODEC_FFMPEG_MASK) { hb_job_set_encoder_options(job, - [[queueToApply objectForKey:@"lavcOption"] + [[queueToApply objectForKey:@"VideoOptionExtra"] UTF8String]); } diff --git a/macosx/HBPresetsManager.m b/macosx/HBPresetsManager.m index 1384a7438..86bd7f4fa 100644 --- a/macosx/HBPresetsManager.m +++ b/macosx/HBPresetsManager.m @@ -432,7 +432,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"fast"; @@ -547,7 +546,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -651,7 +649,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -755,7 +752,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -859,7 +855,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -974,7 +969,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -1089,7 +1083,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -1204,7 +1197,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -1308,7 +1300,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -1412,7 +1403,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; @@ -1516,7 +1506,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"veryfast"; @@ -1620,7 +1609,6 @@ NSString *HBPresetsChangedNotification = @"HBPresetsChangedNotification"; /* Video encoder and advanced options */ preset[@"VideoEncoder"] = @"H.264 (x264)"; - preset[@"lavcOption"] = @""; preset[@"x264Option"] = @""; preset[@"x264UseAdvancedOptions"] = @0; preset[@"VideoPreset"] = @"medium"; diff --git a/macosx/HBQueueController.mm b/macosx/HBQueueController.mm index 9fedcf943..3635ce296 100644 --- a/macosx/HBQueueController.mm +++ b/macosx/HBQueueController.mm @@ -991,16 +991,16 @@ { /* we are using libavcodec */ NSString *lavcInfo = @""; - if (item[@"lavcOption"] && - [item[@"lavcOption"] length]) + if (item[@"VideoOptionExtra"] && + [item[@"VideoOptionExtra"] length]) { - lavcInfo = [lavcInfo stringByAppendingString: item[@"lavcOption"]]; + lavcInfo = [lavcInfo stringByAppendingString: item[@"VideoOptionExtra"]]; } else { lavcInfo = [lavcInfo stringByAppendingString: @"default settings"]; } - [finalString appendString: @"ffmpeg: " withAttributes:detailBoldAttr]; + [finalString appendString: @"Encoder Options: " withAttributes:detailBoldAttr]; [finalString appendString: lavcInfo withAttributes:detailAttr]; [finalString appendString:@"\n" withAttributes:detailAttr]; } diff --git a/macosx/HBVideoController.m b/macosx/HBVideoController.m index af2c6c0db..325145969 100644 --- a/macosx/HBVideoController.m +++ b/macosx/HBVideoController.m @@ -281,8 +281,6 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio /* video encoder */ self.codec = [queueToApply[@"JobVideoEncoderVcodec"] intValue]; - self.lavcOptions = queueToApply[@"lavcOption"]; - /* advanced x264 options */ if ([queueToApply[@"x264UseAdvancedOptions"] intValue]) { @@ -298,7 +296,7 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio [fX264UseAdvancedOptionsCheck setState:NSOnState]; [self updateEncoderOptionsWidgets:nil]; } - else + else if (self.codec == HB_VCODEC_X264 || self.codec == HB_VCODEC_X265) { // we are using the x264 preset system [self setPreset: queueToApply[@"VideoPreset"]]; @@ -312,6 +310,10 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio [fX264UseAdvancedOptionsCheck setState:NSOffState]; [self updateEncoderOptionsWidgets:nil]; } + else + { + self.lavcOptions = queueToApply[@"VideoOptionExtra"]; + } /* Lets run through the following functions to get variables set there */ [self videoEncoderPopUpChanged:nil]; @@ -445,9 +447,11 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio [self updateEncoderOptionsWidgets:nil]; } } - - // Apply the lavcOption - self.lavcOptions = preset[@"lavcOption"]; + else + { + // Apply the lavcOption + self.lavcOptions = preset[@"VideoOptionExtra"]; + } int qualityType = [preset[@"VideoQualityType"] intValue] - 1; /* Note since the removal of Target Size encoding, the possible values for VideoQuality type are 0 - 1. @@ -536,7 +540,7 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio queueFileJob[@"x264UseAdvancedOptions"] = @1; queueFileJob[@"x264Option"] = [self.fAdvancedOptions optionsString]; } - else + else if (self.codec == HB_VCODEC_X264 || self.codec == HB_VCODEC_X265) { // we are using the x264/x265 preset system queueFileJob[@"x264UseAdvancedOptions"] = @0; @@ -546,9 +550,11 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio queueFileJob[@"VideoProfile"] = [self profile]; queueFileJob[@"VideoLevel"] = [self level]; } - - /* FFmpeg (lavc) Option String */ - queueFileJob[@"lavcOption"] = self.lavcOptions; + else + { + /* FFmpeg (lavc) Option String */ + queueFileJob[@"VideoOptionExtra"] = self.lavcOptions; + } queueFileJob[@"VideoQualityType"] = @(self.qualityType + 1); queueFileJob[@"VideoAvgBitrate"] = [fVidBitrateField stringValue]; @@ -708,7 +714,7 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio preset[@"x264UseAdvancedOptions"] = @1; preset[@"x264Option"] = [self.fAdvancedOptions optionsString]; } - else + else if (self.codec == HB_VCODEC_X264 || self.codec == HB_VCODEC_X265) { /* use the x264 preset system */ preset[@"x264UseAdvancedOptions"] = @0; @@ -731,9 +737,11 @@ NSString *HBVideoEncoderChangedNotification = @"HBVideoEncoderChangedNotificatio preset[@"x264Option"] = @""; } } - - /* FFmpeg (lavc) Option String */ - preset[@"lavcOption"] = self.lavcOptions; + else + { + /* FFmpeg (lavc) Option String */ + preset[@"VideoOptionExtra"] = self.lavcOptions; + } /* though there are actually only 0 - 1 types available in the ui we need to map to the old 0 - 2 * set of indexes from when we had 0 == Target , 1 == Abr and 2 == Constant Quality for presets |