Newsgroups: php.internals Path: news.php.net Xref: news.php.net php.internals:126479 X-Original-To: internals@lists.php.net Delivered-To: internals@lists.php.net Received: from php-smtp4.php.net (php-smtp4.php.net [45.112.84.5]) by qa.php.net (Postfix) with ESMTPS id 463DB1A00BC for ; Sun, 23 Feb 2025 15:05:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=php.net; s=mail; t=1740322969; bh=rasAe+Z35BjCN+agLAvwZVD+OZgWoqqiJ1FsLj4xt+Q=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=YoPZtk07sLWgosnekdVTW1kph2X2ItOw3c7i4knuDIlZLJghykiNBmzY0fzAIi9wh 9KN+7mLxC6bIHdLPL44FP21PK+0MoLMVkgukF8MrcI0+0X9WI6lpZY3SgleUxlBmfC TRGi3n5R1UKjWx/kmqluLZPwS6tDFsn17W4Xb7rTchDMIo2xg88Hyb5OlQlz9DdJwI uBpckrOvtkiprHmcVRaDBtBAFTIZ3xT4x8280KMdPgZmaCfdiDRZRrooqm9zpIoodb Is9JDdh7WcG0KKuU6YOmUh0U5cKr7/b2+sRl34feDnxlOdkA5MMORMSaAWrwz7EZs8 zr3TBnB9G0hHw== Received: from php-smtp4.php.net (localhost [127.0.0.1]) by php-smtp4.php.net (Postfix) with ESMTP id D230F18062C for ; Sun, 23 Feb 2025 15:02:48 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 4.0.0 (2022-12-13) on php-smtp4.php.net X-Spam-Level: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,DMARC_PASS,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=4.0.0 X-Spam-Virus: No X-Envelope-From: Received: from chrono.xqk7.com (chrono.xqk7.com [176.9.45.72]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by php-smtp4.php.net (Postfix) with ESMTPS for ; Sun, 23 Feb 2025 15:02:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bastelstu.be; s=mail20171119; t=1740323125; bh=07r12iNZ08yvsIimZI9g/ErFSRLorSmlyQJqafSjDXI=; h=MIME-Version:Date:From:To:Cc:Subject:In-Reply-To:References: Message-ID:Content-Type:from:to:cc:subject:message-id; b=lv2HXNKyia5X2LRbxottgVTtIoIfl7Em/kzvVYpVJYv40scFzCf8h2yENFpermSlu +LBP/Sgq5apmxcAhJ7D5zm64C6V+oxOOoVRKSDgA9MDmfta0ee3ZHzJdObMZEWrCO0 lLVjBKxYffqFf8Z0p2NTUoShRxgiNGJs8CnEdfcnSNMAw4Su3MoF7OlV3kA0Ikuy/I oFJ00WbgNx6y6Deqop40n1697nvcvzVdTP2PYLC9CQl8oNLc2zZhVHuE1pRakEbd2t vqQBhq+SyBG5/kCUguBBG8SiShEMy7jGen0KFWKxrwx5EiL0paTNvfPvH4czYgwl+w WwI69dcUEadPA== Precedence: bulk list-help: list-post: List-Id: internals.lists.php.net x-ms-reactions: disallow MIME-Version: 1.0 Date: Sun, 23 Feb 2025 16:05:25 +0100 To: =?UTF-8?Q?M=C3=A1t=C3=A9_Kocsis?= Cc: Internals Subject: Re: [PHP-DEV] [RFC] [Discussion] Add WHATWG compliant URL parsing API In-Reply-To: References: <1BCB4144-231D-45EA-A914-98EE8F0F503A@automattic.com> <8E614C9C-BA85-45D8-9A4E-A30D69981C5D@automattic.com> Message-ID: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit From: tim@bastelstu.be (=?UTF-8?Q?Tim_D=C3=BCsterhus?=) Hi [dropping Dennis from the Cc list] Am 2025-02-21 13:06, schrieb Tim Düsterhus: > We already had extensive off-list discussion about the RFC and I agree > it's in a good shape now. I've given it another read and here's my > remarks: One more thing that came to my mind, but where I'm not sure what the correct choice is: Naming of `WhatWgError` and `WhatWgErrorType`. They are placed within the `Uri\WhatWg` namespace, making the `WhatWg` in their name a little redundant. For Exceptions the recommendation is to use this kind of redundant naming, to make implicit imports for catch blocks more convenient compared to needing to alias each and every `Exception` class. The same reasoning *could* also apply here, but here I find it less obvious. The alternative would probably be `Uri\WhatWg\Error` and `Uri\WhatWg\Error\Type`. No strong opinion from my side, but wanted to mention it nevertheless. Best regards Tim Düsterhus