madsleejensen / react-native-image-sequence

native implementation for creating frame based image animations
MIT License
107 stars 67 forks source link

Calling RNImage Sequence subsequently after a setTimeOut causes memory leak #29

Open a4averma opened 5 years ago

a4averma commented 5 years ago

Consider the following example

Screen one

class LoadingScreen extends React.Component {
  componentDidMount() {
    setTimeout(() => {
      if (this.props.hasLoggedIn && this.props.hasDevices) {
        this.props.navigation.navigate("Main");
      } else if (this.props.hasLoggedIn && !this.props.hasDevices) {
        this.props.navigation.navigate("RegisterDevice");
      } else {
        this.props.navigation.navigate("BeMain");
      }
    }, 2500);
  }
  render() {
    return (
        <View style={styles.container}>
          <ImageSequence
            images={images}
            startFrameIndex={0}
            style={{
              width: 200,
              height: 200,
              alignSelf: "center"
            }}
          />
          <Spinner />
        </View>
    );
  }
}

Screen Two

export default class MainScreen extends React.Component {
  render() {
    return (
        <View style={styles.container}>
          <ImageSequence
            images={images}
            startFrameIndex={0}
            style={{ width: 150, height: 150 }}
          />
    );
  }
}

This causes a crash in signed release variant of the app. Debug app seems to work fine. How do I resolve this?

madsleejensen commented 5 years ago

Hi could you attach the error message?