Skip to content

examples: ls-files: add ls-files to list paths in the index #4380

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pks-t merged 13 commits into from
May 4, 2018
Merged

examples: ls-files: add ls-files to list paths in the index #4380

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
30e919c
examples: ls-files: list files
Oct 16, 2017
991fc08
examples: ls-files: show current files in index
Oct 17, 2017
52d83dd
examples: ls-files: handle --error-unmatch
Oct 18, 2017
cd39273
examples: ls-files: fix style and refactor
cjhoward92 Oct 23, 2017
d2f99e0
examples: ls-files: fix compile error
cjhoward92 Oct 23, 2017
37cbc3e
examples: ls-files: build file list with array
Oct 23, 2017
3f64a9d
examples: ls-files: use git_array_t to handle files
Oct 26, 2017
e866116
examples: ls-files: address PR and style
Nov 14, 2017
ee14465
examples: ls-files: fix build error
Nov 14, 2017
7d07941
test: ls-files: remove dependency on git_array
Dec 19, 2017
29ca3f3
examples: ls-files: update print_paths to print all cases
Jan 6, 2018
d7394c3
examples: ls-files: print entry path only when entry is found
Mar 27, 2018
8aa437e
tests: ls-files: use puts instead of printf and fix typos
May 2, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
examples: ls-files: fix compile error
  • Loading branch information
@cjhoward92
cjhoward92 authored and Carson Howard committed Mar 27, 2018
commit d2f99e0a7f259511d94a960c54c297db2ec455af
10 changes: 7 additions & 3 deletions examples/ls-files.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ static void usage(const char *message, const char *arg)
exit(1);
}

static void parse_options(ls_options *opts, int argc, char *argv[])
static int parse_options(ls_options *opts, int argc, char *argv[])
{
int parsing_files = 0;
struct args_info args = ARGS_INFO_INIT;
Copy link
Copy Markdown
Member

@pks-t pks-t Nov 6, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not really being used, is it? You could instead just loop over argc/argv[]:

int i;
for (i = 1; i < argc; i++) {
    char *arg = argv[i];
    ...
}

Expand All @@ -58,7 +58,7 @@ static void parse_options(ls_options *opts, int argc, char *argv[])
memset(opts, 0, sizeof(ls_options));

if (argc < 2)
return;
return 0;

for (args.pos = 1; args.pos < argc; ++args.pos) {
char *a = argv[args.pos];
Expand All @@ -77,8 +77,11 @@ static void parse_options(ls_options *opts, int argc, char *argv[])
opts->error_unmatch = 1;
} else {
usage("Unsupported argument", a);
return -1;
}
}

return 0;
}

static int print_paths(ls_options *opts, git_index *index)
Expand Down Expand Up @@ -113,7 +116,8 @@ int main(int argc, char *argv[])
size_t i = 0;
int error;

parse_options(&opts, argc, argv);
if ((error = parse_options(&opts, argc, argv)) < 0)
return error;

git_libgit2_init();

Expand Down