php-src に merge されました

2015/10/25

先日、ひょんなことから本当に細かいtypoに気がついたので、php-srcにPRを送ったところmasterにmergeされました。 つたないカタコト英語でのコミットメッセージですが、修正意図が自明だったためか、何事もなくmergeいただけました。 どのような修正をしたのかというと(聞いたらがっかりされるかもしれませんが)、vimやEmacsでの折りたたみタグ(fold-marker)が機能していない場所を直した、ということです。本当にそれだけです。

php-srcではCODING_STANDARDSにあるように

Documentation and Folding Hooks
-------------------------------

In order to make sure that the online documentation stays in line with
the code, each user-level function should have its user-level function
prototype before it along with a brief one-line description of what the
function does.  It would look like this::

  /* {{{ proto int abs(int number)
     Returns the absolute value of the number */
  PHP_FUNCTION(abs)
{
     ...

}
  /*  }}} */

The {{{ symbols are the default folding symbols for the folding mode in
Emacs and vim (set fdm=marker).  Folding is very useful when dealing with
large files because you can scroll through the file quickly and just unfold
the function you wish to work on.  The  }}} at the end of each function marks
the end of the fold, and should be on a separate line.

{{{ と }}} というデフォルトの折りたたみタグを使いましょうね的なことがあるのを確認したので、これは修正したほうがよいのかなと考えPRした次第です。

そもそもなんでこのtypoを見つけたのかというと、nginx + php-fpm の構築手順を考えていて、php-fpm.confのsampleのありかを探していたところ脱線して当該ディレクトリにたどり着き、何の気なしにfpm_events.cを開いてみたら折りたたみタグがなかったことに気がついた、という流れです。 まったく偶然の出来事でありながら、とてもよい経験になりました。remiさんのアイコンと並んであるのは一切Cを書いておらずちょっと気まずいながらも嬉しいものがあります。