Skip to content

Commit

Permalink
Merge pull request #1 from adityakrshnn/fix/lgtm
Browse files Browse the repository at this point in the history
fix: lgtm fixes
  • Loading branch information
adityakrshnn authored Jun 12, 2021
2 parents 01b8dbe + e4a3592 commit 2d03bfb
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions src/services/filmstrip.service.ts
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,8 @@ export class FilmstripService {

getParameters = (config: FilmstripRequestConfig, mediainfo: Mediainfo): SingleOutputParametersResponse => {
const auxillaryParameters = ["-y"];
let parametersResponse: SingleOutputParametersResponse = {
parameters: [],
outputFilename: '',
};

parametersResponse = this.getTimeIntervalBasedParameters(config, mediainfo)
const parametersResponse = this.getTimeIntervalBasedParameters(config, mediainfo)

const response: SingleOutputParametersResponse = {
parameters: [...parametersResponse.parameters, ...auxillaryParameters],
Expand All @@ -54,7 +50,7 @@ export class FilmstripService {
const selectTime = `select=not(mod(n\\,${config.timeInterval!})),`;
const scale = config.resolution ? `${Utility.getScale(config.resolution)},` : '';
const tiles = Utility.getFilmstripTileString(totalFramesInFilmstrip, config);
const vfString = `${selectTime}${scale}${tiles}`
const vfString = `${selectTime}${scale}${tiles}`;

const inParameters: string[] = ['-i', config.filename, '-frames', '1', '-vf', vfString];
const outParameters: string[] = [outputFilename];
Expand Down

0 comments on commit 2d03bfb

Please sign in to comment.