Skip to content

MD5 Difference Detected functionality implemented.#134

Open
matheusjohannaraujo wants to merge 2 commits into
HashLDash:mainfrom
matheusjohannaraujo:main
Open

MD5 Difference Detected functionality implemented.#134
matheusjohannaraujo wants to merge 2 commits into
HashLDash:mainfrom
matheusjohannaraujo:main

Conversation

@matheusjohannaraujo

Copy link
Copy Markdown
Contributor

No description provided.

@matheusjohannaraujo matheusjohannaraujo changed the title Hot Reload functionality implemented. MD5 Difference Detected functionality implemented. Dec 25, 2021
@HashLDash

Copy link
Copy Markdown
Owner

Boa! Você chegou a testar com os scripts e com algum de interface gráfica? Talvez os apps que fiquem em loop deem algum problema.

@HashLDash

Copy link
Copy Markdown
Owner

Acho importante testar com GUI antes de dar merge. Pq como vc colocou um while, talvez ele trave e não identifique mais mudança. Não seria nada grave, poderia até dar merge. Mas seria interessante testar esse caso.

@matheusjohannaraujo

Copy link
Copy Markdown
Contributor Author

No caso, o ideal é rodar o código transpilado ou binario em uma thread separada, e o main do photon ficar monitorando se existe alteração no código fonte

Se detectar a mudança ele mataria a thread e refaria o processo de transpilar e rodar

@matheusjohannaraujo

Copy link
Copy Markdown
Contributor Author

Testei aqui, se o código photon tiver um loop infinito a verificação da alteração no código não funciona

@matheusjohannaraujo

Copy link
Copy Markdown
Contributor Author

Realmente, tem que usar duas threads para fazer isso funcionar direitinho

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ice box

Development

Successfully merging this pull request may close these issues.

2 participants