Vlákno názorů k článku CodeRabbit: automatická analýza a recenze zdrojových kódů od radekm - Ad C: unsigned char v C může mít...

24. 7. 2025 8:20

radekm

Stříbrný podporovatel

Ad C: unsigned char v C může mít víc než 8 bitů, takže přidání & 0xFF může změnit význam kódu. A pak ještě tu paměť na konci main není třeba uvolňovat pomocí free, to by nejspíš udělal os, ne?

Ad C3: Přijde mi, že on navrhuje smazat case MEDIUM: nebo se špatně dívám? Spíš bych totiž přidal do enumu MEDIUM,.
- Skok na další nový názor. K navigaci lze použít i klávesy N pro následující a P pro předchozí nový názor
24. 7. 2025 8:30

Pavel Tišnovský

Zlatý podporovatel

C - jj pravda

To C3 je zajímavé - jednou mi navrhoval tam přidat prvek, podruhé naopak navrhuje všude odstranit ty větve, Vlastně mám ten druhej PR ještě po ruce: https://github.com/tisnik/test-dependabot-no-devs/pull/1050

(to repo je takovej mix všeho možného, nebrat prosím vážně :-)
- Skok na další nový názor. K navigaci lze použít i klávesy N pro následující a P pro předchozí nový názor
24. 7. 2025 12:50

Wasper

Mno ... Některé konkrétní (nyní poněkud staré, pravda - konkrétně Novell NetWare 3.xx a 4.xx) platformy to explicitní free vyžadovaly. V případě klasického malloc() to skončilo "jen" pípnutím a pětiřádkovým errorem o neuvolněných prostředcích (mám pocit AllocateShortTermMemory).
U jiných (LocalSemaphore) ABEND a celý server šel do rebootu. Takže opatrně bych řekl, že je to platform dependent ;-)

Dále u nás najdete