akira/pdfjs-dist - pdfjs-dist - Gitea: Git with a cup of tea

Tree: b66b705ed7

Author	SHA1	Message	Date
Jonas Jenwald	3e20d30afc	Change the signatures of the `PartialEvaluator` "constructor" and its `getOperatorList`/`getTextContent` methods to take parameter objects Currently these methods accept a large number of parameters, which creates quite unwieldy call-sites. When invoking them, you have to remember not only what arguments to supply, but also the correct order, to avoid runtime errors. Furthermore, since some of the parameters are optional, you also have to remember to pass e.g. `null` or `undefined` for those ones. Also, adding new parameters to these methods (which happens occasionally), often becomes unnecessarily tedious (based on personal experience). Please note that I do not think that we need/should convert every single method in `evaluator.js` (or elsewhere in `/core` files) to take parameter objects. However, in my opinion, once a method starts relying on approximately five parameter (or even more), passing them in individually becomes quite cumbersome. With these changes, I obviously needed to update the `evaluator_spec.js` unit-tests. The main change there, except the new method signatures[1], is that it's now re-using one `PartialEvalutor` instance, since I couldn't see any compelling reason for creating a new one in every single test. Note: If this patch is accepted, my intention is to (time permitting) see if it makes sense to convert additional methods in `evaluator.js` (and other `/core` files) in a similar fashion, but I figured that it'd be a good idea to limit the initial scope somewhat. --- [1] A fun fact here, note how the `PartialEvaluator` signature used in `evaluator_spec.js` wasn't even correct in the current `master`.	8 years ago
Jonas Jenwald	7560f12a17	Enable the `object-shorthand` ESLint rule Please see http://eslint.org/docs/rules/object-shorthand. Unfortunately, based on commit `9276d1dcd9`, it seems that we still need to maintain compatibility with old Node.js versions, hence certain files/directories that are executed in Node.js are currently exempt from this rule. Furthermore, since the files specific to the Chromium extension are not run through Babel, the `/extensions/chromium/` directory is also exempt from this rule.	8 years ago
Tim van der Meij	35730148a7	Convert the files in the `/test/unit` folder to ES6 modules	8 years ago
porlan1	d9e1cb7955	unit test files as UMD modules	8 years ago
Jonas Jenwald	c850968fa7	Remove `globals` that are now unnecessary thanks to the use of various ESLint environments (e.g. Node, ShellJS, Jasmine)	8 years ago
Yury Delendik	0a700fa29d	Updates Jasmine version.	9 years ago
Manas	a2ba1b8189	Uses editorconfig to maintain consistent coding styles Removes the following as they unnecessary /* -- Mode: Java; tab-width: 2; indent-tabs-mode: nil; c-basic-offset: 2 -- / / vim: set shiftwidth=2 tabstop=2 autoindent cindent expandtab: */	10 years ago
Jonas Jenwald	1c66d4a106	Add a `totalLength` getter to `OperatorList`, since the `length` is zero after flushing In the `RenderPageRequest` handler in `worker.js`, we attempt to print an `info` message containing the rendering time and the length of the operator list. The latter is currently broken (and has been for quite some time), since the `length` of an `OperatorList` is reset when flushing occurs. This patch attempts to rectify this, by adding a getter which keeps track of the total length.	10 years ago
Yury Delendik	58c3ea0820	Adds thread abort capabilities.	10 years ago
Jonas Jenwald	0dde08554d	Add a unit-test for skipping `paintXObject` if `Subtype === PS` (PR 4861)	10 years ago
Jonas Jenwald	46a8485db4	Ignore paint form XObject when the name is missing (issue 4558) Fixes 4558 (since the font issues already appear to be fixed).	10 years ago
Collin Anderson	54e984c763	cleaned whitespace	10 years ago
Jonas Jenwald	06b5d97bc6	Remove two instances of leftover console.log debug statements The `console.log` statement in evaluator_spec.js is obviously not needed. In obj.js it could have been replaced by `info`, but that seemed unnecessary given the already existing `error`.	11 years ago
Nicholas Nethercote	081866a184	Use null instead of [] for ops with no args. This reduces peak RSS on one test file from ~600 to ~560 MiB.	11 years ago
Yury Delendik	d8eb8b1de1	Adds Promise to the getOperatorList	11 years ago
Christian Krebs	3e7bcaa892	Handle nested post script arguments in the preprocessor Fix for issue #4785	11 years ago
Yury Delendik	63d5aae3f6	Groups path commands into single command	11 years ago
Yury Delendik	09f8f951c8	Extracts evaluator preprocessor and refactor text extraction	11 years ago
Brendan Dahl	f4942b11f8	Reduce the memory usage of the operator list.	12 years ago
Brendan Dahl	bf72bc94e2	Incrementally render by sending the operator list by chunks as they're ready.	12 years ago
Mack Duan	2ce00279be	Address more of brendan's comments	12 years ago
Mack Duan	dbccbaaa27	Make getOperatorList() calls independent and merge queues at end	12 years ago
Yury Delendik	81f8f92696	Adds web/* and test/* for jshint target	12 years ago
mduan	5ab3bb1e03	Skip commands that have too few arguments - Commands that have too few args will be skipped - Commands that have too many args will generate an info, but still execute	12 years ago
Yury Delendik	43f1946c7a	Add prefixes for literals	13 years ago
Yury Delendik	ec6c185cf5	Allow parsing of the "glued" commands	13 years ago
Yury Delendik	324b867183	Adds evaluator tests	13 years ago