-
Data: 2012-05-06 21:32:03
Temat: Re: [OT] Zarządzanie konfiguracją modułów kodu źródłowego
Od: mk <reverse_lp.pw@myzskm> szukaj wiadomości tego autora
[ pokaż wszystkie nagłówki ]W dniu 2012-05-06 19:00, Sebastian Biały pisze:
> On 2012-05-06 18:18, mk wrote:
>>> Nie. plik lokalny dla projektu nadpisuje parametry domyślne.
>> Używając #undef???
>
> Uzywając def.
>
> Np. tak:
>
> default-config: #define UART_STOP_BITS 1
>
> project-config: #include "default.h", #define UART_STOP_BITS 2
Redefiniowanie makra nową wartością jest nielegalne!!!
>
>> IMO zdecydowanie lepsze jest jednak włączenie najpierw pliku z
>> konfiguracją użytkownika, a potem nagłówek z konfiguracją domyślną -- to
>> zresztą powszechna praktyka.
>
> Zdecydowana lepszośc tutaj dyskusyjna. Oba rozwiązania działają.
Jak gdzie (patrz wyżej).
Można by odwołać jak wspomniałem z użyciem #undef, ale użycie #undef
jest niezgodne np. z MISRA-C i w ogóle niepotrzebnie zaciemnia plik
konfiguracji użytkownika.
>
>> Plik nagłówkowy "defaultconfiguration.h" nie definiuje parametrów już
>> zdefiniowanych (#ifndef), a jedynie te które jeszcze nie zostały
>> zdefiniowane nadając im wartości domyślne (+ ewentualnie jakiś komunikat
>> przy pomocy #warning), obliczając je (np. na podstawie parametrów
>> podanych przez użytkownika) lub walnąć błędem (#error) jeśli parametr ma
>> być obligatoryjne określony przez użytkownika.
>
> W drugą stronę działa to tak samo (nie)skutecznie bez #ifndef, kwestia
> implementacji.
Niezgodne ze standardem.
> IMO to nie konfiguracja powinna szukać bledów w #define i wyliczać
> pośrednie parametry tylko implementacja.
To zależy i to już trochę inny temat... część rzeczy można i należy
sygnalizować już na wczesnym etapie rozpoznania konfiguracji użytkownika
np. zgodność wersji modułu bibliotecznego i jej użytkownika, czy wiele
innych parametrów o globalnym charakterze.
Rozmowa na temat czy "default.h" należy czy nie należy do implementacji
jest również czysto akademicka, tak jak i gdzie najlepiej określać
parametry default.
pzdr
mk
Następne wpisy z tego wątku
- 06.05.12 22:05 Sebastian Biały
- 06.05.12 22:25 mk
- 06.05.12 23:02 Andrzej Ekiert
- 07.05.12 01:25 mk
- 07.05.12 02:21 Michoo
- 07.05.12 15:52 Nijak
- 07.05.12 21:23 Andrzej Ekiert
- 07.05.12 21:42 Paweł
- 07.05.12 22:46 RoMan Mandziejewicz
- 08.05.12 07:56 Paweł
- 08.05.12 08:50 RoMan Mandziejewicz
- 08.05.12 15:13 janusz_kk1
- 09.05.12 09:07 Artur M. Piwko
- 09.05.12 10:03 janusz_kk1
- 25.05.12 16:09 Artur M. Piwko
Najnowsze wątki z tej grupy
- Router LTE z możliwością zmian MTU
- Fajny film widziałem...
- Jaka ładowarka sieciowa do Iphona?
- Taśma izolacyjna do prac elektrycznych
- Recenzja 3.1A ;) w 6 gniazdach...
- Re: Recenzja 3.1A ;) w 6 gniazdach...
- Re: Recenzja 3.1A ;) w 6 gniazdach...
- Re: Recenzja 3.1A ;) w 6 gniazdach...
- Wkrętarki, wiertarki...
- Zasilacz impulsowy 12V 10A, coś godnego uwagi jako zamiennik akumulatora wkrętarki
- Mouser - koszt wysyłki
- [OT] Jak wycinac ksztalt w piance lub styropianie?
- FV--> ciepła woda w kranie
- Szok
- Dziwny schemat wzmacniacza m.cz.
Najnowsze wątki
- 2025-07-08 Router LTE z możliwością zmian MTU
- 2025-07-08 Re: Pożar w Ząbkach a polscy dyletanci
- 2025-07-08 Trójmiasto => Head of Social Media <=
- 2025-07-08 Warszawa => MENA New Business Manager <=
- 2025-07-08 Środa Wielkopolska => SAP FI/CO Internal Consultant <=
- 2025-07-08 Warszawa => Customer Service with Spanish + translation <=
- 2025-07-08 Warszawa => Senior Account Manager <=
- 2025-07-08 Parkometry bez podstawy prawnej
- 2025-07-07 Re: Ząbki się spaliły jak wiejskie, drewniane stodoły sprzed 50 lat
- 2025-07-06 Kup szybko nową ładowarkę do smartfona
- 2025-07-07 TV z Play (dawniej UPC) -- potrzebny dekoder?
- 2025-07-06 Kup szybko nową ładowarkę do smartfona
- 2025-07-07 mija rok jeżdzenia po lewej
- 2025-07-06 Elektryki jednak są NIEBEZPIECZNE
- 2025-07-08 Fajny film widziałem...