パッチ on akky@サイボウズラボ・ブログ

ワードプレスのソースコードから8年間懸案の変数名「おしっこ」が除去

Mon, 07 Mar 2022 00:00:00 +0000

ブログツールとして世界中の多くのサーバで使われている WordPress のソースコードの、数限りなくある中のちょっとした修正は、変数名の"pee"と"tinkle"を書き換えるというものでした。

-442 function wpautop( $pee, $br = true ) { +442 function wpautop( $text, $br = true ) {

wpautop()という関数は、ブログ記事のテキストに2行以上の改行があった時、それを

タグに置換する、という働きをします。

「WordPress(wp) が自動(auto)で p(タグ)をつける」といったあたりから命名された関数だろうと思いますが、WordPress の歴史は長いのでこの関数も昔から存在していたようです。

それで、この関数を書いた誰かは「オートでPする」の響きからの連想か、引数名に $pee(しっこ、幼児語)を使う言葉遊びをしてしまったようですね。

その後のどこかの時点の修正で $pees をループで一つずつ処理する際に、取り出す変数を $tickle (こちらもしっこ、幼児語)が使われています。

# tinkle というとリトルスターぐらいしか連想しませんが、そんな意味もあるんですね。

WordPress の公式バグ管理システムからは、8年前のチケットで初めてこの二つの変数名についての指摘がなされたことがわかります。

Debian の perf を高速化するパッチと、そのGPL的な背景

Tue, 28 Sep 2021 00:00:00 +0000

Cargo の flamegraph ツールがDebian の perf のせいでとても遅いという話。

perf コマンドがプロファイルデータを解析する際に libbfd をリンクすればずっと高速になるのですが、Debian のポリシーでライセンスに違反するため libbfd をリンクすることはできないそうです。

[修正] コメントでの指摘を受け修正しました

perf のライセンスが GPL バージョン2、libbfd のライセンスが GPL バージョン3+ であり、これら二つのプロジェクトを合法的にリンクさせる方法がないそう。

Debian の Makefile では、わざわざ生成された perf コマンドが libbfd をリンクして「いない」ことをチェックするようになっているのですね。

そんな訳で、Debian の perf は dso 一つごとに addr2line コマンドをパイプで呼び出す(そして、毎回サブプロセスは終了する)方式となっています。

しかし、大量の perf 呼び出しを行う flamegraph を作ろうとした @leastfixedpoint さんは、いつまでも帰ってこない perf コマンドに業を煮やし、libbfd をリンクさせない範囲での高速化を考えました。

このパッチでは、addr2line を別プロセスとして起動させたままにしておき、dso をデータとして送り付け結果を受け取っても、addr2line プロセスを終了させないことで、とあるデータ量に対して68倍速く処理が終わるようにできたそうです。

対象のデータ量によっては気にならない程度の速度差なのかもしれませんが、同じようにここの処理速度で困ってる人もいるんじゃないでしょうかね。

via Hacker News

Linuxのソースコードコメントからf*ckを消してhugにしようと提案するパッチが議論を巻き起こす

Thu, 06 Dec 2018 00:00:00 +0000

ヤルコ・サッキネンさん(Jarkko Sakkinen)がリナックスDRM開発者メーリングリストに提案した33行のパッチは、コメント中の"fuck"を"hug"に置き換えるというだけのもの。提案の変更箇所は以下となり、変更内容はたとえばこんな感じです。

In order to comply with the CoC, replace **** with a hug. Jarkko Sakkinen (15): MIPS: replace **** with a hug Documentation: replace **** with a hug drm/nouveau: replace **** with a hug m68k: replace **** with a hug parisc: replace **** with a hug cpufreq: replace **** with a hug ide: replace **** with a hug media: replace **** with a hug mtd: replace **** with a hug net/sunhme: replace **** with a hug scsi: replace **** with a hug inotify: replace **** with a hug irq: replace **** with a hug lib: replace **** with a hug net: replace **** with a hug Documentation/kernel-hacking/locking.rst | 2 +- arch/m68k/include/asm/sun3ints.h | 2 +- arch/mips/pci/ops-bridge.c | 24 +++++++++---------- arch/mips/sgi-ip22/ip22-setup.c | 2 +- arch/parisc/kernel/sys_parisc.c | 2 +- drivers/cpufreq/powernow-k7.c | 2 +- .../gpu/drm/nouveau/nvkm/subdev/bios/init.c | 2 +- .../nouveau/nvkm/subdev/pmu/fuc/macros.fuc | 2 +- drivers/ide/cmd640.c | 2 +- drivers/media/i2c/bt819.c | 8 ++++--- drivers/mtd/mtd_blkdevs.c | 2 +- drivers/net/ethernet/sun/sunhme.c | 4 ++-- drivers/scsi/qlogicpti.h | 2 +- fs/notify/inotify/inotify_user.c | 2 +- kernel/irq/timings.c | 2 +- lib/vsprintf.c | 2 +- net/core/skbuff.c | 2 +- 17 files changed, 33 insertions(+), 31 deletions(-) -- 2.19.1