* Re: [devel] Предлагаю приложить патчик к less @ 2007-04-26 22:47 ` Dmitry V. Levin 2007-04-27 8:34 ` Slava Semushin 2007-04-27 8:38 ` Damir Shayhutdinov 0 siblings, 2 replies; 3+ messages in thread From: Dmitry V. Levin @ 2007-04-26 22:47 UTC (permalink / raw) To: ALT Devel discussion list [-- Attachment #1: Type: text/plain, Size: 613 bytes --] Hi, On Thu, Apr 26, 2007 at 11:37:40AM +0400, Slava Semushin wrote: > В NetBSD к less прикладывают патчик, который, насколько понимаю, > предотвращает закрытие файла, который не был открыт. Это было найдено > с помощью Coverity. Патч не большой, доступен здесь: > > http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/misc/less/patches/patch-ac?rev=1.1&content-type=text/x-cvsweb-markup > > Я предлагаю его приложить к нашей сборке. А какой смысл в этом патче, какую потенциальную проблему он призван исправить? Неужели есть на свете операционные системы, в которых close(-1) несёт угрозу? -- ldv [-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --] ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [devel] Предлагаю приложить патчик к less 2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin @ 2007-04-27 8:34 ` Slava Semushin 2007-04-27 8:38 ` Damir Shayhutdinov 1 sibling, 0 replies; 3+ messages in thread From: Slava Semushin @ 2007-04-27 8:34 UTC (permalink / raw) To: ALT Devel discussion list 27.04.07, Dmitry V. Levin<ldv / altlinux.org> написал(а): [...] > А какой смысл в этом патче, какую потенциальную проблему он призван исправить? Смысл в том, что если файл не открылся, то и закрывать его не надо пытаться. Проблемы для пользователя, здесь нет, зато есть не очень хороший код. > Неужели есть на свете операционные системы, в которых close(-1) несёт > угрозу? Вряд ли есть :) Просто close() вернёт ошибку да и всё. Ок, понял, что патч не нужен... -- + Slava Semushin | slava.semushin @ gmail.com + ALT Linux Team | php-coder @ altlinux.ru ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [devel] Предлагаю приложить патчик к less 2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin 2007-04-27 8:34 ` Slava Semushin @ 2007-04-27 8:38 ` Damir Shayhutdinov 1 sibling, 0 replies; 3+ messages in thread From: Damir Shayhutdinov @ 2007-04-27 8:38 UTC (permalink / raw) To: ALT Devel discussion list > А какой смысл в этом патче, какую потенциальную проблему он призван исправить? > Неужели есть на свете операционные системы, в которых close(-1) несёт > угрозу? Убрать лишний системный вызов aka context switch. ;) Впрочем для интерактивных программ (таких как less) это не актуально - пользователь тормозит больше чем любой context switch :) А для всякой эмбедщины может быть полезно. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-04-27 8:38 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2007-04-26 22:47 ` [devel] Предлагаю приложить патчик к less Dmitry V. Levin 2007-04-27 8:34 ` Slava Semushin 2007-04-27 8:38 ` Damir Shayhutdinov
ALT Linux Team development discussions This inbox may be cloned and mirrored by anyone: git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \ devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru public-inbox-index devel Example config snippet for mirrors. Newsgroup available over NNTP: nntp://lore.altlinux.org/org.altlinux.lists.devel AGPL code for this site: git clone https://public-inbox.org/public-inbox.git