Stlink: एकीकृत चर प्रकार के साथ स्वच्छ कोड

को निर्मित 6 अप्रैल 2020 · 6टिप्पणियाँ · स्रोत: stlink-org/stlink

जहां तक मैं देख सकता हूं, कई फ़ंक्शन निश्चित-लंबाई वाले चर का उपयोग करते हैं। यह उन कार्यों के लिए वास्तव में उपयोगी है जिन्हें निश्चित डेटा प्रारूप से निपटने की आवश्यकता होती है। हालाँकि, निश्चित-लंबाई वाले चरों का दुरुपयोग संभावित निर्माण विफलताओं और अनावश्यक कास्ट का कारण बन रहा है।

उदाहरण के लिए src/usb.c में:

send_recv txsize और rxsize को size_t rxsize रूप में स्वीकार करते हैं, लेकिन वास्तविक संदर्भ में, उन तर्कों को int या unsigned int डाला जाना चाहिए, बिना एक निश्चित लंबाई, और जो कोई भी इसे कॉल करता है वह एक अलग प्रकार के तर्क प्रदान करता है ( uint32_t या इस मामले में जादू संख्या)। 32 बिट से 64 बिट तक अनजाने कास्ट कॉलिंग प्रक्रिया पर लागू होता है। साथ ही, बिल्ड विफलता को रोकने के लिए, कई स्पष्ट कास्ट की आवश्यकता होती है।

मेरा सुझाव है कि निश्चित लंबाई के चरों को उनके वास्तविक उपयोगों के अनुसार बदलना।
अधिक पोर्टेबल होने के लिए फजी रहना।

अगर दूसरों को यह सार्थक लगे तो मैं इसे आज़माना चाहूंगा।

codfeedback codrefactoring needissuer-feedback

स्रोत

chenguokai

👍1

सभी 6 टिप्पणियाँ

इसे एजेंडे में रखने के लिए Thx। चूंकि यह पूरे कोडबेस में एक सामान्य समस्या है, मुझे इस समय यकीन नहीं है कि हमें इसे सीधे v1.6.1 में डाल देना चाहिए। @martonmiklos ने पहले कोड शैली से संबंधित कुछ सफाई करने का अनुरोध किया था, जिसे मैंने v1.6.2 मील के पत्थर में धकेल दिया था। हो सकता है कि यह इस मुद्दे के लिए भी एक अच्छी जगह होगी, हालांकि यह इसे थोड़ा और आगे बढ़ाएगी।

Nightwalker-87 6 अप्रैल 2020

हो सकता है कि मैं पहले स्थानीय रूप से कुछ सफाई कर सकूं और विकास प्रक्रिया का पालन कर सकूं।😃

chenguokai 6 अप्रैल 2020

खैर निश्चित रूप से आप कर सकते हैं, लेकिन से दूर शाखा कृपया सुनिश्चित करें develop के बजाय और रखने के लिए, इस शाखा निम्नलिखित master , जो अब लगातार परिवर्तन के लिए प्रयोग किया जाता है, लेकिन केवल विज्ञप्ति और हॉटफिक्सेस के लिए।

Nightwalker-87 6 अप्रैल 2020

👍1

ILP32, LLP64 और LP64 के बीच संगतता के संबंध में, मुझे लगता है कि डेटा प्रकार long को int32_t से बदलना एक अच्छा विचार होगा, लेकिन यदि कोई भविष्य में इससे संबंधित आश्चर्यों से इंकार करना चाहता है रूपांतरण, हम जहां भी संभव हो निश्चित चौड़ाई पूर्णांक प्रकारों में जाने पर विचार कर सकते हैं ...

Nightwalker-87 9 मई 2020

👍1

@chenguokai : मैं इसे आप पर छोड़ दूँगा। आप योगदान दे सकते हैं, यदि आप चाहते हैं और आपके पास कुछ समय है।

Nightwalker-87 3 जून 2020

ज़रूर, मैं रिलीज़ से पहले करूँगा। अभी व्यस्त हो रहा है

chenguokai 7 जून 2020

👍1

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Stlink: एकीकृत चर प्रकार के साथ स्वच्छ कोड

सभी 6 टिप्पणियाँ

संबंधित मुद्दों