Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zephir filetype #270

Merged
merged 5 commits into from Apr 10, 2015
Merged

Zephir filetype #270

merged 5 commits into from Apr 10, 2015

Conversation

b4n
Copy link
Member

@b4n b4n commented May 15, 2014

Experimental support for Zephir.

Keywords and build command come from https://github.com/taophp/zephir-geany-fileconf

b4n added 5 commits May 15, 2014 16:35
This Zephir parser uses the PHP parser with minor tweaking not to
require the initial PHP open tag and skip function return type hints.
@taophp
Copy link

taophp commented May 15, 2014

Happy to see that you used my work. About the build command: I haven't test it from Geany. I worry about the path: zephir build does not accept any argument, and must be called from the Zephir project directory. Is Geany able to deal with that ? If not, I think a command like:

cd `direname %f`/.. ; zephir build

should work... at least with Posix systems...

@b4n
Copy link
Member Author

b4n commented May 15, 2014

Using the newer [build-menu] syntax you can specify the directory in which run the command. However, although I don't know Zephir, I suspect files can be nested deeper than one-level inside the module, or maybe even at modules' root, can't they? If they can, using an hard-coded bath won't always work.

@taophp
Copy link

taophp commented May 15, 2014

@b4n You're right. In fact, Zephir directories map the namespaces tree...

@taophp
Copy link

taophp commented May 15, 2014

Great! I've updated my Geany to your zephir-filetype branch: highlighting and symbols locating work well. That's much, much better.
A last improvement should make me an happy man: as Zephir coders are expected to largely use PHP standard functions, the help provided when typing a PHP file (function names completion, and popping function signatures) should very useful when typing a Zephir file.
I tried to change my filetypes.Zephir.conf but failed...

@b4n
Copy link
Member Author

b4n commented May 15, 2014

@taophp copying or linking php.tags (which gets installed in $prefix/share/geany/) to ~/.config/geany/tags/std.zep.tags should do it.

@taophp
Copy link

taophp commented May 16, 2014

@b4n Thx! It's almost perfect now! The only thing that's still missing is the location of variables and properties, but that not an issue for me.
Thanks again for your fast and efficient action!

@b4n b4n merged commit 4e0193d into geany:master Apr 10, 2015
b4n added a commit that referenced this pull request Apr 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants