Board.KolibriOS.org

Official KolibriOS board
It is currently Sat May 25, 2019 8:21 pm

All times are UTC+03:00




Post new topic  Reply to topic  [ 98 posts ]  Go to page Previous 13 4 5 6 7 Next
Author Message
PostPosted: Mon Oct 17, 2011 10:38 pm 
Offline
Just Flooding

Joined: Sat Jan 06, 2007 2:30 pm
Posts: 269
to turbanoff
Code:
sed 's/\t/\ \ \ \ \ \ \ \ /g'

Какую ещё корректировку надо?
Хех, помню как на меня тут ругались (4 года назад) за оформление исходников. А я просто использовал только редактор fasm'а с его фичей optimal saving когда используются вперемешку табы и пробелы по принципу 8 пробелов = 1 таб.
Ныне использую только табы, ну да это не важно совсем.


Top
   
PostPosted: Mon Oct 17, 2011 11:33 pm 
Offline
Kernel Developer
User avatar

Joined: Thu Sep 03, 2009 1:52 pm
Posts: 1619
Nable, это, во-первых, некорректная замена, а во-вторых, даже не пытается учесть правила помимо табуляции.
turbanoff, я конвертировала наколеночным скриптом
Code:
#!/bin/bash
FILE=$1
expand -t8 $FILE | perl -lne 'if(!$z&&m/^(\s*[^;"\x27\$][^;"\x27\s]*:)(\s*)([^;\s].*)$/){print "$1\r\n"," " x (length($1)+length($2)),$3}else{print}$z=(m/\\\s*(;.*)?$/)' > $FILE.1
perl -n /home/users/clevermouse/beautifyer/beautifyer.pl $FILE.1 > $FILE
rm $FILE.1

expand раскрывает табуляцию, следующий регэксп выделяет метки и печатает их на отдельной строке, отступы настраивает приложенный скрипт
Attachment:
beautifyer.pl.7z [3.5 KiB]
Downloaded 651 times
Для массового исправления, соответственно, из командной строки
Code:
for a in `find -iname '*.asm' -o -iname '*.inc'`;do ./beautifyer.sh $a;done

_________________
Сделаем мир лучше!


Top
   
PostPosted: Sat Dec 10, 2011 7:16 pm 
Offline
User avatar

Joined: Fri Jun 27, 2008 3:22 pm
Posts: 988
Вопрос: распространяются ли правила форматирования кода на драйвера?


Top
   
PostPosted: Mon Dec 12, 2011 1:35 am 
Offline
Just Flooding

Joined: Sat Jan 06, 2007 2:30 pm
Posts: 269
тут не так давно писали что ты на svn не сможешь затолкнуть ничего текстового, содержащего \t, так что похоже что распространяются.


Top
   
PostPosted: Mon Dec 12, 2011 2:19 am 
Offline
Site Founder
User avatar

Joined: Sun Aug 08, 2004 8:55 am
Posts: 689
Asper wrote:
Вопрос: распространяются ли правила форматирования кода на драйвера?

А почитать тему?
CleverMouse wrote:
Ещё раз: на репозиторий вне kernel/trunk правила не распространяются.

Соответственно, если исходники лежат не в /kernel/trunk/drivers (а, скажем, в /drivers), то правила форматирования на них *не распространяются*. С другой стороны, я бы заметил, что это не значит, что их *не стоит* там применять.

_________________
in code we trust


Top
   
PostPosted: Mon Dec 12, 2011 5:31 am 
Offline
Kernel Developer

Joined: Wed Mar 08, 2006 6:25 pm
Posts: 3952
mike.dld wrote:
С другой стороны, я бы заметил, что это не значит, что их *не стоит* там применять.
Спасибо, но лучше не надо. Особенно к портированным.


Top
   
PostPosted: Mon Dec 12, 2011 12:52 pm 
Offline
Site Founder
User avatar

Joined: Sun Aug 08, 2004 8:55 am
Posts: 689
Про применение правил к C/C++ коду никто никогда ничего не говорил, а портированного ASM кода я как-то не помню, по крайней мере в сфере драйверов. Так в чём проблема? И я вообще-то имел в виду (вероятно, не совсем корректно выразился) самостоятельное соблюдение правил, без принуждения. Ну да ладно... Может быть, когда-нибудь, кто-нибудь поймёт...

_________________
in code we trust


Last edited by mike.dld on Mon Dec 12, 2011 12:59 pm, edited 1 time in total.

Top
   
PostPosted: Mon Dec 12, 2011 12:54 pm 
Offline
Kernel Developer
User avatar

Joined: Thu Sep 03, 2009 1:52 pm
Posts: 1619
Из заглавного сообщения темы:
CleverMouse wrote:
svn-сервер будет просто отклонять коммиты, выполнение которых означало бы появление в kernel/trunk файлов *.asm или *.inc, нарушающих оформление

_________________
Сделаем мир лучше!


Top
   
PostPosted: Mon Dec 12, 2011 5:47 pm 
Offline
User avatar

Joined: Fri Jun 27, 2008 3:22 pm
Posts: 988
Ok, спасибо.
mike.dld wrote:
Может быть, когда-нибудь, кто-нибудь поймёт...

Я понял и ничего против заявленных правил форматирования не имею, хотя бы потому что эти правила по большей части соответствуют моему собственному стилю.
Вопрос я задал, поскольку собираюсь в ближайшем будущем залить код HDA драйвера на SVN.
Кстати код драйвера HDA находится в нескольких файлах, может создать для него отдельную папку на в /kernel/trunk/drivers или заливать всё в кучу?


Top
   
PostPosted: Fri Jan 27, 2012 3:23 pm 
Offline
Kernel Developer
User avatar

Joined: Thu Sep 03, 2009 1:52 pm
Posts: 1619
Asper, лучше создать отдельную папку.

_________________
Сделаем мир лучше!


Top
   
PostPosted: Fri Feb 24, 2012 6:14 pm 
Code:
; init selectors
        mov     ebx, [BOOT_VAR+0x9040]          ; offset of APM entry point
        movzx   eax, word [BOOT_VAR+0x9050]; real-mode segment base address of
                                                                                ; protected-mode 32-bit code segment
        movzx   ecx, word [BOOT_VAR+0x9052]; real-mode segment base address of
                                                                                ; protected-mode 16-bit code segment
        movzx   edx, word [BOOT_VAR+0x9054]; real-mode segment base address of
                                                                                ; protected-mode 16-bit data segment

Наша новая система с поликерамидами - ваши волосы всегда великолепны! :lol:


Top
   
PostPosted: Fri Feb 24, 2012 6:36 pm 
Offline
Kernel Developer
User avatar

Joined: Thu Sep 03, 2009 1:52 pm
Posts: 1619
Глюки старой версии
Code:
; init selectors
    mov ebx,[BOOT_VAR+0x9040]         ; offset of APM entry point
    movzx eax,word [BOOT_VAR+0x9050]    ; real-mode segment base address of
                              ; protected-mode 32-bit code segment
    movzx ecx,word [BOOT_VAR+0x9052]    ; real-mode segment base address of
                              ; protected-mode 16-bit code segment
    movzx edx,word [BOOT_VAR+0x9054]    ; real-mode segment base address of
                              ; protected-mode 16-bit data segment

Spoiler: Show
Attachment:
kernel_asm_2000.png
kernel_asm_2000.png [ 5.79 KiB | Viewed 4929 times ]

сохранены для совместимости.

_________________
Сделаем мир лучше!


Top
   
PostPosted: Fri Feb 24, 2012 6:55 pm 
Вообще то раз уж сказано А, то должно быть сказано Б. Почему в одних случаях можно, а в других нельзя? Надо весь код к 80 символьной границе приводить. Это же не Си какой-нибудь, чтобы рассусоливать.


Top
   
PostPosted: Sun Apr 22, 2012 1:39 am 
Offline
User avatar

Joined: Tue Aug 25, 2009 4:45 pm
Posts: 794
Раньше это было актуально. Но сейчас, имхо, ни к чему ограничивать длину строки до 80. Если уж ограничивать, то 100-120


Top
   
PostPosted: Sun Apr 22, 2012 1:56 am 
Меню больше волнует жуткая предвзятость к табам. Глупо ограничивать удобство использования.


Top
   
Display posts from previous:  Sort by  
Post new topic  Reply to topic  [ 98 posts ]  Go to page Previous 13 4 5 6 7 Next

All times are UTC+03:00


Who is online

Users browsing this forum: No registered users and 1 guest


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
Powered by phpBB® Forum Software © phpBB Limited