Hi Lucas,

I am fully aware that evaluate() is not called in some cases and that attack info is not called in some cases (hence the comment in MovePicker::score<CAPTURES>() which says :  "If we have attack info, use it to tell apart urgent captures").

In fact there are two cases:
1) if attack info is present, I want to use it to give a bonus for some "urgent" captures (captures of hanging pieces by unsupported pieces) to tell them apart
2) if attack info is not present, well, it is my intention that the attackInfo values would be zero for both White and Black, so the urgent capture condition will fire for all the moves and a constant (500) will be added to all the capture values during sorting, which doesn't alter move ordering.

However it seems that Clang initializes arrays inside a struct with zeros (I have checked this locally), but some other compilers may not, so I will be more explicit in my initializations of attack info in the beginning of search() and qsearch(). Or maybe it is something else, I don't know. I will sleep over it, and then post a message in the forum if I still don't understand it.

I apologize, of course, for the small waste of resources :-( But I promise there was no warning there, and that I tested it locally.

Stéphane


Exception message surely should say there is no sim, right ? That's what the if() was testing...


Does mutating a plant change it's name?


Hi Dustin,

thanks for the pull request but I think this doesn't fix the issue. Here you just clear the view value of the `ngModel` when the entered query is not defined and not if the external model (the `ngModel` itself) is cleared. There are also failing tests in your pull request -> https://travis-ci.org/guylabs/ion-autocomplete/builds/63249979

The `searchQuery` is the internal model for the search input field for the list and not the input field where you define the external model.

Thanks and regards,

Guy


This url seems a _liittle_ incorrect.


Hi @mafagafogigante yes, I would say so. You could see if you can find how others translate the same word in a similar context however we are very grateful for your help on this. If you pick up any tips or anything whilst doing this please let me know, I am keen to document this process to help others.


Do you have a sample pcap file with this option and
1) only `Purge Originator System-ID`
2) `Purge Originator System-ID` and `Received from System-ID`?


@skeptical agreed on both counts, nice work @valib !


add a comment here ?


should the ctor be private ?


lkdfjls;a


Why use a list and not a set? If there's a huge amount of distinct links, this is unnecessary slow.


and maybe rename to stackTrace ?


Did you had a chance to look at it?


This might be my last commit for a bit, depending on how today goes. This API is a load of shit, and to be honest, I don't really have the motivation to work on reimplementing the clusterfuck. Especially not looking forward to sorties and stuff.
At this point, it'd be much better to just re-write the entire client from scratch using HTML5 canvas or something using apiv2, because then at least I'd have control over what is sent or not sent, and could remove like 90% of the redundancy (call these three API end points, then promptly discard them and use port end point instead).
Also, I decided to start writing these just as exam season starts back up, and as a year 10 I get the lucky advantage of stressing over my GCSEs at 15 instead of 16.
Maybe I should post to reddit or himeuta channel and see if I could get any help writing this, because it's far from a one-man project.


that won't work. The link is empty and not clickable. look at the current website, figure out what code makes this DIV have the climateinteractive logo as background through the "inspect element" function of chrome, and then add an onclick='location.href="http://www.climateinteractive.org"' to that DIV.


'Canoobly' oil? Can't you spell?


Xenophage plant entry is immutable.


Root cause of this problem is incorrect input list for bad gradients. We can find some irrelevant input gradients indices (e.g. like out of range index or repeated index) and discard them, but I think the safest way is to ask human rater to review the incorrect passed gradient list and correct the wrong indices.


oops... debug :-)


I mistake this commit... :scream:
see https://github.com/kuronekomichael/peak/commit/bc667e92eb6e7b64afb14444c2e158a9c93e090a


Fixed spelling


Hum, is it about the execfile?
Why configparse can't be use for this?


src/test/java/nl/tudelft/ti2806/pl1/zoomlevels/PointGraphConverterTest.java line 33: heeft die Thread.sleep(best wel heel lang) een belangrijke functie? NodePlacer is niet threaded toch?


I actually thought that was a typo referring to new_seq, I'll remove it.


Very minor suggestion, should we do a signal_cb.result !=  signal_cb check and then throw the timeoutException ?.  That may lead to dirty code but this is the situation which lead to deadlock in our case the callback chain just finished in time to join, better to have result than exception


We did this. And the answer was a "no", but this was 2 or more years ago. At some point biopython had the plan to split the package into several smaller packages, but I guess this will also take some time.


I think this should be `if server is not None`, no?


Nice PR!


Hi Martin, no, I made sure to base this branch on master. First I updated our master with latest changes (from upstream repo), and then created new branch out of it. You can verify this by looking at past commits, or by peeking inside RELEASE file (OTRS 5.0.x). I'm not sure why GitHub insists to compare my branch with rel-4_0 by default?!


Why is this necessary?


The following code is not needed as the parent do not act on double click (besides being a copy paste error :) ) imo


いいね


現状のレイアウトはどのリソースも「layout.erb」です。
この記述は特定のファイルに特定のレイアウトを使いたい場合の命令文です。
study.htmlにstudy.erbというレイアウトを使用しなさいという意味


first commit read me


Fixed in #86.


I did at first but I don't like that period before the brackets. And I'm
too lazy to do it manually :p
On May 20, 2015 8:46 AM, "Underground78" <notifications@github.com> wrote:

> @XhmikosR <https://github.com/XhmikosR>: Could you change just the
> English string without reseting the translations?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/XhmikosR/mpc-hc/commit/d6226f54a07fd82d11b4e258021167daccc063da#commitcomment-11276734>
> .
>


ソースの先頭に「-」を付けると、その要素とその子要素が非表示になる


there is copyall(df.global.cursor) that is shorter way to do the same (also useful in other places where a copy is needed)


Define the function outside `render` (as another method), and then you won't need `that`. You'll just need to call it as `this.localize(...)` instead of just `localize(...)`.


Noticed that this is [`assertNotContains` in `master`] (https://github.com/aisapatino/sjfnw/blob/master/sjfnw/grants/tests/test_reading.py#L45).


We could ditch the Wiki article completely after this BTW.
On May 20, 2015 8:47 AM, "Underground78" <notifications@github.com> wrote:

> Ok, I guess a FAQ entry would make sense.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpc-hc/mpc-hc.org/commit/7abe06db97391349242e0c5025dfdfc407137b6f#commitcomment-11276738>
> .
>


0.12088 when used trend ^ 0.85. so far best model


if (err) is not needed since err doesn't get assigned again.


As you want, I guess it makes sense. The wiki article is somewhat easier to
edit thought.


Hi, my friend.
You can help me?
We have similar devices.
And I started receiving a mistake recently:

target thumb C++: libstagefright <= frameworks/av/media/libstagefright/WAVEWriter.cpp
In file included from frameworks/av/media/libstagefright/OMXCodec.cpp:92:0:
./frameworks/native/include/media/openmax/OMX_AsString.h: In function 'const char* asString(OMX_INDEXEXTTYPE, const char*)':
./frameworks/native/include/media/openmax/OMX_AsString.h:523:14: error: 'OMX_IndexConfigAutoFramerateConversion' was not declared in this scope
         case OMX_IndexConfigAutoFramerateConversion:    return "ConfigAutoFramerateConversion";

You can help me to solve?
My repository:
https://github.com/l9rebuild
https://github.com/l9rebuild2


changing a group's scope is not something that can be done without rechartering, so I think that piece needs to go too.


We shall name the macros in uppercase. --> -D__ALTERA_ARM__
Unfortunately I overlooked this for zynq. We need to clean it up also for zynq separately.


@bagder do you think anyone would misinterpret this and do `curl_easy_escape(curl, "field1=foo&field2=bar", 0);`. maybe it would be better to just say
> If you need to url-encode your data fields you can use \fIcurl_easy_escape(3)\fP.

?


thanks!


Well I could do that if you want.


probably because rel-4_0 is the default branch of otrs/OTRS.


Don't indent preprocessor expressions.


That's weird, how can you always be in that case?  I mean if it's a lucky coincidence then OK, but that shouldn't happen frequently, no?

In any case I applied your suggestion in this revised commit: tsuna/async@3552d39e7b6c8f27d152bc3109369fd9f045d3f9


- 虾米
- xiami


Thank you for your pull request.

A few notes:
* The added members do not follow the naming convention.
* Can't 'int editedTabIndex' be avoided using the current tab index?
* I added 2 line notes

The added behavior is interesting but I think that there are cases when one do not it.
Would it be possible to add an option for this? (for the three classes)


@m-g-sonar doesn't eclipse remove whitespaces in empty lines?


Given I went through the same BS at high school (I'm self-employed, no longer a student (my school life was annoyance after annoyance), I wish you all the best with the exams. Please don't stress yourself to the point that you drop this project, just take it easy. I'll most likely review and learn various bits and pieces, tinkering as I go in my own branch.

The redundancy was the same thing that made me throw in the towel on my own API clone. One guy that did 90% of the API stuff in KanColleViewer who was hostile towards me mentioning writing a DMM server clone (insert random reasons here) said the server does the battle and then replays it for the client to show what happened. If it comes up with a "ship damaged, what to do?" or night battle option, it usually does another request.

I have considered redoing KanColle's client in Unity3D using the WebPlayer build module, but the issue is that the WebPlayer is going out of fashion for WebGL. Unity3D can also deploy for desktop (Win/Mac/Linux) but I honestly don't have the time to work magic and remake a game. I did however make Mario fangames back in the day...

I really think if we are going to do a Kantai Collection themed fan(?)game, we could use what we've done now. ChinaColle exists, and it even has playable bosses (I've seen Hoppo-chan in an admiral fleet). That's just my thoughts. And if it's going to be a legal issue with DMM/Kadaowa Games then I'd rather put my energy into another KC-like game that isn't a complete copycat.

Good luck, @SunDwarf.


Such a test is really not the same... there are numbers and booleans too.


Too generic Exception!


@ctrengove as I said I've the need of this update so if I can do something, help with test and so on, please let me know. I will be glad to help you to make this process faster ;)


Too generic


Not python 3 compliant? Use `format()` please :)


Nice change


You can edit this page via the Web interface too :p
On May 20, 2015 9:31 AM, "Underground78" <notifications@github.com> wrote:

> As you want, I guess it makes sense. The wiki article is somewhat easier to
> edit thought.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpc-hc/mpc-hc.org/commit/7abe06db97391349242e0c5025dfdfc407137b6f#commitcomment-11277055>
> .
>


OK, I've merged in the current master and resolved all the merge conflicts. Next thing is to get the tests working (again), and re-do the migrations.


Thumbs up


Sure, along with the translation updates.
On May 20, 2015 9:31 AM, "Underground78" <notifications@github.com> wrote:

> Well I could do that if you want.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/XhmikosR/mpc-hc/commit/d6226f54a07fd82d11b4e258021167daccc063da#commitcomment-11277066>
> .
>


I wonder why it is named CFG_DEMO_BOARD_ZYNQ for Zynq but CFG_DEMO_BOARD_NAME for altera?


fddd


I'm not sure those two versions are going to make a notable difference. Leave it like that for now and we'll see how it runs with people IMO.


Pleas cleanup indentation in the commit message


import httplib


Should be Optional<User>


Surely the fact that the old `tidy` executable remains preinstalled on recent versions of Mac OS X would mean that the linter's command would remain `tidy`?


@mansa-dev

should be:

```
if (env1 == 'development')
```

Also, use descriptive variable names, so, `env1` could be `currentEnvironment`.

Also: why did you add this check for environment?


gg


Yes, that fooled me, sorry about that. I will update the default
branches to point to master as this is the one relevant for development.
Regards, mg

Am 20.05.15 um 08:32 schrieb Moritz Lenz:
> probably because rel-4_0 is the default branch of otrs/OTRS.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/muehlbauer/otrs/commit/a49eae2e3c4abebe14deab0729bdcd61be88eaa3#commitcomment-11277073>.
>

--
Martin Gruner
Senior Developer R&D

OTRS AG
Bahnhofplatz 1a
94315 Straubing

T: +49 (0)6172 681988 0
F: +49 (0)9421 56818 18
I:  www.otrs.com/

Geschäftssitz: Bad Homburg, Amtsgericht: Bad Homburg, HRB 10751,
USt-Nr.: DE256610065
Aufsichtsratsvorsitzender: Burchard Steinbild, Vorstand: André
Mindermann (Vorsitzender), Christopher Kuhn, Sabine Riedel

Grenzenlose Kommunikation mit der OTRS Business Solution™! Bereichern
Sie Ihren IT Support und Kundenservice mit smarten Business Features!
https://www.otrs.com/solutions/otrs-business-solution-on-premise/?lang=de/#OTRS-Business-Solution-Basic-Features


construct like this:
```this.user = Optional.of(user);```


@mansa-dev ah, I see you change the environment check in the next commit. no problem.


Sorry, my comment probably was too short. What i wanted to propose was, that you either hold bc breaking changes in the branch for the next major version (in this case ``2.0``) or add a notice to the installation section of the readme, that there will be breaking changes in ``1.*`` updates and that it is advised to use a specific version. This should avoid unexpected behavior. But that's merely a suggestion, i didn't want to bug you..


Please use correct indentation and braces around each else statement.


how to play???


hehe


We have a internal class that schedules a TimerTask to a HashedWheelTimer task which triggers the callback chain of this Deferred with a partial result at about around the same time [-5ms] as the join timeOut. That is the reason we hit this bug in the first place.


Change User to ```Optional<User>```


No space after a cast. Please check the whole code!


Make new parameter Optional


This file probably needs to be removed? It's showing instead of the readme.rst when you visit: https://github.com/flask-admin/flask-admin/


This will be
```
if (user.isPresent()) {
...
```


测试


OK :+1:


Change to Optional


And last one to Optional


If most calls directly print strings, you could also turn print/gPrint into non-format functions, and add printf/gPrintf for formatted printing.


Can we use user.id instead of user.email? Email could change in future ID won't.


another option would be to use util.DaemonThread


Missing `#include <stdio.h>` in global.c.


Implicit declaration of `printf`


Keeping the same from / to semantic will help people porting scripts from other messaging API to telegram-python.

Anyway, very nice progress, thanks!


@mcharleb Can we change the workflow now to do feature branches / PRs rebased against master instead of pushing to the linux branch, with everybody using master as the reference branch? I want to avoid that a rebase of the linux branch causes teamwork issues, but I also don't want to continue to produce a lot of merge commits on master.


Yes, it's awful and I don't even look at it. I suspect that it's something to do with being wrist-mounted instead of foot-mounted. I'd be interested to see what the difference would be wearing it around your ankle. It would make it harder to use of course, but might be worth an experiment.

To be honest, I'm not really interested in cadence - I'm only concerned with speed and distance. My wife is more interested in the cadence when swimming, but that's a lot more accurate anyway due to the swimming motion being much smoother and more regular than arm motion when running.

Dan, if you want to cater for pauses in the recorded data when calculating your average, you can. If you're interested, look at the ttbin/ttbin.h file and the STATUS_RECORD structure - the watch records when you start/stop/pause the activity, so you can use the "start" status for resetting your average calculation.

Ryan


Push a produccióoooooon duuud.


Awesome, I've now added a link to that on the http2 explained front page: http://daniel.haxx.se/http2/


I'm pretty sure the majority of integers are numbers...


optinalStats -> optionalStats


Sorry, I am not able to get you here!!


Who needs to make textures when you can steal them from other mods  :icecream:


Wait, why? Notepad++ maintains backward compatibility with VS2010? What?


Move bracket to next line because our code checker checks if opening brackets are located on the same line or in the same column as closing brackets.


@mgruner Hi Martin,
please take a look at this if you have some time.

Sorry for the name, I pushed this, but the name is wrong because I cloned Dusan's machine.


Move bracket to next line. See above.


worst comment ever


Yah i didn't touch that and so I just use the same Exception on the other case too. However, I can make the change to squeeze in this PR. Sounds okay?


@notspiff I wanted to PR this piece of code, but I don't fully understand it - m_fFPSOverride is only set via SetFps. So Whenever this is non zero - the GetCrtc call failed - does it also help, if you initialize fps = 0.0 here: https://github.com/xbmc/xbmc/blob/master/xbmc/windowing/X11/WinSystemX11.cpp#L1471

As the code further down does not set fps at all, when it think that there is no current mode: https://github.com/xbmc/xbmc/blob/master/xbmc/windowing/X11/XRandR.cpp#L499

This is the only way I think a non zero, but more or like random value could go into this.

@FernetMenta ping


Good point. Let me fix it :)


???  空的啊


getting derps

Copying: /home/joshndroid/temafloundersource/out/target/common/obj/APPS/Gallery2_intermediates/classes.jar
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:63: error: cannot find symbol
                Settings.System.HEADS_UP_BG_COLOR,
                               ^
  symbol:   variable HEADS_UP_BG_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:73: error: cannot find symbol
                Settings.System.HEADS_UP_TEXT_COLOR,
                               ^
  symbol:   variable HEADS_UP_TEXT_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:112: error: cannot find symbol
                Settings.System.HEADS_UP_BG_COLOR, intHex);
                               ^
  symbol:   variable HEADS_UP_BG_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:120: error: cannot find symbol
                Settings.System.HEADS_UP_TEXT_COLOR, intHex);
                               ^
  symbol:   variable HEADS_UP_TEXT_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:168: error: cannot find symbol
                    .setPositiveButton(R.string.dlg_reset_bliss,
                                               ^
  symbol:   variable dlg_reset_bliss
  location: class string
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:172: error: cannot find symbol
                                    Settings.System.HEADS_UP_BG_COLOR,
                                                   ^
  symbol:   variable HEADS_UP_BG_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:175: error: cannot find symbol
                                    Settings.System.HEADS_UP_TEXT_COLOR,
                                                   ^
  symbol:   variable HEADS_UP_TEXT_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:160: error: cannot find symbol
                                    Settings.System.HEADS_UP_BG_COLOR,
                                                   ^
  symbol:   variable HEADS_UP_BG_COLOR
  location: class System
packages/apps/Settings/src/com/android/settings/temasek/HeadsUpColors.java:163: error: cannot find symbol
                                    Settings.System.HEADS_UP_TEXT_COLOR,
                                                   ^
  symbol:   variable HEADS_UP_TEXT_COLOR
  location: class System
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
9 errors


I wonder, have you met using this package as non-extension?
If so, how did/would that look like?


In commit message: "Add support in stack, for Altera ARM processor"
Remove comma


@theory, can you show an example configuration?


@vied12 What was the point of replacing "get" by "find_one here"? You decided to use "/posts" endpoint for this purpose, instead of "/blogs/_id/posts"?


My take,

We have logic all over the place to figure out the host name to pick up for auto registration.
I'd like to suggest the agent resolving the hostname to post and not removing the extra hostname parameter and logic on AgentRegistrationController.

Can't we do this instead on AgentRegistrationPropertiesReader

```
    public String getAgentAutoRegisterHostname() {
        return agentAutoRegisterProperties.getProperty(AGENT_AUTO_REGISTER_HOSTNAME, SystemUtil.getLocalhostNameOrRandomNameIfNotFound());
    }
```

And on SSLInfractructure

```
           PostMethod postMethod = new PostMethod(serverUrl){{
                addParameter("hostname", agentAutoRegisterProperties.getAgentAutoRegisterHostName());

```


Is this intentionally duplicated to previous conditional registration on line 155?


Use the  openPOWERLINK data type macros for all datatypes! (e.g. UINT64)


No space after a cast. Please check the whole sources!


This is a test comment


jjk


een Object dat een random kleur aan neemt.
als het Object groter word dan " X:2 Y:2 Z:2 " dan word het object "GeDestroyed"


@RobertAudi: ha! Of course have `which` in four different places in npshell -- not at all a unified ;)

I'll push an update to use `type` and functions.


Note that originally isFirstIteration() would be true for the first iteration on each node, i.e. with five nodes there would have been five 'first' iterations.  But now ?  Maybe there will only be ONE 'first' iteration, on the first node.  The first iterations on each of the other nodes will NOT be considered as being the 'first' so the simulation behaviour may be changed.


Great, does it work also if name is empty?


No "else if"

```c
if ()
{
}
else
{
    if ()
    {
    }
    else
    {
        if ()
        {
        }
       else
        {
        }
    }
}
```


テスト GIT-1


Test [GIT-1]


You don't need this, everything in Cube 2 works off the virtual filesystem. Just use:
`listdir("demos", false, "dmo", files);`


@TysonKerr check the following files that I changed but wasn't too sure of how they work:
`trialLoader.php`, `trialTester.php`, `Data/index.php`, `generate.php`


would that mean that the script should look in all the repos in order to find each addon specified in the list?


Can we use "xclip -o" somehow to implement paste support as well?


clean!


Sorry I didn't comment earlier. I like what you did here, but I'm not going to take it vanilla. I've split it locally, and already pushed the part with the plural handling. I'll incorporate the rest with some modifications.


My question is if it's possible to change your original format to md,
rst, texinfo etc.

On 2015/5/20 14:51, Daniel Stenberg wrote:
>
> Awesome, I've now added a link to that on the http2 explained front
> page: http://daniel.haxx.se/http2/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bagder/http2-explained/commit/0826b8915992db651791d916c98800955b611416#commitcomment-11277259>.
>


Well, magit depends on emacs 24.4 AFAIK https://github.com/magit/magit/blob/next/magit-pkg.el


Rather than this mess, I suggest you open the file and read/verify the demo header. You can cache the results to speed up successive access (so you're not opening a gazillion files every time). This has the bonus of working with all demo files.


Thanks :+1:


root category has no parent and will be deleted too?


I think it would be difficult to suggest a way to install xclip that works on all distros. I'll put in some kind of error message if system() fails.


I removed `get` because it was already defined in the parent class, and I added `find_one` to allow to retrieve a post by `id` with the items included


yeah I forgot to add it. fixed.


ah, its for tests only


see issue #283


hardware/boards/altera-c5soc/mn-soc-shmem-gpio/include/dualprocshm-mem.h:85: error: closing curly bracket not in the same line or column


Is the changelog automatically generated? I think it should be `2.1.0` here, shouldn't it?


Evet


seems to still have the same problem


I will, because I think the topic needs proper discussion before implementing anything.
I was jumping in to solve the issue at first, but it proved to be more complex than I thought.


please install from `requirements/base.txt` and move this line to `base.txt`


Не вижу пока.


include `base.txt` in `test.txt`


alvin merged the OMGWTFBBQLINEOFCODE from his branch into the master


For Zynq, the variable CFG_DEMO_BOARD_ZYNQ is used to identify the board.
But as we already export variables CFG_DEMO_BOARD_NAME for all boards, I did not want to add another variable CFG_DEMO_BOARD_C5SOC again, to identify the board.


Please verify that you use the data type macros in all new and modified files.


Okay. Any reason for not indenting it, it is more readable this way.


Bonjour,

j'ai un petit souci, votre version ne trouve aucun nouveau epsode alors qu'il y e a de dispo pour exemple

2015-05-20 09:18:14 INFO     SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Performing episode search for Naruto Shippuuden
2015-05-20 09:18:14 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Checking if found episode 17x51 is wanted at quality Unknown
2015-05-20 09:18:14 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: any,best = [1, 2, 4, 16, 32, 64, 128, 256, 32768] [] and found 32768
2015-05-20 09:18:14 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status: 3 (Wanted)
2015-05-20 09:18:14 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status is wanted/unaired/skipped, getting found episode
2015-05-20 09:18:14 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Using ALL regexes
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Found episode by absolute_number:412 which is 17x51
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Parsed Naruto.Shippuden.412.VOSTFR.par.Fansub-Resistance.(1280*720).-.version.h265 into Naruto Shippuden - S17E51 [ABS: [412]] [ANIME VER: 1] [ABD: False] [ANIME: True] [whichReg: ['anime_bare']]
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Checking if found episode 17x51 is wanted at quality Unknown
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: any,best = [1, 2, 4, 16, 32, 64, 128, 256, 32768] [] and found 32768
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status: 3 (Wanted)
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status is wanted/unaired/skipped, getting found episode
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Using ALL regexes
2015-05-20 09:18:15 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Found episode by absolute_number:412 which is 17x51
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Parsed Naruto.Shippuden.412.VOSTFR.par.Fansub-Resistance.(848*480).-.version.MQ into Naruto Shippuden - S17E51 [ABS: [412]] [ANIME VER: 1] [ABD: False] [ANIME: True] [whichReg: ['anime_bare']]
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Checking if found episode 17x51 is wanted at quality Unknown
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: any,best = [1, 2, 4, 16, 32, 64, 128, 256, 32768] [] and found 32768
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status: 3 (Wanted)
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Existing episode status is wanted/unaired/skipped, getting found episode
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Using ALL regexes
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Found episode by absolute_number:412 which is 17x51
2015-05-20 09:18:16 DEBUG    SEARCHQUEUE-MANUAL-79824 :: [nyaa_rss_90j] :: Parsed Naruto.Shippuden.412.VOSTFR.par.Fansub-Resistance.(1280*720).-.version.MQ into Naruto Shippuden - S17E51 [ABS: [412]] [ANIME VER: 1] [ABD: False] [ANIME: True] [whichReg: ['anime_bare']]
2015-05-20 09:18:16 INFO     SEARCHQUEUE-MANUAL-79824 :: [Cpasbien] :: Performing episode search for Naruto Shippuuden
2015-05-20 09:18:25 DEBUG    SEARCHQUEUE-MANUAL-79824 :: Episode has no release name, replacing it with a generic one: %SN - %AB - %EN
2015-05-20 09:18:26 DEBUG    SEARCHQUEUE-MANUAL-79824 :: formatting pattern: %SN - %AB - %EN -> Naruto Shippuuden - 412 - Neji's Judgment
2015-05-20 09:18:26 DEBUG    SEARCHQUEUE-MANUAL-79824 :: Episode has no release name, replacing it with a generic one: %SN - %AB - %EN
2015-05-20 09:18:26 DEBUG    SEARCHQUEUE-MANUAL-79824 :: formatting pattern: %SN - %AB - %EN -> Naruto Shippuuden - 412 - Neji's Judgment
2015-05-20 09:18:26 INFO     SEARCHQUEUE-MANUAL-79824 :: Unable to find a download for: [Naruto Shippuuden - 412 - Neji's Judgment]

Une idèè.

meme probleme avec les serie, si je fais une recherche manuel il snached le fichier mais en automatique rien.


Sounds great! :)


contrib/dualprocshm/include/dualprocshm-c5soc.h:78: error: closing curly bracket not in the same line or column
contrib/dualprocshm/include/dualprocshm-c5socarm.h:182: error: closing curly bracket not in the same line or column
contrib/dualprocshm/include/dualprocshm-nios2.h:76: error: line should not begin with operator
contrib/dualprocshm/include/dualprocshm-target.h:191: error: closing curly bracket not in the same line or column


Maybe make another edit?


\o/ Chrome have just pushed out Stable 43, and the anims are now working!


What about incluting tailing `.0` to release in docs?


Fair enough


Need to make class variable


Currently we doesn't have options for removing from the settings panel start muted checkboxes. We also changed the type of startMuted config options to integer. If you set startMuted to N in the config.js, every participant after the (N -1)th participant will start muted.


Inherit from Form (from flash_wtf import Form), you do not need  RedirectForm because we do not use the form for redirection. Also see the other form classes


> Please, keep it in the development group of your Gemfile.

I think we can be more accurate if we say to _only_ keep it in the development group (I assume some people have bit in their `group :development, :test` and the like), plus an example:

> Please, keep it in only in development group of your Gemfile.
>
> group :development do
>   gem 'web-console'
> end


This will introduce more indented code or more goto/return statements.
Any reason for preferring this!


This..shouldn't exist.


Here indentation is 8 tabs, please use 4 tabs


correct
we currently use these repos
https://github.com/xbmc/xbmc/blob/master/addons/repository.xbmc.org/addon.xml#L11

it might change in the future if we have better logic on the server side.


Sorry for slow response.

Shouldn't we also need a check that "target" parameter doesn't contain a host name at line 104? Otherwise malicious user can still put a full URL to the "target" param.


Why not use: 'for course in courses' ?


I'm pretty sure this is wrong, the Log:: functions will automatically add a \n after every message


This file seems copied from see_courses :).


check if nodeCopy worked


Joe,

There will be only one entry for each contribution ID in the temp table since there is a group by being done on the contribution IDs. After that, we are performing an inner join with the temporary table to eliminate those contribution IDs with unpermissioned FTs.


This ending tag corresponds with what div? :D


This shouldn't be needed, all C++ standard library headers are already included by Common.h which should be included everywhere (it is included by q_shared.h, which is included everywhere)


Indentation seems 8 here.


This function feels somewhat useless, why not just throw the exception directly instead of calling this?


@sagarjadhav Thanks :)


It would be better to just not set `RedirectStandardOutput = true` here.


Does loadSets() need the same treatment here?


Old compiler requred the # at the first line. Now, it is possible to indent and it's just a matter of style.
I prefer it on the first line to emphasize that it's a preprocessor statement. This is the used style in
(nearly) all files and we should keep a single style in the whole sources.


Yes, you're right.


done


Hi @sschaef, this change broke the ide-validation for PRs to scala/scala, we get this:

```
++ /home/jenkins/workspace/scala-2.11.x-integrate-ide/uber-build//uber-build.sh /home/jenkins/workspace/scala-2.11.x-integrate-ide/uber-build//config/validator.conf e4c0ae417817413eb5a8a445f77c827f7817f682 2.11.7-e4c0ae4-SNAPSHOT
/home/jenkins/workspace/scala-2.11.x-integrate-ide/uber-build//uber-build.sh: line 44: ./uber-build-config.sh: No such file or directory
Build step 'Execute shell' marked build as failure
```

I guess you'd have to do something like
```
uberBuildDir=$( cd "$( dirname "$0" )" && pwd )
source $uberBuildDir/uber-build-config.sh
```

https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-ide/933/console


i don't have working xrandr on this box due to its setup (2 quattros, each driving 2 screens, plus xinerama). thus the call to g_xrandr in that function doesn't write the fps (hz) value properly and yup that is the cause. just initializing that bugger in UpdateCrtc() takes care of the problem.


Right now it is expected to exactly match the version in CHANGES.md. Why?


Yes, there is no "else if" statement in C. If you use it in this way you hide that the code is more complex as it seems. You hide that each if statement is nested. So either the code should look as complicated as it is or it should be simplified. (E.g. return or break in the if clause and don't use an else).


The test file is not considered as java sources (located in src/test/files), so it does not apply java formatting rules. I missed that one.


cool


Why have you used the class here and not `atom-workspace`?


I'm refering to Sphinx docs and comment in config: `the full version, including alpha/beta/rc tags`, so if we rerelease X.Y with some fixes and bump patch version it looks reasonable to update it here. But it is just a question, not a valuable suggestion, so just skip it :-)


I would highly suggest making everything that is a link into an actual link. Using JS for links means that certain features are broken (right-clicking, middle-clicking, tabbing through links, etc).

Is there a particular reason to use JS in this case?


I am not sure if this is what I should be using branches for, but it seemed better than forcing people to copy paste code or publishing a commit that I know does not work properly.


There is pkgconfig usage in this format: `PKG_CHECK_MODULES(VAAPI libva libva-drm)`. LIBNAME can be different from PKGCONFIGNAME, and one LIBNAME can reference multiple PKGCONFIGNAMEs.


Thanks!


WebGL/Unity isn't the way to go - I don't know unity or anything required for it. HTML5 + something like Phaser or PlayCanvas or EaseJS (quick google search) is a much easier method and is supported on pretty much all browsers. Plus I have a WebStorm pro licence, so I could use that for JS dev.

Also, you can't copyright an idea, only assets and art.


Does this line print duplicate confirmation after specific FindXXX.cmake and pkg_check_modules() printing their confirmation?


I forward this issues to karl who asking to allow multiple value to
motor_vehicle.

Le 20/05/2015 00:33, Nelson A. de Oliveira a écrit :
> Isn't it wrong to have a multi-valued |motor_vehicle| key?
> Per http://wiki.openstreetmap.org/wiki/Key:access |motor_vehicle| means
> any motorized vehicle.
>
> Taking a look at
> http://taginfo.openstreetmap.org/keys/motor_vehicle#values I see some
> |motor_vehicle=agricultural;forestry| for example, but from what I see,
> they actually should be split into |agricultural=yes| +
> |motor_vehicle=forestry| (with possibles |some_other_vehicles=no|)
>
> The same for |motor_vehicle=agricultural;destination| →
> |motor_vehicle=destination| + |agricultural=yes| (with possibles
> |some_other_vehicles=no| too)
>
> Or I am misunderstanding something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osm-fr/osmose-backend/commit/d784b47f542db49fa7e36d22c4c6a20fa1a054b5#commitcomment-11272880>.
>


This is an indication of programming error. Should not appear in production.


ok, we can live with this for the moment, thanks.


![Uploading ResearchAndDevelopment.png…]()


Does the job :) Another way of doing it, using the [`substring` built-in function](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/substring):

``` js
return bigString.substring(startA, endA) + bigString.substring(startB, endB);
```


Can you use the actual library name "TurboJPEG"?


精简写法


In the same spirit as above:

``` js
  return text.indexOf(searchString);
```

I can see you are pretty much an algo guy :)


Additionally, it's very error prone. If you sometimes need to add additional code in such a construct!


@myabc I think all plugin specs that need a factory. Its is something like ~1000 specs, depending on the configuration.

Example spec: https://github.com/finnlabs/openproject-costs/blob/release/4.1/spec/requests/api/cost_entries/cost_entry_resource_spec.rb


``` js
return text.lastIndexOf(searchString);
```


just a curiosity... why this function? Shoudn't uninstall-package already remove the node from packages.config? I suspect no :smirk: but why? :smile:


Just one way of doing it:

``` js
return text.substring(text.indexOf(searchString) + searchString.length, text.lastIndexOf(searchString));
```


I do like the reuse of `findFirstMatch` and `findLastMatch` though!


adding "var" causes my linter to complain because it's defined above; if I'm explicitly resetting it to 0 to start the loop, I don't think "var" should be necessary.


fgfg


fdgdfgdfgdfgdf


Note that the `===` operator being a *boolean* operator, it returns `true` or `false`, which is what you want to return from your function. You can therefore shorten the implementation like this:

``` js
function(stringOne, stringTwo) {
  return stringOne === stringTwo;
}
```


I'd rather use the `!==` operator, so the return value is a direct match for the test:

``` js
function(one, two) {
return one !== two;
}
```


It should, let say that my trust in NuGet is very low...to say :-)


I dont understand how work's mp4 progressive download.
In nginx official mp4 module works "?start=2268.099". But nginx-vod-module it doesn't :(


No space after *


Inicio de proyecto.


In the same spirit:

``` js
return (lower < middle) && (middle < upper);
```


Hm, I guess so, I didn't add iten sets editing yet so didn't notice this...


That might not be completely true -- pipes might be used even during
decoding (for example as audio sources).  Also, pipe functionality is an
official part of the Kaldi I/O, so we should strive for having it working
under windows as well
y.

On Tue, May 19, 2015 at 11:22 PM, Kirill Katsnelson <
notifications@github.com> wrote:

> Good point, I will add a test to see if running under bash. However, the
> point becomes rather moot in this case, since pipes are not going to be
> even used in native Windows library decoder.
>
> The path tinkering is harmless. Windows paths are transparently supported.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/5d2cc33b614294c36aa6e7e0e5619d69e9ca7aaf#commitcomment-11271654>
> .
>


Yep. It has been removed. I was testing things. Still puzzled why combining the two files caused errors when they work separately just fine.


Don't forget to write documentation for that generator.


Dont get this, ARGN is used for REQUIRED, (see usage of auto_detect_lib below)
and LIBNAME is only a variable, it CAN be diffrent but dos not have to be.
So problem?


Lets rather store into ``wait`` when selecting ``--nowait``. Otherwise our brains get lost with double negatives.


Looks good. Another way of doing it:

``` js
function(number) {
  return (!((number >= 10 && number <= 20) || (number > 42 && number <= 75) || (number > 1 && number < 6)));
}
```

Saves a local variable (`inRange`) and some `if/else` statements, but the readability may not be as good as with your version.


Might yes


Add spaces around operators (not unary operator)


Any reason for the quotes around the keys here?


Same story:

``` js
function(name, price) {
  return ((name == 'NYTimes' || name == 'LATimes') && price >=1);
}


Is there a closing tag of this?


libusb-1.0.pc has this:
```
includedir=${prefix}/include
Cflags: -I${includedir}/libusb-1.0
```

It should work fine with LibUSB_INCLUDE_DIR/libusb-1.0, or add a libusb-1.0 suffix to libusb `#include`, or just use LibUSB_CFLAGS.


It's only a status information. I don't care personally here. I prefer more output instead of less


OK. I didn't see that you had predeclared it. It's a small thing, but could you also cache the length in the loop?

`for (idx = 0, len = outers.length; idx < len; idx++)`


Complete #CRvj


I fixed it in a later commit.


Why not just ``while is_any_busy``


testgit 2


sorry went to bed didn't see you post.

i manually changed my settings but it looked like this ...

 postgrey startup options, created for Debian

# you may want to set
#   --delay=N   how long to greylist, seconds (default: 300)
--delay=180"
#   --max-age=N delete old entries after N days (default: 35)
# see also the postgrey(8) manpage

#POSTGREY_OPTS="--inet=10023"
POSTGREY_OPTS="--inet=127.0.0.1:10023
#POSTGREY_OPTS="--inet=127.0.0.1:10023"

# the --greylist-text commandline argument can not be easily passed through
# POSTGREY_OPTS when it contains spaces.  So, insert your text here:
#POSTGREY_TEXT="Your customized rejection message here"


:+1: LGTM


Yes, that was always the intention; having it inline while prototyping was convenient.
I'm also considering simplifying by making a convenience class that inherits QUndoCommand and keeps an undohelper available and simply calls undoChanges for the QUndoCommand::undo() virtual.


Nested `if` is bad design. You can dramatically simplify your implementation using boolean operators:

``` js
function(first, second, third, fourth) {
   if ((first == 3 || first == 5 || first == 7) &&
      (second == 2) &&
      (third >= 5 && third <= 100) &&
      (fourth < 9 || fourth > 20)) {
        return "correct";
    } else {
        return "incorrect";
    }
}
```


The file is called FindTJPEG it does not provice a package config. So in the end it is variable here


Following the same idea that boolean operators return `true` or `false`, which is what you want to do with your method, you can leverage them to simplify the implementation:

``` js
function(item, money) {
    // return 'true' if a user can afford a given item according to the price chart below, false otherwise:
    // 'MacBook Air' - $999
    // 'MacBook Pro' - $1299
    // 'Mac Pro' - $2499
    // 'Apple Sticker' - $1
    // return 'false' if the 'item' is not in the above list apple products

    // Jean-Denis, let me know if you want this coded with if-then-else. Below is simpler, though.
    return (item == "MacBook Air" && money >= 999) ||
      (item == "MacBook Pro" && money >=1299) ||
      (item == "Mac Pro" && money >=2499) ||
      (item == "Apple Sticker" && money >= 1);
}
```


Inconsistency between TurboJPEG and TJPEG.

Also, perhaps move `${MY_DIR}/../../depends/libjpeg_turbo` as default paths to FindXXX.cmake?


说明
系统基于 ThinkPHP 和 ExtJS 开发。

Git 基础
可以下载 GitHub 客户端，这里重点提一下命令行开发

获取仓库代码

安装 Git，详情自己找到案
git clone https://github.com/fantengfei/12366.git
提交版本

git status
git add .
git commit -m "提交说明"
git pull --rebase
git push
以上步骤不能缺少

开发说明
根据地址 www.12366.ha.cn/Application/Home/Common/html/tree.html 打开 tree.html;
找到变量 treepanel ，在其中添加树节点，结构如:

{
text: "增值纳税申报表附列资料（一）",
leaf: true,
url: "__APP__/Home/TableNode1/table2"
};
依据 thinkPHP的 MVC 思想进行页面编写


Remove this empty line.


Why are you defaulting to 100 if the backlog size is 0? a) We are not doing this anywhere else, why is this case different? b) This way it is not possible to define an alarm callback that is triggering an alert without a backlog.


Nope. Just ignorance on my part.


where to add google api key ??


Following the same idea that boolean operators return `true` or `false`, which is what you want to do with your method, you can leverage them to simplify the implementation:

``` js
function(item, money) {
  return (item == "MacBook Air" && money >= 999) ||
    (item == "MacBook Pro" && money >=1299) ||
    (item == "Mac Pro" && money >=2499) ||
    (item == "Apple Sticker" && money >= 1);
}
```


@elemoine i refer to them in the template, so I do believe we need to quote them for the compiler , correct me if I am wrong


1: M100 needs more reviewers
1: M101 signature is missing
2: M102 unrecognized bullet/signature
1: M110 missing component name
1: M191 must not end with a dot '.'


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: Merge branch 'cdslabs' of https
1: M190 line is too long (80 > 50)


3: L102 copyright year is outdated, expected 2015 but got 2014


Note that it doesn't use any `if`/`else` statement. Which is good :)


Исках само да попитам защо качвате Microsoft.Practices.EnterpriseLibrary.Common.dll и Microsoft.Practices.EnterpriseLibrary.Validation.dll като те могат да се инсталират с nuget?


The downloader uses a priority queue implementation I did a while back based on heaps and a bit optimized block based storage opposed to a simple backing slice. http://godoc.org/gopkg.in/karalabe/cookiejar.v2/collections/prque

I'm not really suggesting we use it, just thought I'd note it, that if you'd like, make you could take a look at it. For the above code, it does miss two methods: `peek` and `contains`. The first I could add without any problems, the second I'd rather not and use a set beside the prque. But maybe it's not worth the bother.


I do not like this change as implemented because it is cluster-wide... With many chunkservers it can put a lot more stress to the network. I'd like to modify topology syntax somehow to be able to isolate such change per host or group of hosts. Alternatively it could be implemented as chunkserver config option (something like "do not respond to local queries") but either of those changes would require much more work...


Not that I understand why that happens, but as soon as I remove the `:require => false` it works again.

The point I don't understand is that manual requiring does not seem to work, even if I put `require 'factory_girl_rails'` directly into `cost_entry_resource_spec.rb` it will not work.


lol, trebuia ea sa le urce si am facut pe laptopul meu cu contul ei si tot de la mn s-a pus:))


On my system it is not provided
adding this to ```#include``` enforces a version which is not good, and b) assuming it is present on a public path


You're right.


Yes it would, I think the document has stabilized enough so that we can do that. But someone would need to do the conversion job...


it surely is copy-paste error, will remove it :)


Hello, we would like to ask you why you have removed lightmap from basic and lambert?
I'm asking because we want add light via baking (also for existing scenes) and now it's not supported anymore for this type of mesh.


This could be rewritten as:

```js
    if (service in this.services_) {
      var googUri = new goog.Uri(this.services_[service].url);
      // …
    }
```

Isn't is more elegant than using a local variable than may be either a string or a boolean?


@teryk there is something wrong here: it's a dependency action, it should not use UsersWsAction marker interface.


GRILL OUT!


UndoHelper has a lot of optimization potential; once finished in recordAfterState() it could for example throw away all the stored state it doesn't need based on the bit flags in Info::changes. Another candidate is scoping down the amount of clips it xml-serializes and analyzes, improving on both memory and performance.


We probably don't want to use a generic keyword like `rule` for specifying the semantics. Can we just say one of the three we support next to the rule? For example:

```
Q(x) :- ...
weight = f
label = l
ratio
```
Or perhaps `semantics = ratio` may be good enough.

Also we should have these inference rule annotation syntax optional.


I am not sure about current tab index - it may be broken if in some case the tab is switched before the code reaches the `finishRename`  method. It's a wild guess, i just used the solution provided on SO answer.


no, kill it ;)


Should take core of it, thx for reporting: https://github.com/xbmc/xbmc/pull/7165


Ok, I've just installed puppet and run some `puppet resource` commands, looks very usefull to have as a "testinfra fixture" (and facter too).


that was required for me to build your code, but i am not really sure about it - that might've been the borked Qt installation or something. Will try on another machine.


I have renamed it and can commit the changes when we have changed all that is necessary.


#WelcomeToAxsl This Is Wher You Start I Guess :)
HHaha It's Where I'm Start'in Anyway :)


Doesn't radosgw on a non-mon node still need ceph::default and ceph::conf?

Thinking about it again, would it be enough to just do `include_recipe "ceph::keyring" unless node.roles.include?("ceph-mon")`, and not touch anything else? (Note: I haven't tried this myself)


So, is this funcionality not needed for SDL2? Or has it been disabled till it's implemented with the SDL2 semantics?


Just a tiny bit! ;-) Removed the line in 57719c745cd76fc578b47920733ce7c97c17c71d. Those who want the dev version can just install from github. Thanks!


I would like to have Py2 support out of the box, so that would be great! :+1:


Exception catch can be moved to line 40 with an additional return added after the fail-flash.


Delete this section


bad example; delete it


it's bad because you don't need input redirection to accomplish the task. `cat inputfile` would work just fine


again, bad example


Woopsie. I might have forgotten to push some stuff on my home computer. Accepting the pull request though.


The first condition also needs this.


`xargs` is a very useful command, but you're going to need a rather large explanation about how it works


Fix consistency with whitespace here?


The `-` to represent stdin is a common unix idiom.  Explain this explicitly, and try to find a few other examples using it.


I like that your examples are building off the previous examples.  That's great!


@lrytz I corrected it here: https://github.com/scala-ide/uber-build/commit/5dc073cc554efc9e739177b9c93f6715e350b4c5


done!


Which URL leads you to believe that?


Perhaps make this output a feature enable/disable list? Like:

```
TurboJPEG: yes
OpenGL: no
OpenCL: yes
```


I'd prefer your examples not directly answer the quiz questions like this.  Otherwise it's a great example.  Try to find something similar, but not exactly this.


(for token) ..................
有辦法改commit message 嘛XD


Locally built GLFW on all platforms provides both CMake and pkgconfig files. Linux distro provided libglfw-dev includes pkgconfig files. This file is not needed?


LOL :D

I though I commited the frameworks/base commit but I think it was in my portable hdd!!!


If this file is needed, please resolve this TODO.


it would probably be good to have several `xargs` examples


Are there any differents to the merge request #49?


Looks good, thanks! https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-ide/940/console


ги


This is a great start.  You're on the right track.  I like that your examples start simple and get more complex at the end.  I'd like to see a few more complex examples in the final product.  I'd aim for getting ten of them, each with a 1-2 paragraph description of how it works.  The examples you have right now are really good because they accomplish real tasks that people might actually care about.  We want all the examples to be that way.

Any questions?


I think this construct is not needed .


Changed in 07461a0.


@jfindlay Why so? (Just curious). At the moment you've hard-coded to only two modules, but what shall I do if I drop another one later? My code is dynamic, your is static. Is this is OK?


I don't think Singleton pattern in itself is a bad idea, though it can be abused. I prefer to limit the use of them. An alternative in this particular use case is to create a FileHelper Factory class that handles the creation of FileHelper.   Within the factory class, it can handle how many instances are permitted to be created. Then you argue that this is just another form of singleton with added abstraction....though I prefer this approach over the former.

I respectfully disagree in the case of this class violating the single responsibility principle. It is only responsible for a single task, returning a single instance of the fileHelper class. That could change in the future if we introduce methods to handle file operations.

Although I agree testing singletons are difficult, the current tests  only test that a single instance is running.


What is the purpose of this method? Is it there because certain plugin requires it but it is now deprecated? What does the FIXME stands for? What should be done with the method?


looks good and "scalable" => we can add as many widgets as we want with the filter.
ready to merge!


You probably meant "return *all* the records" instead of "return *are* the records"


Why do these random integers suddenly appear here?  Use a descriptive boolean flag please, e.g., `tmpStmtIsForNewTable`.


Good stuff, but this one kinda feels like a sore thumb to me… namespace wise.


First check causes it run setup every launch due to it always being true


If for some reason the instantiation dies early in ``__init__`` before the attribute is set up and the ``__del__`` method is called automagically to clean up  (e.g. when the exception is caught because you can catch a missing input file in this way) then it avoids an `AttributeError`.

Of course, it might be clearer to start with

```python
self.dcdfile = None
self.dcdfile = open(...)
```

or whatever there ought to be.

Also, it's obviously nicer to write `if self.dcdfile is not None` instead of the code-equivalent `if not self.dcdfile is None`.


Why this was reverted? What was the problem? Thanks.


Please refer to line 153-156. We have three different cases here. @netj


Even if it's Altera code I prefer to clean it up:
contrib/trace/trace-c5socarm.c:86: error: keyword 'for' not followed by a single space
contrib/trace/trace-c5socarm.c:89: error: keyword 'while' not followed by a single space


As far as I can tell, from is a keyword I just can't use, which is why I changed it:
```
surye@atlantis ~/Projects/tg (python-bindings) $ python
Python 3.4.0 (default, Apr 11 2014, 13:05:11)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> class tmp():
...     from = 3
  File "<stdin>", line 2
    from = 3
         ^
SyntaxError: invalid syntax
>>> class tmp():
...     src = 3
...
>>>
```


Where do `lib/bcc` and `lib/gcc` come from?


What is the reason for changing the method name? Such changes in the API should be always done via pull requests and should wait for approval. Thanks


What's the point of computing the checksum in a separate thread when the main thread is just waiting for the worker thread to finish?


Comment 1


minor: 2-space indent


Impossible to read.  This is not a hardware circuit (`resolveColumnFlag == 1`?) or machine generated code.  You'll want to spend more lines here making it readable.


I see your derp, and raise you… d320a2eeb663ffeb3416301e35ec1b5eb9cb594a :grin:


This isn't a very good example because it's not very well motivated.  Typing `dd` really isn't that much of a burden.  Something like commenting a line, however, which involves a dozen or so key strokes is.


Should be unneeded with latest commit e8eba3f19d1b1ded883ff843217a8bc90063aea9  (didn't see your pull earlier)


yep, that should be the aim of that filter, just a stub for now :)


Typo `}`


Again, try to pick examples that people might actually want to add to their vimrc files.  In this case, you're actually overwriting a builtin vim command (I use `a` in visual mode all the time), so I would find this particularly annoying.


Wrong indentation


The two conditions have the same content.


This is a much better example.  Just delete the first two and start here.


@rmichaelis Login, Password and Submit buttons are not translated.


I think this config value only controls whether we put a checksum on uploads? Please be more explicit if setting it to "" also means we ignore checksums for downloaded files.


Don't include all the text from your previous examples; only include the text that's relevant to whatever you're talking about.


Last chance to think of a better name than USRVALID ("user" == "not internal")


I used generalized-Streett in my diploma thesis, but I never implemented it and never published it elsewhere. So I'm OK with this.


Gemfile、およびGemfile.lockの変更はこのコミットメッセージとは無関係です。
作業単位でコミットを分けるようにしましょう。


Use ADD_DEFINITONS only for definitions (-D...)
Use "SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} .....")" instead.


I'd be tempted to remove the association, as it implies this will go in Map objects at some point.


This is again a bad example because no one would want this in practice.  Much more useful is to delete all the whitespace at the end of lines when saving (I have this in my vimrc for example).  Another good example is if you're opening a C++ file, automatically adjust the indentation to your preferred style (e.g. 4 spaces vs 2 spaces vs tabs).  I'd include both of these in your writeup.


This is a good example.


Attention au git add my_file !


Оценка "хорошо-"
Прокомментировать, увы, не могу, т.к. проверяла работу Татьяна


you'll need to explain a bit about `strftime`


:+1:

わかりやすくて良いです！


@lots0logs : See issue #347


wait~~~周末给你搞不行么。。


Where's the case for `None`?  Don't wrap with `Option` type if you'll always have some value.

These number assignment also makes it impossible to read for normal humans.  Either put a name around those integers and only use them from everywhere, or define and use an [`Enumeration`](http://stackoverflow.com/a/1321754/390044) type.


The method was introduced and then changed in the same pull request, which was approved; Name was changed because at first its name was based on getPipelineOwner() method. But this name was wrong because actually, implementation of the getPipelineOwner() returned (and should have returned) owner of the pipeline execution.


It was more that there are a bunch of other strings which you probably don't want to be modified (e.g., uriString, uriBaseString) that were not final; I was wondering whether there was something I'd missed. Given our general code doesn't mark things within local method scope as final, I'd suggest removing the qualifier.


What happened here https://github.com/UnifiedViews/Plugin-DevEnv/commit/d69926435aaf3daf383b7cacac21837865600a8c
should had happened also here! If one chage dependency in minor version, one should update minor of itself.


Har du tilføjet en knap til Unit tests? - Er det ikke lidt mærkeligt? - Test burde ikke have noget med GUI'en at gøre.

Hvorfor er det vi ikke bare bruger den indbyggede testrunner eller NUnit?


Use ENDIF()


Maybe it makes sense to move this recurring pattern of if-set-access to a utility function?


@NobodysNightmare this doesn't make much sense to me either. Having two `spec_helper` in the `$LOAD_PATH` doesn't seem to make much sense to me either though:
* https://github.com/finnlabs/openproject-costs/blob/release/4.1/spec/requests/api/cost_entries/cost_entry_resource_spec.rb#L29
* https://github.com/finnlabs/openproject-costs/blob/release/4.1/spec/spec_helper.rb


Sorry, had to do some rabase juggling to remove incorrect email address from commit log


apps/common/src/lcd/lcdl-c5socarm.c:200: error: semicolon is isolated from other tokens


Well written


Actually I am not sure what was the problem, there was an exception preventing even displaying of pipelines table. I did not have time for investigating at the time so I reverted the commit, but I will return to that when I have time.


Ok ```tools/editconf.py``` didn't worked as I expected. @JoshData do you agree ?


I propose to move the descriptive enum to the header just in front of `read()`.


You should move the if statement examples before the previous section.  That way you can do an `if filetype is cpp` kind of thing like I suggest above.


Use braces and correct indentation.


Is this class deleted? I'm not seeing it in the proj file?


One of my PR needs to use something like `auto_detect_lib(VAAPI libva libva-drm)`, or maybe more components. Do not want to duplicate variables of each component.


@bmwiedemann feel free to do that. But to be honest, I don't think it matters much :-)


Since local libusb copy is required now, you can find it at `depends/libusb/lib/pkgconfig/libusb-1.0.pc`.


Can't locate Pod/Tree/Stream.pm ...
Is the dist compete?


It doesn't for me


That would be a premature abstraction.


В данном исполнении работа претендует на оценку "плохо"  в смысле "очень плохо".
1) нет связи между таблицами. вам нужно использовать FOREIGN KEY, как это указано в пунктах основного задания
2) для ID лучше использовать численные а не строковые типы с настроенным генератором чисел, например AUTOINCREMENT


vanligtvis skulle jag dela upp denna i mindre commits. Men det var ju trots allt en omtenta, så ville inte slösa tid.


Implementation will come with merging pull request https://github.com/UnifiedViews/Core/pull/382 into develop;
Now, we can provide only dummy implementation as actor concept is not in develop yet.
This was discussed with @skrchnavy as we needed to merge Plugin-DevEnv into develop as further changes in Plugin-DevEnv were needed


@julienlancelot FYI, I think you forgot to remove UsersWsAction dependency. Fix already pushed by @jblievremont


Overall it's a very nice start.  You're definitely heading in the right direction.  Things you're missing:

1. You should mention comments near the beginning

2. You should provide the full vimrc file that you create as a standalone file in the repo

3. I'd like to see at least two separate examples for each of the sections you have

4. It would be really nice to see an example of a function or a loop.  The tricky part about this is finding an example that is both simple and well motivated.

Any questions?


no prob, nothing lost.

  _____

From: SchizoDuckie [mailto:notifications@github.com]
Sent: Wednesday, 20 May 2015 20:34
To: SchizoDuckie/DuckieTV
Subject: Re: [DuckieTV] optional_permissions urls required for cookies
(bebe17a)

Sorry, had to do some rabase juggling to remove incorrect email address from
commit log

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/bebe17aa3ab9149d749449379f9
66872f200e1ee#commitcomment-11278698> it on GitHub.
<https://github.com/notifications/beacon/AGnK-AfOezQjVgcckOArsb4sjp7dX_DKks5
oLD56gaJpZM4Ehf0R.gif>

---
This email has been checked for viruses by Avast antivirus software.
http://www.avast.com


apps/common/src/system/system-c5socarm.c:147: error: keyword 'if' not followed by a single space
apps/common/src/system/system-c5socarm.c:359: error: keyword 'while' not followed by a single space
apps/common/src/system/system-c5socarm.c:371: error: keyword 'while' not followed by a single space
apps/common/src/system/system-c5socarm.c:380: error: keyword 'while' not followed by a single space


Probably want to warn about an unknown transChecksum value somewhere. Once during startup?


Hi I already change to char* but still getting error. Using Ubuntu 14.04 http://s28.postimg.org/3uhvfd2e5/Screen_shot_2015_05_20_at_4_08_22_PM.png


camelcase


I think I would be really nice to reintroduce this in a  customizable fashion, because this is not at all consistent: e.g. Nature and PNAS want "ref. 1", while others just want the number. Even if one needs just the number, sometimes it has to be in brackets and sometimes not. This missing behaviour could be a major blocker for some people in academia to switch from Word to markdown/pandoc!

Maybe it makes sense to propose an update to the CSL to support this? (Not sure how one would do that…)

see also #125

Hope that helps!
Thanks!
Bela


`any` is implicit


yes i do .. where to post it ?

On 5/20/15 8:02, fxlb wrote:
> Do you have a sample pcap file with this option and
> 1) only |Purge Originator System-ID|
> 2) |Purge Originator System-ID| and |Received from System-ID|?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/the-tcpdump-group/tcpdump/commit/be03cc0b4ae9370ea0483d1504f4a950b22625d3#commitcomment-11276839>.
>


Can the version of Karaf we are using be updated? If not we should create a Jira to address this discrepancy in the future and also add a comment to this commit at this line to mention why we are overriding the version of arquillian.

Also, does this need a <scope>test</scope>?


No space between type and *


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/15169/testReport/) for this commit**


This should be removed...


indentation is off. tabs?


Please coordinate with @markasfour to make sure you're both not duplicating material.  See #1583


レイアウトファイル「study.erb」がstudy[文字列].htmlでも適用されるようにした


Performed a review


I'll modify the download logic to take that into account tks


should be lambda instead of function


Shouldn't we also tell the server which checksumming algorithm was used?


@gazpachoking Ugh, I didn't really read the functions carefully. What would be the proper way of avoiding a domain delay on the get_token function?

Searching on RarBG is limited to 1 request per 10 seconds, but the token getter is not limited.


Nice change


this is not gardening, but nevermind.


The header contains the type:value. So the server knows on per-request basis which type of checksum was sent.


I assume this is fixed in future commit


Typo in the message ("filei")


Can we use a different file for credentials? I already have some other credentials there, for maven central.


> the problem is the PHP and how it handle \ in strings.

I've updated comments in tests to make it clear what should be in the template regardless of PHP slashes mess.

I've only missed `\\\{{foo}}` use case, but I'll try to fix it soon.


It looks a bit like this switch on the checksum type string is shared between upload/download and could be a function.


probably not. I've probably forgotten to update the MANIFEST.


The examples in the sessions below are great!  But it's going to be very hard (and boring) for someone to follow along without seeing any pictures.  The best way to solve this problem would be to incorporate animated gifs into the writeup that show screenshots of exactly what each of these examples looks like.

Lookup the ImageMagic suite of utilities for an easy way to create these animated gifs from the command line.


I was actually refering to the document: there is no definition of `generalized-Streett` in this document, and the example above was actually about `Streett`, so I think this use was a simple typo.


Why do we need `tmpStmt`? (Keeping its misleading name aside: it's actually not a `Statement` type).  if we're going to keep use the first of bodies (`tmpStmt.bodies(0)`) and use the same head for every body, why not just use `cqBody` and `stmt.q.head`?  Sometimes naming things is a must, but in this case there are too many meaningless names that feels like it's trying to make an average person give up reading the code.


commenting out code is considered as bad readability, usually the rule is that code that is not excecuted should not be commited. we are using version control so just delete.


Indeed the GLFW provides CMake config files, but they didn't work for me, so I wrote this one. I'll try take a look at it again to see if I can get it working or perhaps make a pull request to GLFW.


A malformed checksum header seems like a cricial error...


Thanks @jblievremont @teryk. It make me think that this WS need to be removed.


use .data('api-key') to get data attributes


Does not make sense at all, true. However, the Validator class moves the worker into a thread properly, so this classes were just a left over. 1f7274c2f270c401290238173ee2fd5d06038adf cleans that up. Thanks for spotting.


In particular, there is a lot in that pull request about how to do key remapping.  Most of my comments apply to both pull requests.


Damn, I alway fall in the keyword.kwlist trap!


should be private and use proper variable names.


Use ENDIF(). Please check all locations.


what's this?


I think you definitely want to log it properly but this should be recovered "transparently" to the user by retrying.


size e.g. 3,3 should not be hard coded like this. Create another variable to store the size.


0 changed files -> because I deleted the conflicting code lines and kept the original master userlist.txt ..


Fixed in ddfe3fa7ab028cae4ebf976314c791c7e50b1156


Please use SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ...")


I think this could be more efficient/shorter.


Instead of using generic names like `file1`, it would be better to tie the examples more directly with the homework assignments and labs.  Maybe you're opening up `Makefile`, `README.md`, and `rshell.cpp` for example.


This function is not required at all.


1) реализация метода DELETE
удаляется последнее сообщение независимо от того, какому пользователю оно принадлежит (а нужно было передавать ID сообщения, которое нужно удалить + проверять, что удаляет сообщение его автор)
удаляется методом удаления объекта из коллекции, а не маркировки с помощью флага deleted (зачем тогда вам этот флаг вообще нужен?)
если размер истории = 0, то случится неприятный Exception, который у вас никак не обрабатывается

2) реализация метода PUT
аналогично нет проверки на то, является ли редактор сообщения его автором

в остально, ничего интересного. учитывая срок сдачи и недочёты, оценка "нормально" т.е. приблизительно 5


It's possible to integrate your `Makefile`s with vim using the `:make` command.  If you're looking for something else to write about, that would be relevant.

Also, if you type `:! <cmd>`, then `<cmd>` will get passed to your system shell and executed.  I frequently run things like `:! ls` to see the files in my current directory.


Nice idea! I have a feeling that you're inspired by Khan Academy (or maybe not). :P


好像不行 XDDDD


@NobodysNightmare can you define *not work*? Is it the same error?
`uninitialized constant FactoryGirl` or `Factory not registered: hourly_rate`?


thanks. I'll check that out.


Try to motivate *why* you might want to do this.  One thing I've done in the past a lot is opening the same file in two different viewports.  That way I can easily edit different locations in the same file at the same time.


these could use a comment explaining that they start the validation process and will eventually emit validated() or validationFailed().


You can execute common.py?


This is a great start!  I think if you make some high quality animated gifs to go along with your current writeup, that would be a full credit submission.  If animating the gifs is too hard, then doing static screenshots plus the two additions I mention above would be full credit.


Any questions?


There is a PR concerning the cmake script I think in progress here - but this is not yet merged. https://github.com/glfw/glfw/pull/460

I don't see any harm in having the backup FindGLFW until it is properly merged.


Yep :) At least I haven't noticed regressions :D


maybe: argument of uploadValidation()? Currently it's a bit confusing since it does not affect downloadValidation at all...


@JohnAlbin  I don't understand why you need to extend %icon again?


Yes, always the same error: `Factory not registered: whatever_factory_you_like_to_use`


Sometimes, its a little more clear when using the complete ENDIF(..) as to the branch's extent.


@jvanbaarsen Yes please :+1:


Maybe a QScopedPointer so one can't accidentally forget to delete? (ditto for upload)


##This is a markdown file


倒计时


@manens bump ?


minor: This would be more decoupled if it didn't depend on SyncFileItem. It only needs the file path and to output the checksum...


commit


I guess an old folder structure of libusb... But we can probably remove the borland part. And then we need updated paths for linux.


It is recommanded to use subprocess.call instead of os.system.

https://docs.python.org/2/library/subprocess.html#subprocess-replacements
It's better with shell=False.


Hello one of these commits is causing reboots:
https://github.com/TeamHorizon/android_kernel_oneplus_msm8974/commit/827911860221cb4cf8b83f3b58f758433df372e0
https://github.com/TeamHorizon/android_kernel_oneplus_msm8974/commit/0d4fffadbcccffd483b027c38ef748e9d0dd757b
https://github.com/TeamHorizon/android_kernel_oneplus_msm8974/commit/1a4ae92d7a43c966b13a15b6deb8bb278f79f4ce

I suggest you revert to this commit until we have found out what is causing these reboots
https://github.com/TeamHorizon/android_kernel_oneplus_msm8974/commit/1318d2d9e93cc96bbf6bc3f32ad937c8d2603b10


Could you add a comment here that this parameter is optional and is only needed if there exist multiple sessions with the same mapname.


I'm pretty sure we'd want  `15.09` instead of  `15.03-2`


Le video player non synchronise avec le son a cause de faible frame rate. Il faut trouver une autre solution


Yes, I agree, but sometimes it's confusing and often you forget to adapt the ENDIF() if you change the IF(). To have a unique style for the whole code you should use blank ENDIF()


Great, I didn't see that _checksum had more than the final value, all clear then.


I purposely left off a link to this page because its still just barebones. We can easily link to it when the page is actually finished.


should be decimal, not integer


redundant


дп


Super!


assuming you're referring to the test, if you don't the tests will fail on a system with another language.


should these be sanitized individually?


must be unique value


Add `nohup.out` to `gitignore`!


This modification to `ConjunctiveQuery` is a cute example that illustrates why we should stick to the record syntax in patterns and `.copy()` method as much as possible.  Fortunately, there weren't many other lines that deconstruct InferenceRule, but in other common cases, the unnecessary code change can be much larger by adding/renaming/reordering/removing fields to a frequently used class of the parse tree.


Don't remove this, it's needed for 3.10


`--pul`?


why is it string?


why is it string?
add index NOT UNIQUE!


Возможно я что-то делаю не так, но при проверке оказалось, что PUT и DELETE не отправляют запросы на сервер вообще. Глядя в код вижу, например, что функция deleteMessageAjax нигде не вызывается. в место неё используется просто deleteMessage.

EditMessage похоже редактирует только последнее сообщение...
функционал edit + delete, я бы сказал, не то что работает не через Ajax, а практически не работает

Исправно работает только GET & POST

Оценка "нормально".
Надеюсь, след лаба будете сделана лучше.


add movie_id


Speculatively trying to unpack something causes a lot of misleading error messages to be printed for normal channels:
```
$ nix-channel --update
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  5659  100  5659    0     0  65.8M      0 --:--:-- --:--:-- --:--:-- 65.8M
these derivations will be built:
  /nix/store/c0kb267gnpa0fvn9bqz3ar6yslic4jmw-nixos-14.12.673.0672315.drv
building path(s) ‘/nix/store/9ljw6sy1a61z9j126na95pa971wparfp-nixos-14.12.673.0672315’
bzip2: (stdin) is not a bzip2 file.
/nix/store/rygv74phd82c106qynz7l0rmg4rvrlzd-gnutar-1.27.1/bin/tar: This does not look like a tar archive
/nix/store/rygv74phd82c106qynz7l0rmg4rvrlzd-gnutar-1.27.1/bin/tar: Exiting with failure status due to previous errors
builder for ‘/nix/store/c0kb267gnpa0fvn9bqz3ar6yslic4jmw-nixos-14.12.673.0672315.drv’ failed with exit code 2
error: build of ‘/nix/store/c0kb267gnpa0fvn9bqz3ar6yslic4jmw-nixos-14.12.673.0672315.drv’ failed
downloading Nix expressions from ‘https://nixos.org/releases/nixos/14.12/nixos-14.12.673.0672315//nixexprs.tar.xz’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
unpacking channels...
```

Maybe you can just check whether the filename ends in `.tar.*`?


Pas necessaire


You might notice that I've re-added compat.lua, and there is two reasons for that.

1. I'm too lazy to change all .itemid, .uid, .type and .actionid (will be changed eventually)
2. It's just to prevent a large group of newbies from complaining about errors because they're not capable of figuring out why there's a lot of console errors.


add validation


You can leverage some built-in functions:

``` js
array.reverse().unshift(1);
return array;
```

Uses method-chaining (because `array.reverse()` returns an array, you can call other functions on it directly) and some built-in functions. Because [`unshift`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/unshift) returns the length of the modified array, not the array itself, we need to `return array`.


Good


@derekclee It finally turned out as a wrong decision, but performance impact is really marginal. Does it raise any problem?


@jvanbaarsen I have one suggestion to improve the changelog. Maybe I can also open a thread in the forum. What about adjusting the contribution guidelines a bit such that:

* a changelog entry must be included in every *merge* commit
* the commit description must include a proper description (rather than one "fixes a bug with XYZ" line)
* …

That ensures that

* the changelog could be autogenerated directly from git. just pick the merge commits for a "release highlight" summary
* a detailed changelog can be generated easily when there are proper commit messages
* nobody has to maintain the "conflicts mess" in the changelog file any more
* the generated changelog can be placed somewhere on the homepage

Personally I document everything in my commit messages, rather than on the bugtracker. This ensures that I can generate various metadata (changelog, authors, bug descriptions, ...) for free.


Sorry.  Looks like I shouldn't have committed it.
I'll put it back to what you had (and had tested) later on today.
Thanks for testing it (and finding a problem).


@rsiebert: I think this is a wrong commit since the default language (stored int "values/") should be english.
@nickiman: you need to create your own translation into "values-sp/".

@rsiebert: please revert this commit.


Hmmm... If you dont know where your store is going to store things, should Tracking be turned off? It means we remove the Active Record pattern we have at the moment. @bleroy what do you think?


I found the problem in neovim-python.


Using some built-in functions and [regular expressions](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Regular_Expressions):

``` js
return inArray.toString().replace(/,/g, "+");
```


This won't work, you're calling append() with two arguments, you're not appending a tuple.
Plus, if you're catching GitError, you won't catch the IOError that can be raised when the repo already exists.
Please make sure you're testing those changes.


I'd prefer this pattern to avoid nesting conditionals:
```
IF(ENABLE_OPENCL)
  FIND_PACKAGE(OpenCL)
ENDIF()

IF(ENABLE_OPENCL AND OPENCL_FOUND)
  ...
ENDIF()
```


The CSS class for a sucess message is "success", not "OK". See the "newpackage" function in lib/views.py for an example.


Just an alternative syntax you may cross from time to time: `return ++array.length;`


you should also check that both graphs are the same, not just the same size


The "else" here is useless: if the condition is true, an exception is raised, so the rest of the function will not be executed anyway.


Just trying to understand what's happening here. Why delay the fast-prototype check ? and Why `8` ?


No, however... I am not sure having SaveChangesAsync is right here either.... I think we need this at the end of a request, so we could inject a handler that knows when the end of the request is, and call it then.


Already told you, don't mind changin name -> prefix.

How do you call the macro then? Does `<xacro:kuka_lwr_gazebo prefix="${prefix}"/>` work if you delete the macro name?


file is missing a newline at the end


> I commented out that section from the product pom to get the build working again.

Thanks - all working again here!


cygwin中会出现错误


Also look at:
http://cpants.cpanauthors.org/dist/Pod-Tree
And:
http://perlmaven.com/search/cpants

:)


The same concept, with a built-in function and one less local variable:

``` js
if (list.indexOf(item) < 0) {
  list.push(item);
}
return list;
```


I used the ``` slice() ``` function in the refactored version

``` javascript
return bigString.slice(startA,endA) + bigString.slice(startB,endB);
```

It does the same job. Is one better than the other?


At this moment the value of ServiceName is static: 'SP'. It would be nice if this value is filled with the value of "name" from de form.
Could you distinguish the Name (en) and Name (nl)?


Absolutely crazy work for VB6, well done!


Hi,
Thancs for this loader. Its very helpful. Is there a way to load the content of "content-page-..." to the main container of the posts? I am trying to implement your loading script with Masonry layout but this means that the content of every page has to be inside the same container.


Basically, all you'd need is:

``` js
list.sort().reverse()
```


Addressed in 64b65b7


I think it is safe to move the code as you just did.


By the way, it would be a good idea to translate (into English) the function name...


Not better, just different. Because it's all about details, I'd rather point you at a resource on that topic: http://stackoverflow.com/questions/2243824/what-is-the-difference-between-string-slice-and-string-substring (all answers are valid, with some good links).


This is nice. Me like!


Addressed in 7dcc853.


Why not using the functions we write? :)
Else in the refactored version I don't use them anymore because it's shorter to write it with the build-it functions.


test
https://github.com/ffggee/testtest/blob/master/teszt#L3-L7


Closes #8


我也遇到"SignatureDoesNotMatch"問題。原因是瀏覽器無法識別簽名出現的特殊字符，需對簽名做URL編碼，例如將"="轉換為"%3D"，就可以正常使用了。


We don't need to define this line, because "font-awesome" was defined in _init.scss file. So I will remove this line!


You should consider running the zombies server side as it makes more sense.


I need the parent and origin to be able to assemble the arm at will, take a look at [our VITO robot xacro](https://github.com/CentroEPiaggio/vito-robot/blob/master/vito_description/robot/vito.urdf.xacro).

These two params play the role of the $ROBROOT variable of the real robot (check Sec. 4.13 on page 53 of the [manual](https://www.dropbox.com/s/y0fl1bai9kcnnou/KSS_56_lr_SI_en.pdf?dl=0))

Otherwise, I would need to define a piece to attach the robot to, which makes xacros noisy. I don't know if there is a way to make this params optional, that would be a better solution.


this was definitely a bug, but it's fixed in master afaik


I change input but real date input.
The way I use to do was more "user friendly" because they can input in month 1,3,4 or/and a year. YEAR and MONTH are ported into pgsql function.
Anyway with two date input it's better because always compatible whatever database engine and more Dolibarr UX respectfull


Although not usually a problem with modern browsers, the last comma could break the JavaScript engine in Internet Explorer. Plus, it's not needed and most people would say it's ugly, so I'd fix the syntax this way:

``` js
return {
  title: courseTitle,
  duration: courseDuration,
  students: courseStudents
}
```

It's true having the last comma already in place makes it (marginally…) easier/faster to add a new key/value pair to the object, but very few people would leave it :) Note that I also moved the closing bracket to its own line, matching the `return {` line where lies the opening, matching bracket.


We can also use a direct python function call instead of a subprocess.
```
module = __import__('sam-'+args.command, fromlist=[])
module.main(args.options)
```


PUT и DELETE работают только на строне клиента, запросов на сервер не отправляют никаких
Исправно работает только GET + POST

оценка "нормально"


Should we bit a bit more defensive here with content and target incase they are nil?


Победный коммит на 9 мая :)


[![LGTM](http://lgtm.in/p/QKabWlr8q)](http://lgtm.in/i/QKabWlr8q)

TestFormData にユーザーが入力した値をセットする処理が今回の部分になります。登録・更新系は XXXFormData というクラスで、今回のように登録する情報を増やす場合は、このクラスの修正が必要となることを覚えておくとよいと思います。


I try to avoid auto-generated files in the repo, unless you really need it for some reason. In that case, you should add the generation of the file in the CMakeLists.txt


That is different - that just tells Python it can exit without waiting until they finish.  It doesn't prevent those threads from trying to access objects that no longer exist whilst they're running, which is the cause of the exceptions.  For cleanliness I prefer to shut down threads properly; it's nice not to get exceptions dumped to the console and have to check what they are (or worse,  one just starts to ignore them).


It should be "It is possible **to** assign..."


Just to improve your knowledge of the JavaScript language. Here you could have used the `in` operator:

``` js
if (!(newProp in object)) {
  // …
}
```

The behaviour is not the same though. The difference between `hasOwnProperty` and `in` is [explained here](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/hasOwnProperty). It's all about the *prototoype chain*. What do you know about that concept?


It should be "It is recommendable **to** set..."


Check out for more info and download at our website: http://matala3.bugs3.com/


@gabrielalmir10 do you have .editorconfig plugin installed ?


Что-то я вчера похоже фигню написал. Лучше завести атрибут `n_samples` по-умолчанию равный двум. Тогда:

```python
for d in range(self.n_samples):
    # ...
```


I did these exercices in this way because I found it more readable, and with the old habit of the code being optimised with the compilation (for example I didn't pay attention to the use of additional variables or if/else statments).

I learned that's not how it's supposed to be and my refactored version looks like your propositions.


Another, more straightforward way of doing the checks:

``` js
// Let's use a simpler object to store the names and prices.
var priceList = {
  "MacBook Air" : 999,
  "MacBook Pro" : 1299,
  "Mac Pro" : 2499,
  "Apple Sticker" : 1
}

// We only need to check for a specific item in our object: no need for ||.
return (item in priceList) && (priceList[item] <= money);
```


format is more readable:
```
'[{first}:{last}] {nbFiles} files'.format(first=sequence.getFirstTime(), last=sequence.getLastTime(), nbFiles=sequence.getNbFiles())
```


Merger


the webpage is called 'flugzeuginfo', that's why :-)


this is changed in a version of pom.xml to 2.1.0-SNAPSHOT


Excellent :)


Should exit with an error code if there is an error.


Cool! More readable too!


Hey, i thought i share this with you, its a CDN config issue actually at soundcloud's side. Some of there amazon servers are just not sending the Access Control Allow Origin header, while others do. Yes chrome implemented the CORS stuff meanwhile, but the tracks that are not working are just tracks served from a cluster that does not have ACAO set.
I have reported this to soundcloud here: https://www.soundcloudcommunity.com/soundcloud/topics/some-soundcloud-cdn-hosted-tracks-dont-have-access-control-allow-origin-header
And on stackoverflow: http://stackoverflow.com/questions/29778721/some-soundcloud-cdn-hosted-tracks-dont-have-access-control-allow-origin-header

Not much happening since. I'm curious about your new project though :)


This re is the problem for https://github.com/rfk/pypyjs/issues/109


I see. The code is still implying some special use case, when you need to decorate both Latte\Engine and ILatteFactory. That's why I'm curious about it.


Would be nice to configure the machine like this

```ruby
def initialize(rows, positions)
```


Here, the check `if (array.length != 0)` is not really needed, because the `for` construct will simply not run if `array.length` equals 0 and you eventually `return sum` no matter what.


You should fix these spaces => tabs.


Spaces. Should be tabs.


Use ```rows``` here


```ruby
def initialize(positions)
```


I used theses built-in functions for the refactored version, but not as a one line command (I am sometimes unsure of the order of the commands).


I agree. I see this might not have been possible because there's the `wait()` function within the module, which I'd rather have it named something more explicit like `wait_for_free_tp()` or similar (`TranslationProject.wait_to_be_free()` if done as a `staticmethod`)


This should be in tabs, not in spaces. Consider using http://editorconfig.org/


I mean, by adding `if …` you save some computation time by avoiding checking for `array.length` in the `for` construct… but you actually need to check for `array.length` to perform the `if` statement :cake: So I'd stick with the simplest code possible.


Good to know, I hadn't seen this possibility.


我成功加入了 @imsobear  功能，嘿嘿 ^_^


Need to find a better way to do that.


I don't understand this. Why would those dirs not exist?
How about this:
```
preupgrade ()
{
    # Stop the package
    ${SSS} stop > /dev/null

    # Save some stuff
    rm -fr ${TMP_DIR}/${PACKAGE}
    mkdir -p ${TMP_DIR}/${PACKAGE}

    # Revision 7 moves  scripts, backups and Config dirs to ${INSTALL_DIR}/var/
    if [ `echo ${SYNOPKG_OLD_PKGVER} | sed -r "s/^.*-([0-9]+)$/\1/"` -le 6 ]; then
    mv ${INSTALL_DIR}/var/domoticz.db ${TMP_DIR}/${PACKAGE}/var/
    mv ${INSTALL_DIR}/var/domocookie ${TMP_DIR}/${PACKAGE}/var/
    mv ${INSTALL_DIR}/backups ${TMP_DIR}/${PACKAGE}/var/
    mv ${INSTALL_DIR}/scripts ${TMP_DIR}/${PACKAGE}/var/
    mv ${INSTALL_DIR}/Config ${TMP_DIR}/${PACKAGE}/var/
    fi

    mv ${INSTALL_DIR}/var ${TMP_DIR}/${PACKAGE}/

    exit 0
}

postupgrade ()
{
    # Restore some stuff
    mv ${TMP_DIR}/${PACKAGE}/var/domoticz.db ${INSTALL_DIR}/var/
    mv ${TMP_DIR}/${PACKAGE}/var/domocookie.txt ${INSTALL_DIR}/var/
    mv ${TMP_DIR}/${PACKAGE}/var/backups ${INSTALL_DIR}/var/
    mv ${TMP_DIR}/${PACKAGE}/var/scripts ${INSTALL_DIR}/var/
    mv ${TMP_DIR}/${PACKAGE}/var/Config ${INSTALL_DIR}/var/

    rm -fr ${TMP_DIR}/${PACKAGE}

    exit 0
}
```


Without ssl=on postgres wont start with this line in pg_hba.conf


For the archives, I have updated the one-liner to skip already compressed files, and fork 8 processes. Now maintaining [a gist](https://gist.github.com/fxn/427dca61ec44adf8253b) with the snippet.

@amw I think the task should run after `:normalize_assets` because the NGINX module recommends the mtime of the compressed and uncompressed files to be the same. In my one-liner I delegate that to `gzip(1)`, because it automatically sets the mtime that way.


@open is a boolean already -> nicer solution with 1 line needed


This is supposed to handle logouts, but it seems to break everything. Anybody know why?

https://github.com/plataformatec/devise/wiki/OmniAuth:-Overview#logout-links


寫一點建議ㄎㄎ


It's just BC


`cp inputA inputB output`
Need to use dedicated arguments for input(s) and output instead of a common one.


Tanken är att när man parsar XML-filen så ger man ett fel om det är en state som saknar transition. Det var därför vi hade ett felmeddelande där.


You could use one of the binary-to-hex convertors (xxd, uuencode or base64) and paste the encoded result as a comment.


It's just BC, people were using still the old one. I like to not break BC if it's possible.


Here's the error message:
```
Exiting
/home/vagrant/.rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/actionpack-4.0.1/lib/action_dispatch/routing/route_set.rb:434:in `add_route': Invalid route name, already in use: 'destroy_user_session'  (ArgumentError)
You may have defined two routes with the same name using the `:as` option, or you may be overriding a route already defined by a resource with the same naming. For the latter, you can restrict the routes created with `resources` as explained here:
http://guides.rubyonrails.org/routing.html#restricting-the-routes-created
```


Need to declare mutual exclusion groups.
https://docs.python.org/2/library/argparse.html#mutual-exclusion


Ok


For the closing bracket, that was more a small omission, I usually do it on a new line.

For the last comma, I like it better with it already there, but I can understand if it's best without. I'll take the habit to leave it.
I had an issue with the config file of sublimetext because of this.


tutaj dodaje sobie studenta z danymi i on mi się wypisuje. a jak chce dodać kolejnego studenta to co muszę ?


This will call the function, instead you need to pass the reference to convert.
You need do remove the parentheses, to just pass a reference.


close


I just don't understand the syntax. Is https_validate_cert an integer? You are creating a tuple, not a dict.


no powtorzyc to co robisz w linikach 5 - 16


One more thing: I don't know which version of the robot you have, and don't remember where my meshes came from, but I realized that the meshes do not match the geometry of the robot we have, take a look at the [CAD](https://workbench.grabcad.com/workbench/projects/gcD7ogLY-vAXEQAdT63_IonX9Nk77QjqUiJXTZV12teqlA#/space/gcLZU2Jx3pu93V41x-QoeNkwbLBBilA0yP-NWwe0IBXkMM/link/170801/version/1) (check the sketch and comments on the right)


have attached two packet captures:

  poi.pcap packet #2 shows a POI TLV with one system-ID
poi3.pcap packet #3 shows a POI TLV with two system-IDs

On 5/20/15 11:42, fxlb wrote:
> You could use one of the binary-to-hex convertors (xxd, uuencode or
> base64) and paste the encoded result as a comment.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/the-tcpdump-group/tcpdump/commit/be03cc0b4ae9370ea0483d1504f4a950b22625d3#commitcomment-11279846>.
>


Oh, `thecl`.


Using the [`toLocaleLowerCase`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/toLocaleLowerCase) built-in function, along with [`split`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/split) which is definitely something you'd want to use:

``` js
return inputString.toLocaleLowerCase().split(",");
```


I noted one difference between the `master / build.ps1` and `2272 / build.ps1`.
This concerns the definition of `$MSBuildExe` variable.
I think the definition is correct in `2272 / build.ps1`.


Gemein war natürlich #19 ;-)


noted b


Sure, but other implementations are going to try and implement the hash algorithm here so it is compatible with Compose, making this implementation even more sticky. It'll always be preferred to support existing behaviour than break deployments, however minor the breakage is.

I'd suggest also removing whitespace, which makes it pretty much canonical JSON and much easier to make compatible with other languages - see [first note here](https://docs.python.org/2/library/json.html#basic-usage).


On 20.05.2015 11:48, marvin2k wrote:
>
> close
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goldhoorn/package_set/commit/34a85a0718af22807a6f042088eaf27912a226db#commitcomment-11279912>.
>
*updated pr*
https://github.com/rock-core/package_set/pull/46


That was the solution I was looking for.
I only didn't knew how to add the space in the property's name.


Test


Facter is very easy and all it takes is `json.loads(subprocess.check_output['facter', '--json']))`


just remove this????


to nie może być pętla?


Ok I trust you that you don't trust NuGet :smile:


8 constructions is needed for slack tracking to stabilize although here it's unrelated.

Not sure what you mean by delaying fast prototype check?