fix listener memory leak

mihomo-party-org · Aug 3, 2024 · d9120e4 · d9120e4
1 parent 6484e3e
commit d9120e4
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/src/renderer/src/components/sider/outbound-mode-switcher.tsx b/src/renderer/src/components/sider/outbound-mode-switcher.tsx
@@ -4,7 +4,7 @@ import { patchMihomoConfig } from '@renderer/utils/ipc'
 import { Key } from 'react'
 
 const OutboundModeSwitcher: React.FC = () => {
-  const { controledMihomoConfig, patchControledMihomoConfig } = useControledMihomoConfig()
+  const { controledMihomoConfig, patchControledMihomoConfig } = useControledMihomoConfig(true)
   const { mode } = controledMihomoConfig || {}
 
   const onChangeMode = async (mode: OutboundMode): Promise<void> => {

diff --git a/src/renderer/src/components/sider/sysproxy-switcher.tsx b/src/renderer/src/components/sider/sysproxy-switcher.tsx
@@ -10,7 +10,7 @@ const SysproxySwitcher: React.FC = () => {
   const navigate = useNavigate()
   const location = useLocation()
   const match = location.pathname.includes('/sysproxy')
-  const { appConfig, patchAppConfig } = useAppConfig()
+  const { appConfig, patchAppConfig } = useAppConfig(true)
   const { sysProxy } = appConfig || {}
   const { enable } = sysProxy || {}
 

diff --git a/src/renderer/src/components/sider/tun-switcher.tsx b/src/renderer/src/components/sider/tun-switcher.tsx
@@ -11,7 +11,7 @@ const TunSwitcher: React.FC = () => {
   const location = useLocation()
   const match = location.pathname.includes('/tun')
 
-  const { controledMihomoConfig, patchControledMihomoConfig } = useControledMihomoConfig()
+  const { controledMihomoConfig, patchControledMihomoConfig } = useControledMihomoConfig(true)
   const { tun } = controledMihomoConfig || {}
   const { enable } = tun || {}
 

diff --git a/src/renderer/src/hooks/use-app-config.tsx b/src/renderer/src/hooks/use-app-config.tsx
@@ -8,7 +8,7 @@ interface RetuenType {
   patchAppConfig: (value: Partial<IAppConfig>) => Promise<void>
 }
 
-export const useAppConfig = (): RetuenType => {
+export const useAppConfig = (listenUpdate = false): RetuenType => {
   const { data: appConfig, mutate: mutateAppConfig } = useSWR('getConfig', () => getAppConfig())
 
   const patchAppConfig = async (value: Partial<IAppConfig>): Promise<void> => {
@@ -18,6 +18,7 @@ export const useAppConfig = (): RetuenType => {
   }
 
   useEffect(() => {
+    if (!listenUpdate) return
     window.electron.ipcRenderer.on('appConfigUpdated', () => {
       mutateAppConfig()
     })

diff --git a/src/renderer/src/hooks/use-controled-mihomo-config.tsx b/src/renderer/src/hooks/use-controled-mihomo-config.tsx
@@ -8,7 +8,7 @@ interface RetuenType {
   patchControledMihomoConfig: (value: Partial<IMihomoConfig>) => Promise<void>
 }
 
-export const useControledMihomoConfig = (): RetuenType => {
+export const useControledMihomoConfig = (listenUpdate = false): RetuenType => {
   const { data: controledMihomoConfig, mutate: mutateControledMihomoConfig } = useSWR(
     'getControledMihomoConfig',
     () => getControledMihomoConfig()
@@ -21,6 +21,7 @@ export const useControledMihomoConfig = (): RetuenType => {
   }
 
   useEffect(() => {
+    if (!listenUpdate) return
     window.electron.ipcRenderer.on('controledMihomoConfigUpdated', () => {
       mutateControledMihomoConfig()
     })