This topic is now locked.
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, CCGHQ Admins
6 posts
• Page 1 of 1
This topic is now locked.
by Max mtg » 06 Nov 2011, 22:06
I am pretty sure that jendave is an idiot.
Because he:
1. Added a multitude of useless auto-generated javadocs to follow some stupid checkstyle rules ("there must be javadoc, no matter that the function header is as much informative") (11666 for instance)
2. Calls merging different functionalities into a single file a "refactoring". (see questdata in 11672)
3. Has spent a lot of time commiting all those "checkstyle" changes that yield no use at all. (all off above and much more)
4. Must have commited lots of other crimes we haven't found yet.
Blame on you, jendave.
I'm crying now, once I saw my efforts to write neat and compact code undone. Won't write a single line of code for such a dumbass, who thinks chicken-style rules more valuable that others' joy of creating things.
Did there anyone benefit from his changes?
Because he:
1. Added a multitude of useless auto-generated javadocs to follow some stupid checkstyle rules ("there must be javadoc, no matter that the function header is as much informative") (11666 for instance)
2. Calls merging different functionalities into a single file a "refactoring". (see questdata in 11672)
3. Has spent a lot of time commiting all those "checkstyle" changes that yield no use at all. (all off above and much more)
4. Must have commited lots of other crimes we haven't found yet.
Blame on you, jendave.
I'm crying now, once I saw my efforts to write neat and compact code undone. Won't write a single line of code for such a dumbass, who thinks chicken-style rules more valuable that others' joy of creating things.
Did there anyone benefit from his changes?
Last edited by Max mtg on 06 Nov 2011, 22:20, edited 1 time in total.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
This topic is now locked.
by friarsol » 06 Nov 2011, 22:15
Max, you really need to find a more constructive way to bring up your issues. Calling people an idiot and a dumbass are neither constructive, nor useful in a group environment such as ours.
- friarsol
- Global Moderator
- Posts: 7593
- Joined: 15 May 2010, 04:20
- Has thanked: 243 times
- Been thanked: 965 times
This topic is now locked.
by Max mtg » 06 Nov 2011, 22:27
Ok, I've fixed the title to sound more correct.
I've raised the issue a few times. The group seems to be fine with current so-called javadocs. Then its cancer.
I've raised the issue a few times. The group seems to be fine with current so-called javadocs. Then its cancer.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
This topic is now locked.
by slowe » 07 Nov 2011, 04:43
Max, I understand where you're coming from. I don't develop Forge, but sometimes I poke around in the code, and it's frustrating to find auto-javadocs instead of useful comments.
However, I'd advise against personal attacks like this. If you'd made this topic to bring up points 1-3, explained that you thought they were not helpful to the project, and asked jendave why he makes those changes, you'd accomplish what you want more quickly and with less hard feelings. If you insult and attack someone, he's not going to feel as comfortable working out a solution... and it distracts from the real issues.
We know jendave's not an idiot, and you're not making any friends by using that tone. Calm down and argue maturely.
However, I'd advise against personal attacks like this. If you'd made this topic to bring up points 1-3, explained that you thought they were not helpful to the project, and asked jendave why he makes those changes, you'd accomplish what you want more quickly and with less hard feelings. If you insult and attack someone, he's not going to feel as comfortable working out a solution... and it distracts from the real issues.
We know jendave's not an idiot, and you're not making any friends by using that tone. Calm down and argue maturely.
This topic is now locked.
by Sloth » 07 Nov 2011, 07:52
No one on this forum uses language that is even close to yours, Max. If you want your voice to be heard do it in another way.
I will refuse to say anything on this topic if it is started in this manner.
I will refuse to say anything on this topic if it is started in this manner.
-

Sloth - Programmer
- Posts: 3498
- Joined: 23 Jun 2009, 19:40
- Has thanked: 125 times
- Been thanked: 507 times
This topic is now locked.
by Max mtg » 07 Nov 2011, 17:53
Sloth, you have heard me, and the pitiful jendave has.
Noone here is brave enough to overthrow the vexatious checkstyle rules or blame on jendave for forcing them on code that never needed those autogenerated shitty comments.
Noone here is brave enough to overthrow the vexatious checkstyle rules or blame on jendave for forcing them on code that never needed those autogenerated shitty comments.
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
6 posts
• Page 1 of 1
Who is online
Users browsing this forum: ArchieRoW and 26 guests