Hacker Newsnew | past | comments | ask | show | jobs | submit | arsham's commentslogin

Rock In Peace.


I understand the author wants to compare the options but I cannot resist to leave a few comments here:

1. I'm not against test libraries/frameworks. In fact I've used Ginkgo and it is fantastic. Some people have came up with clever ways to create those lib/frameworks, but learning another library doesn't come close to enhancing your ability to write better tests.

2. Repetition is the nature of writing tests. In fact, the more you write the same code for testing different parts of the logic, the more strong your API will get. I mean when you make a mistake, you will break more tests. The drawback is you need to know the API well, otherwise you'll face refactoring hell.

3. The more you get near to standard library in general, the more new-comers to your code understand it. You cannot assume everyone knows the library you use in your code, but you can be sure if they are reading your tests, they know a bit of Go and that is quite enough.

4. Table driven tests help with reducing repetition and is so much fun. You can also add more test cases in only one line in the future if you need to. Here is my take on testing the code in the article:

    func TestOperations(t *testing.T) {
		tcs := []struct {
			name string
			a, b int
			f    func(int, int) int
			want int
		}{
			{"Add 1 2", 1, 2, tmp.Add, 3},
			{"Subtract 5 3", 5, 3, tmp.Subtract, 2},
			{"Multiply 5 6", 5, 6, tmp.Multiply, 30},
		}
		for _, tc := range tcs {
			t.Run(tc.name, func(t *testing.T) {
				result := tc.f(tc.a, tc.b)
				if result != tc.want {
					t.Errorf("result = %d, want %d", result, tc.want)
				}
			})
		}
	}
Of course this is for a trivial program. In real world you find yourself test more complex functions and you will create one Test... function for each function.

5. The happy path of tests in BDD style frameworks are indented way too much. This distracts you a bit from the test logic.

Thanks for sharing this awesome article.


I've also switched to Opera a couple of months ago. I'm running Opera touch on android too. I've found Opera on desktop (Linux) to be faster than Firefox and Chrome and the only functionality I'm missing is "to move the tabs to left and right". Even the chromecast works on Opera, which is fantastic. I've been using the Flow functionality ever since I've started using Opera, and I think that is the second reason (well, after it being fast) I'm going to stick with Opera.


Thank you for doing the comparison. Would you do the same against the latest version (v0.5.0) please? Thank you.


Cheers mate!


Handling signals are not implemented yet. I appreciate it if you file an issue when you find any. Thanks.


That is correct. The project is at its early stages. I want to see what the community need the most and shape the project towards that goal. On the other hand I tried to avoid optimisations until most of functionalities are implemented.


It's a very nice idea and you should be proud of what you've built, but my personal opinion is that speed is a core feature of `grep`.

A good place to start would be this: why GNU grep is fast[1] - Starting with the Boyer-Moore string search algorithm and reading through the optimizations done in GNU grep.

p.s. there's an implementation of Boyer-Moore hiding in Go's standard library.

[1] https://lists.freebsd.org/pipermail/freebsd-current/2010-Aug...


Thanks mate, I will definitely have a read.


Note that you don't need Boyer Moore for the common case. ripgrep for example will very rarely use Boyer Moore. Its work horse is much simpler and typically faster: https://github.com/rust-lang/regex/blob/master/src/literal/m...

In Go-land, you should be able to replace uses of memchr with IndexByte[1], which should be implemented in Assembly on most platforms.

Of course, for any of this to have a big impact, you'll want to take Mike Haertel's advice on avoiding line breaking and stop using bufio.Scanner. :-)

[1] - https://golang.org/pkg/bytes/#IndexByte


So far I've been only concerned about code's simplicity until I understand what there needs to be done. This is not going to be grep or ripgrep. My intent was to make a tool I needed so I started working on it. I thought someone else might like it, now it is joyful to see people are looking at the project.

There are a couple of places I wish I would have done better. Using bufio.Scanner actually bothers me a lot. Also in the Read() method it reads everything from all readers into a buffer instead of pulling what it needs to check.

Thanks for suggestions :)


There is a --no-colour/--no-color argument you can pass.


It sounds interesting. Is the language open-sourced?


> The language will be open-sourced later in 2018.


Given the relationship between eul author and deadlines it won't happen in 2018. Author also promised to open-source eul itself at first but last statement is "I'm not 100% sure it's going to be possible."[0]

Current state of eul app also hinting that most likely nothing of value would be lost if this language won't be open-sourced.

[0] - https://github.com/eul-im/eul/issues/45#issuecomment-3356427...


That's nice. I'm using https://github.com/bhilburn/powerlevel9k with zsh.


Author of P9k, here. Glad you're enjoying it! =)


Mate, P9k is fantastic, thanks for developing it!


Thanks so much! Wait till you see the next release... we have some huge improvements coming =)


Can't wait!


Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: