Thomas Smith

Results 70 comments of Thomas Smith

https://github.com/wbond/package_control/pull/1392 Technically, this package does not reload dependencies, it only unloads them. Packages need to be explicitly loaded so that commands and such will be properly registered. Dependencies can be...

What's the output of the following console command? ```python from AutomaticPackageReloader.reloader.reloader import resolve_dependencies; resolve_dependencies('debugtools') ``` It should be a tuple of two sets: respectively, dependencies and packages to reload.

I'm looking over the code trying to find something, and I did notice the following: - When reloading a dependency, we unload all dependencies before reloading packages. This could cause...

@evandrocoan try the `refactor-reload` branch and see if that helps.

Ah, that makes sense. This should be solvable (after #28). Whenever a package causes `sys.stdout` to be patched, it should ensure that the patch will be undone in `plugin_unloaded`. (This...

I think the reloader logic definitely makes sense for a dependency. I am of two minds about whether sublime_lib is the right place versus a new dedicated dependency (hypothetically, "package_util")....

Is this issue live, or does #20 resolve it?

The argument list can't be distinguished from a parenthesized expression for two reasons. The lesser reason is the line-lookahead limitation. The greater reason is that the JavaScript syntax is not...

It's unfortunate that there's no perfect solution, but it's still a good idea to highlight the body correctly. This is what the core syntax does. > if sublime allows multiline...

It might actually be possible using subexpression calls, because those aren't deterministic context-free. The cases where I expect that particular expression to fail are parentheses in strings or comments. You'd...