search

mikolalysenko / node-latex

node.js wrapper for LaTeX
MIT License
25 stars 20 forks source link

Added a ready notification #8

Open chaser92 opened 10 years ago

chaser92 commented 10 years ago

In my use case I'd like to know if LaTeX processing succeeded before I started actually reading from stream (e.g. in a promise) - therefore I added an event when LaTeX process succeeded.

mikolalysenko commented 10 years ago

Can't you just check for an "end" event on the stream? Not sure why this is needed.

chaser92 commented 10 years ago

Because I don't even want to read from the stream when I don't know whether the compilation succeeded. No PDF output will be in the stream anyway

2014-07-15 2:47 GMT+02:00 Mikola Lysenko notifications@github.com:

Can't you just check for an "end" event on the stream? Not sure why this is needed.

— Reply to this email directly or view it on GitHub https://github.com/mikolalysenko/node-latex/pull/8#issuecomment-48978293 .

Pozdrawiam, Mariusz Kierski

chaser92 commented 10 years ago

It doesn't hurt to have this notification, does it? No one needs to subscribe after all.