SOURCES: mozilla-const_cast.patch (NEW) - new patch
Jakub Piotr Cłapa
loc w toya.net.pl
Pią, 26 Lis 2004, 17:32:02 CET
Paweł Gołaszewski wrote:
> On Fri, 26 Nov 2004, [ISO-8859-2] Jakub Piotr Cłapa wrote:
>
>>A co ja mam pisać do jednolinijkowego patcha,
>
>
> Napisać co on robi.
> To, że jest to jedna linijka wcale nie znaczy, że nie ma on dużego
> znaczenia... Z małego wycinka kodu przeważnie wiele nie wynika przy
> oglądaniu commit-loga.
W tym wypadku akurat nic więcej niż jest w wycinku nie dałbym rady
napisać...
>>który możliwe, ze nawet nie jest on poprawny (niby się buduje i działa,
>>ale nie mam pojęcia, czy to tak powinno być).
>
> ...co świadczy, że nie powinieneś wogóle go dodawać, bo nie wiesz co on
> robi... ładnie...
Bez niego się nie buduje, po zaaplikowaiu takiej poprawki już tak.
Poprawka wydłubana przeze mnie z podobnego wywołania tej samej funkcji w
innym kontekscie. Lepsza taka niż żadna, nie?
>>Wersje patch nijak sie mają do wersji specy i ciężko czasem jedno z
>>drugim powiązać, więc myślę, że zdecydowanie lepiej patche opisywać w
>>commitlogu speca.
>
> w specu _też_ powinno to być opisane. Conajmniej jaki patch został dodany,
> jeżeli jest on opisany przy commicie samego patcha.
>
> Ciężko powiązać? Jakoś nigdy nie miałem z tym jakichkolwiek problemów...
> Od tego są tagi i daty.
Ale nie tagujesz przecie każdej zmiany. Daty są, ale ich uzywania nie
nazwałbym najłatwiejszym.
--
z wyrazami szacunku,
Jakub Piotr Cłapa
-------------- następna część ---------
Załącznik, który nie był tekstem został usunięty...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3387 bytes
Desc: S/MIME Cryptographic Signature
Url : /mailman/pipermail/pld-devel-pl/attachments/20041126/1bb590fc/smime.bin
Więcej informacji o liście dyskusyjnej pld-devel-pl