fix: solve the problem of being unable to distinguish between no loop and infinite loop.

Author: toFrankie

README.md

@@ -1,50 +1,57 @@
 # gify
+
 JavaScript API for decoding/parsing information from animated GIFs using ArrayBuffers.
 
 ## Why?
+
 Once I saw [vinepeek](http://www.vpeeker.com/), I immediately wanted to build a similar site for animated GIFs. The only problem was, there was no way to quickly determine the duration of an animated GIF, which varies in different browsers. Thus, gify was born over a weekend.
 
 ## Requirements
+
 gify requires [jDataView](https://github.com/vjeux/jDataView) for reading binary files. Please pull the latest from their repository.
 
 ## Methods
-* **isAnimated**(sourceArrayBuffer) (bool)
-* **getInfo**(sourceArrayBuffer) (gifInfo)
+
+- **isAnimated**(sourceArrayBuffer) (bool)
+- **getInfo**(sourceArrayBuffer) (gifInfo)
 
 ## info Properties
-* **valid** (bool) - Determines if the GIF is valid.
-* **animated** (bool) - Determines if the GIF is animated.
-* **globalPalette** (bool) - Determines if the GIF has a global color palette.
-* **globalPaletteSize** (int) - Size of the global color palette.
-* **globalPaletteColorsRGB** ([r,g,b]) - An array of objects containing the R, G, B values of the color palette. ([Beppe](https://github.com/Beppe))
-* **height** (int) - Canvas height.
-* **width** (int) - Canvas width.
-* **loopCount** (int) - Total number of times the GIF will loop. 0 represents infitine.
-* **images** ([images]) - Array of images contained in the GIF.
-* **isBrowserDuration** (bool) - If any of the delay times are lower than the [minimum value](http://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser-compatibility), this value will be set to true.
-* **duration** (int) - Actual duration calculated from the delay time for each image. If isBrowserDuration is false, you should use this value.
-* **durationIE** (int) - Duration for Internet Explorer (16fps)
-* **durationSafari** (int) - Duration for Safari in milliseconds (50fps)
-* **durationFirefox** (int) - Duration for Firefox in milliseconds (50fps)
-* **durationChrome** (int) - Duration for Chrome in milliseconds (50fps)
-* **durationOpera** (int) - Duration for Opera in milliseconds (50fps)
+
+- **valid** (bool) - Determines if the GIF is valid.
+- **animated** (bool) - Determines if the GIF is animated.
+- **globalPalette** (bool) - Determines if the GIF has a global color palette.
+- **globalPaletteSize** (int) - Size of the global color palette.
+- **globalPaletteColorsRGB** ([r,g,b]) - An array of objects containing the R, G, B values of the color palette. ([Beppe](https://github.com/Beppe))
+- **height** (int) - Canvas height.
+- **width** (int) - Canvas width.
+- **loopCount** (int) - Total number of times the GIF will loop. `0` represents infitine. `-1` represents display once.
+- **images** ([images]) - Array of images contained in the GIF.
+- **isBrowserDuration** (bool) - If any of the delay times are lower than the [minimum value](http://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser-compatibility), this value will be set to true.
+- **duration** (int) - Actual duration calculated from the delay time for each image. If isBrowserDuration is false, you should use this value.
+- **durationIE** (int) - Duration for Internet Explorer (16fps)
+- **durationSafari** (int) - Duration for Safari in milliseconds (50fps)
+- **durationFirefox** (int) - Duration for Firefox in milliseconds (50fps)
+- **durationChrome** (int) - Duration for Chrome in milliseconds (50fps)
+- **durationOpera** (int) - Duration for Opera in milliseconds (50fps)
 
 ## image Properties
-* **identifier** (string) - Image identifier (frame number or embeded string).
-* **top** (int) - Image top position (Y).
-* **left** (int) - Image left position (X).
-* **height** (int) - Image height.
-* **width** (int) - Image width.
-* **localPalette** (bool) - Image has a local color palette.
-* **localPaletteSize** (int) - Size of the local color palette.
-* **interlace** (bool) - Image is/is not interlaced.
-* **delay** (int) - Delay time in milliseconds.
-* **text** (string) - frame text. aka Plain Text Extension.
-* **comments** ([comments]) - Array of comment strings.
-* **disposal** (int) - Disposal method. (0-7). See [this](http://www.w3.org/Graphics/GIF/spec-gif89a.txt) for more details.
+
+- **identifier** (string) - Image identifier (frame number or embeded string).
+- **top** (int) - Image top position (Y).
+- **left** (int) - Image left position (X).
+- **height** (int) - Image height.
+- **width** (int) - Image width.
+- **localPalette** (bool) - Image has a local color palette.
+- **localPaletteSize** (int) - Size of the local color palette.
+- **interlace** (bool) - Image is/is not interlaced.
+- **delay** (int) - Delay time in milliseconds.
+- **text** (string) - frame text. aka Plain Text Extension.
+- **comments** ([comments]) - Array of comment strings.
+- **disposal** (int) - Disposal method. (0-7). See [this](http://www.w3.org/Graphics/GIF/spec-gif89a.txt) for more details.
 
 ### Example
-``` json
+
+```json
 {
   "valid": true,
   "globalPalette": true,
@@ -139,8 +146,10 @@ gify requires [jDataView](https://github.com/vjeux/jDataView) for reading binary
 ```
 
 ## Resources
-* [What's In A GIF - Bit by Byte](http://www.matthewflickinger.com/lab/whatsinagif/bits_and_bytes.asp) - Hands down the best write up on GIFs I've found.
-* [GIF98](http://www.w3.org/Graphics/GIF/spec-gif89a.txt) - GIF98 RFC.
-* [Animated GIF Frame Rate by Browser](http://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser-compatibility) - An awesome breakdown of how each browser renders animated GIFs. Thanks to Jeremiah Johnson for doing the hard work.
-* [GIF Format](http://www.onicos.com/staff/iz/formats/gif.html) - GIF blocks.
-* [Hexfiend](http://ridiculousfish.com/hexfiend/) - Awesome open source HEX editor (OSX)
+
+- [What's In A GIF - Bit by Byte](http://www.matthewflickinger.com/lab/whatsinagif/bits_and_bytes.asp) - Hands down the best write up on GIFs I've found.
+- [GIF98](http://www.w3.org/Graphics/GIF/spec-gif89a.txt) - GIF98 RFC.
+- [Animated GIF Frame Rate by Browser](http://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser-compatibility) - An awesome breakdown of how each browser renders animated GIFs. Thanks to Jeremiah Johnson for doing the hard work.
+- [GIF Format](http://www.onicos.com/staff/iz/formats/gif.html) - GIF blocks.
+- [Hexfiend](http://ridiculousfish.com/hexfiend/) - Awesome open source HEX editor (OSX)
+- [Netscape Looping Application Extension (GIF Unofficial Specification)](http://www.vurdalakov.net/misc/gif/netscape-looping-application-extension) - About Loop Count.

gify.js

@@ -78,7 +78,7 @@ var gify = (function () {
       globalPalette: false,
       globalPaletteSize: 0,
       globalPaletteColorsRGB: [],
-      loopCount: 0,
+      loopCount: -1, // `-1` means no loop
       height: 0,
       width: 0,
       animated: false,
@@ -170,9 +170,13 @@ var gify = (function () {
               var subBlock = readSubBlock(view, pos, true);
               switch (type) {
                 case 0xff: // APPLICATION EXTENSION
-                  /* since multiple application extension blocks can
-                     occur, we need to make sure we're only setting
-                     the loop count when the identifer is NETSCAPE */
+                  /**
+                   * Since multiple application extension blocks can occur,
+                   * we need to make sure we're only setting the loop count when the identifer is NETSCAPE.
+                   *
+                   * If GIF exported by Photoshop without loop, the Application Extension does not contain the NETSCAPE identifier, only the XMP Data identifier.
+                   * If GIF loops forever or loops n times (n > 0), the NETSCAPE identifier must be present in the Application Extension.
+                   */
                   var identifier = view.getString(8, pos + 1);
                   if (identifier === 'NETSCAPE') {
                     info.loopCount = view.getUint8(pos + 14, true);