Ctags: PHP heredoc (<<

@jespinal , apakah Anda berbicara tentang perubahan ini: https://wiki.php.net/rfc/flexible_heredoc_nowdoc_syntaxes ?

$ git diff |cat
git diff |cat
diff --git a/parsers/php.c b/parsers/php.c
index e3fdc241..ace25561 100644
--- a/parsers/php.c
+++ b/parsers/php.c
@@ -682,6 +682,8 @@ static void parseHeredoc (vString *const string)
            int extra = EOF;

            c = getcFromInputFile ();
+           if (c == ' ' || c == '\t')
+               c = getcFromInputFile ();
            for (len = 0; c != 0 && (c - delimiter[len]) == 0; len++)
                c = getcFromInputFile ();

$ cat input.php
cat input.php
<?php
// Taken from https://github.com/universal-ctags/ctags/issues/2717
// submitted by <strong i="5">@jespinal</strong>
class LivingBeings {

    public function doSomething()
    {
        $foo = <<<FOO
        FOO;
    }

    public function doSomethingElse()
    {
    }
}
$ u-ctags -o - input.php
u-ctags -o - input.php
LivingBeings    input.php   /^class LivingBeings {$/;"  c
doSomething input.php   /^    public function doSomething()$/;" f   class:LivingBeings
$

@masatake itu tidak cukup, karena penanda akhir dulu harus berada di jalurnya sendiri , sedangkan versi baru menghilangkan batasan itu. Saya tidak menemukan penjelasannya dengan sangat jelas:

Implementasi yang saya usulkan menghindari masalah ini dengan memeriksa untuk melihat apakah ada kelanjutan dari penanda yang ditemukan, dan jika demikian, maka apakah itu membentuk pengidentifikasi yang valid.

tetapi saya akan mengatakan bahwa itu berarti bahwa kecuali ada karakter pengenal setelah baris yang diawali dengan penanda pengakhiran, itu memang penanda pengakhiran. Jadi END; adalah penghentian (mengingat penandanya adalah END ), tetapi ENDFOO tidak.

BTW, karena ini adalah perubahan sintaksis yang tidak kompatibel, saya tidak tahu apa yang ingin kami lakukan. Tapi saya kira jika PHP senang untuk memecahkannya, kita juga bisa, terutama karena itu tidak mungkin menimbulkan masalah. Idealnya saya kira kita akan menggunakan sintaks saat ini untuk *.php[1-6] dan yang baru untuk sisanya, tetapi itu mungkin terlalu merepotkan untuk nilainya.

@jespinal , apakah Anda berbicara tentang perubahan ini: https://wiki.php.net/rfc/flexible_heredoc_nowdoc_syntaxes ?

Maaf, @masatake , untuk beberapa alasan saya tidak diberitahu tentang pertanyaan Anda.

Ya, saya sedang berbicara tentang perubahan itu. Tapi itu sebenarnya diimplementasikan di PHP 7.3 (versi stabil saat ini adalah 7.4, dan versi 8 sudah dekat). Saya tidak yakin mengapa ini tidak dilaporkan sebelumnya mengingat basis pengguna yang luas dari keduanya, ctags dan PHP.

Saya menambahkan beberapa tangkapan layar cuplikan kode yang berasal dari contoh sebelumnya untuk (semoga) menjelaskan apa yang mereka anggap sintaks valid/tidak valid sehubungan dengan sintaks heredoc/nowdoc baru (RFC tidak cukup jelas, saya berpikir).

Dalam contoh ini, ' TEXT ' (yang kedua) adalah penanda akhir. Jadi, yang ketiga ' TEXT; adalah string yang tidak valid secara sintaksis dalam tampilan parser, seperti yang diharapkan hanya titik koma atau koma:

Kasus serupa dengan yang sebelumnya:

Apakah itu titik koma atau koma, parser php akan senang. Misalnya

        echo <<<TEXT
            some string 
        TEXT, 'some other string';

Di mata parser, ini sama dengan:

    echo 'some string', 'some other string';

Berikut ini adalah contoh yang valid, karena parser mengetahui bahwa ' TEXT ' dan ' TEXTUAL ' adalah dua string yang berbeda:

Berikut beberapa cuplikan yang tidak valid karena lekukan yang salah. Khususnya, untuk pernyataan RFC: "Jika penanda penutup menjorok lebih jauh dari garis tubuh mana pun, maka ParseError akan dilemparkan:"

@jespinal terima kasih, tetapi jika Anda memiliki teks normatif itu lebih baik :) Selalu sulit untuk menebak logika hanya berdasarkan beberapa kasus, sedangkan jika kami memiliki teks normatif, kami hanya dapat mengimplementasikannya dan semoga berhasil. Dan sebenarnya, saya pikir kita sudah cukup dengan tautan @masatake dan info Anda :+1:

@masatake Saya tidak menjanjikan apa-apa mengingat sedikit waktu yang saya temukan akhir-akhir ini, tetapi saya akan mencoba untuk segera melihatnya kecuali -- Anda mengalahkan saya :)

BTW @jespinal jika tidak ada yang mengeluh saya benar-benar berpikir itu karena sangat sedikit penggunaan sintaks tersebut, dan kami mendukung sintaks pra-7.3, jadi satu-satunya kasus di mana orang akan melihat masalah adalah dengan penggunaan sintaks 7.3+, yang menyiratkan menggunakan neredoc/nowdoc di tempat pertama :)

Anda tidak aktif untuk sementara waktu. Jadi saya tidak berharap mendapat komentar dari Anda.
Tapi, sekarang kami mendapatkan tanda "ditugaskan sendiri" dari Anda. @b4n , terima kasih atas

@masatake Anda bijaksana untuk tidak berharap banyak dari saya, karena saya memang tidak menemukan waktu untuk banyak kontribusi UCtags/Geany akhir-akhir ini :disappointed: . Saya mencoba mencari cara untuk mengalokasikan waktu di sini lagi, jadi saya harap saya akan lebih aktif lagi, tetapi saya belum bisa berjanji.

Meskipun demikian, lihat #2734 untuk perbaikan masalah yang dihadapi :)

Terima kasih!