Replies: 10 comments 3 replies
-
Files that you're referring to are imported from official SQLite sources (though from quite an old version of SQLite). I'm not familiar enough with their details of implementation to comfortably modify them. It would take significant effort to analyze them first. I don't think the benefit is worth the effort. |
Beta Was this translation helpful? Give feedback.
-
How much does this software depend on the component “LEMON LALR(1) parser generator”? 💭 I suggest to avoid ignorance of return values a bit more. |
Beta Was this translation helpful? Give feedback.
-
A lot. This is the parser used to understand SQL syntax of SQLite (for code assistant, code formatter, editing DDL objects, and a lot more). Why do you ask? |
Beta Was this translation helpful? Give feedback.
-
💭 I hope that more code reviewers and developers will become interested in corresponding software improvements. |
Beta Was this translation helpful? Give feedback.
-
I see. Just keep in mind, that SQLiteStudio uses relatively old version of Lemon parser and I had no need to update it so far. Updating is difficult, because I already introduced significant SQLiteStudio-specific changes to the Lemon included with SQLiteStudio. |
Beta Was this translation helpful? Give feedback.
-
🤔 How do you think about to improve static source code analysis also for this software? |
Beta Was this translation helpful? Give feedback.
-
I use embedded clang static analysis (as QtCreator plugin) since few months and I'm improving it on-the-fly whenever I touch files. |
Beta Was this translation helpful? Give feedback.
-
💭 Would you become interested in aspect-oriented software development? |
Beta Was this translation helpful? Give feedback.
-
Why all these questions? |
Beta Was this translation helpful? Give feedback.
-
💭 Some advanced development tools can help more to fix remaining open issues. 🤔 Can any more developers become aware that error detection belongs to known cross-cutting concerns? |
Beta Was this translation helpful? Give feedback.
-
Would you like to add more error handling for return values from functions like the following?
Beta Was this translation helpful? Give feedback.
All reactions