1 |
Hello! Please feel free to modify the Makefile and open a pull request! |
2 |
What you mention about return values not checked, I believe they are part |
3 |
of sc, but |
4 |
please specify them if you can, since there's still a lot of sc for modify. |
5 |
Thanks! |
6 |
|
7 |
Andrés M. |
8 |
|
9 |
2014-11-04 10:23 GMT-03:00 Alec Ten Harmsel <alec@××××××××××××××.com>: |
10 |
|
11 |
> |
12 |
> On 11/02/2014 06:16 PM, Andrés Martinelli wrote: |
13 |
> > Hello there!! |
14 |
> > I am working on a terminal spreadsheet based on "sc", but with some |
15 |
> > adds like undo/redo.. |
16 |
> > you can find it here: |
17 |
> > |
18 |
> > https://github.com/andmarti1424/scim |
19 |
> > |
20 |
> > Any new ideas and/or contribution is always welcome! |
21 |
> > Thanks! |
22 |
> > |
23 |
> > -- |
24 |
> > Andrés M. |
25 |
> |
26 |
> If I can offer some constructive criticism based on my short experience |
27 |
> helping Walter: |
28 |
> |
29 |
> * Your build process could use some work; you shouldn't be hard-coding |
30 |
> variable values like LN and CC in a Makefile, these are handled by make. |
31 |
> Your Makefile could be shortened by ~150 lines by relying on built-in |
32 |
> rules and using some built-in make expressions to list your source files |
33 |
> as well. |
34 |
> * A bunch of your .c or .h files are marked as executable... why? |
35 |
> * No install command. Not really a huge deal as only one important file |
36 |
> is produced (src/scim), but would be nice to have |
37 |
> |
38 |
> If you're unfamiliar with make and have no immediate plans to switch to |
39 |
> either the autotools or cmake, I would be more than willing to make some |
40 |
> changes to your Makefile and open a pull request. If you would like to |
41 |
> email me personally with questions about make, feel free to do that as |
42 |
> well. |
43 |
> |
44 |
> There are a couple things I saw in the code as well: |
45 |
> |
46 |
> * system("echo -n 'Press enter to return.'") |
47 |
> * Lot of ignored return values on functions that you should check, like |
48 |
> write() and fgets() |
49 |
> |
50 |
> If these were in the sc code and you know about them but just haven't |
51 |
> gotten to fixing them yet, no worries. |
52 |
> |
53 |
> Alec |
54 |
> |
55 |
> |
56 |
|
57 |
|
58 |
-- |
59 |
Andrés Martinelli |