Comments (4)
@elfring Hello, thanks for your suggestion, which I appreciate.
Please can you be so kind to file a formal PR (pull request) with a working(!) and tested(!) code, which can be added to the present methods.
from agrep.
I suggest to avoid ignorance of return values a bit more.
Are you interested to apply aspect-oriented software development?
How do you think about to encapsulate error detection and corresponding exception handling as a reusable aspect in your software?
from agrep.
@elfring you wrote
I suggest to avoid ignorance of return values a bit more.
Yes, that was what I tried when porting this "piece" of software to OS/2 and DOS in 90ies - later I backported to Linux. The software was and is ugly (even the main author, Udi, agreed) and needs massive refactoring. So the underlying goal should be: keep it working (agrep can be compiled for many platforms, and we should not break this), but clean up messy things.
Personally, I don't have any free resources to fix also these things, and would be happy, if I can avoid all compiler warnings (there are so many, I tried to fix the "missing function prototype" issues, but could not even fix all of them)
@elfring perhaps you can starting working and helping me here: try to help to bring the compiler warnings to an absolute mininum.
from agrep.
Can a tool like "Artistic Style" help here?
How do you think about to improve static source code analysis also for your software?
Do you find information sources like the following useful?
- Article "A Reusable Aspect for Memory Allocation Checking"
- Document "Aspect-Oriented Software Development - An Introduction" by Johan Brichau and Theo D'Hondt
- Paper "On Typesafe Aspect Implementations in C++" by Daniel Lohmann and Olaf Spinczyk
from agrep.
Related Issues (20)
- [TODO] Fix the built-in copyright message text HOT 1
- Analyse agrep source code with Coverity Scan HOT 1
- missing (greplike) recursion -R HOT 1
- get match position in record
- utf-8 support HOT 4
- Missing (greplike) context lines -B and -A
- illegal pattern with "-w -i B" and a comma in expression HOT 1
- Make errors — MacOS Catalina, v10.15.7 HOT 5
- 32 characters limit: 64bit? HOT 5
- Create git tags and GitHub releases for released versions of agrep
- Add install target in Makefile HOT 1
- Remove OS, compiler version, date, and time from version string
- Remove duplicate files HOT 1
- Add missing options to help text and reformat
- Possible missing breaks in switch statement in agrep_init
- Remove references to webglimpse.net HOT 1
- deletions in output beginning 1023 characters after record start HOT 1
- build error HOT 4
- output by cost instead of # errors HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from agrep.