Wow, I spell checked it but accepted it without thoughts. Fixed


It's nice to have the flexibility of all the alternative structures, but perhaps we should stick to a canonical way in docs and samples?


trash......


:+1: great job on adding tests


drobný přepis :D


Thanks, fixed.


return_value_handling (lefticus) - Win64-Windows-8.1-VisualStudio-12: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


I agree. I will submit a new PR to deprecate RC4 in next week.


@kasper93:

1. right clicking on the timer, then hitting Esc makes the timer toggle to remaining time
2. right clicking and moving the mouse anywhere leaves the hand cursor. Is that normal/can it be improved?


nice one


Using your formula ( :D ) Didn't study this yet, but  I can tell you that you missed the init-pro part :D Don't load activation classes when in customize. :D


Thanks. I wrote the table in pdf for a support documentation for reviewing to know why some define were added and removed. It does not matter not to check it if you do not need it.


Thanks. Fixed it.


I did
```
pacman --needed -S bash pacman pacman-mirrors msys2-runtime
pacman -Su
```
If that's correct I am on the latest.
The build of cmake (default, I didn't change anything) just broke in package() with exactly the same message as when I try to open cmake:
```
/repos/MINGW-packages/mingw-w64-cmake/src/build-x86_64-mingw32-bin-cmake.exe: error while loading shared libraries: ?: cannot open share object file: No such file or directory
==> ERROR: A failure occurred in package()
       Aborting....
```
This is in a msys shell. When I ran the cmake binary package from the distro I did it in a mingw32 shell.


Added them.


Übung 1


blockchain .info aip/v1


method_missing (arBmind) - i386-Windows-8.1-VisualStudio-14: OK (336 of 336 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


you tinging


I will add a new commit to fix this.


goodbad


Fixed


Ok, open ````/repos/MINGW-packages/mingw-w64-cmake/src/build-x86_64-mingw32-bin-cmake.exe```` in dependency walker (http://www.dependencywalker.com) and see what dll is missing


Yes, fixed it.


Is just my impression or the preview loads faster now?
If isn't just a wrong suggestion.. this could be a great improvement!


What's the reasoning behind signing your comments with " - [name]"? I've come across a significant amount of signed comments while scanning through the code base, but I have never seen any other project use this convention.


Thanks 🍻


gyp handles MSVS_VERSION which has a suffix of e which stands for Express version. https://github.com/iojs/io.js/blob/v1.x/tools/gyp/pylib/gyp/MSVSVersion.py#L227-L321
It is not the case in this commit but `MSVS_VERSION == 2012`  would fail when VS2012Express is used.


I know :) I'm on it thanks


Thanks. fixed it and sorry for many typos.


method_missing (arBmind) - Win64-Windows-8.1-VisualStudio-14: OK (336 of 336 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


@jehiah noticed you're testing r.backoffCounter but using the local backoffCounter when calling the backoff strategy/logging. shouldn't have any impact since startStopContinueBackoff is the only method which updates r.backoffCounter.


'Hebrew'


@kwalrath is it needed? Normally `bower install` should kick-in automatically from the npm's post-install script: https://github.com/angular/angular/blob/e18920884e562d18e279a117f86d6e07e1e9bde3/package.json#L24

Did you have to do it manually?


Related issue: #27


hebrew


Thanks. Fixed it.


> That is because you use EXE packers. But it's still worth doing since it's a one time thing :)

I don't think the EXE-Packer (UPX) or the Setup's compression (NSIS) can save any bits on PNG files, as PNG files have already been DEFLATE'd. Compressing already-compressed files does *not* work well.

Anyway, if we look only at the "raw" EXE file size, taken directly from MSVC compiler – *no* UPX or anything – the whole PNG re-compression stuff has reduced the size to exactly **99,5%** of the "old" size.

(Savings are actually a bit higher for the whole Setup package, as the latter also includes the manual)


'hebrew'    : 'Hebrew',


I am moving if out of a for loop here (line 1404/1406).
I did benchmark the commit before and there was a slight speedup (there's a ticket on trac about a possible internal check).
About the blank new line, ok I could have skipped that, sorry for that.


this is ok..


@jbustio could apply this update to develop branch also


when removed, background worked.


Fixed style and I agree that it  has a bug like 3.10 > 4.0. I fixed it to store a string of the version float number.
```diff
--- a/configure
+++ b/configure
@@ -368,13 +368,13 @@ def get_llvm_version(cc):
         '''
     sys.exit()

-  match = re.search(r"(^clang version|based on LLVM) ([3-9])\.([0-9]+)",
+  match = re.search(r"(^clang version|based on LLVM) ([3-9]\.[0-9]+)",
                     proc.communicate()[1])

-  if match is None:
-    return 0
+  if match:
+    return match.group(2)
   else:
-    return int(match.group(2) + match.group(3))
+    return 0
```


Like so?:
![cmakelibnotfound](https://cloud.githubusercontent.com/assets/1502057/7101124/aabfdbb8-e03f-11e4-92bd-d5c4805dde60.PNG)


Added it.


else if ( language == 'hebrew' ) {
// Hebrew *is* Windows-1255
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1255'), targetEncodingCharset );
}
else if ( language == 'russian' || language == 'ukrainian' || language == 'bulgarian' ) {
// Cyrillic *is* Windows-1251
dataBuff = iconv.encode( iconv.decode(dataBuff, 'windows-1251'), targetEncodingCharset );
}


Haven't given it a thought yet, but this loop can probably be combined with the earlier one where the layout of the labels is happening. We will have all information required *after* the labels are laid out. Might need to do two switch case statements though (or put the whole thing in two functions called one after the other within the same loop.


@MacDada no cześć, zrobiłem kilka poprawek w kodzie, zaraz wrzucam też nową wersję sapera i biorę się za przepisanie kółka i krzyżyka obiektowo. Tutaj mam do ciebie pytanie odnoście wymodelowania tej gry - czy potrzebne są 2 klasy (board i field), czy więcej? Dla każdej klasy utworzyć osobny plik?


@ohagendorf hier ist revealjs dazugekommen. Doku und Repository gibts *natürlich* bei GitHub --> https://github.com/hakimel/reveal.js


@schism well, cooperation would be alright. On my side, it's fine to send a acknowledgement before taking parts of GPL software, it's a good way of having a cooperation (let alone the ebuilds you've made have a great possibility of breaking Spike pentesting distro because it's Sabayon repos based)
If there is any chance of promoting your work through an organized group like Spike distro, I would request @necrose99 to keep you up-to-dated with further modifications in their copied ebuilds or even join the boat (this may be more useful to you and their team)


Try open ````libarhive, libcurl```` in dependency walker and see if they not have missing libraries.


currently, while we are in the build phase we are depending on the full framework, we will look into how many libraries we needed to use before the release


Ligne 15 dans la classe humain, je ne comprend pas pourquoi tu changes la valeur de
"valueOf(cannette.vider(tailleGorgee))"
en String


That was just a wrong copy-paste :)


libarchive-13.dll:
![libarchive-13 dll](https://cloud.githubusercontent.com/assets/1502057/7101149/cd819cde-e041-11e4-8435-2848cac102ad.PNG)
libcurl-4.dll
![libcur-4 dll](https://cloud.githubusercontent.com/assets/1502057/7101150/da52e81e-e041-11e4-8d36-90673810eb31.PNG)
This is with `depends` running inside a mingw32 shell.


I split them so that it's possible to unit test them without linking the whole project + ogre. It's only temporary, but that was the most straight-forward way I could find to solve it.


Testing my Github


should `Object.class` here be `j.l.Object` or `c.l.Object`? (Currently, it appears to be the former, which may have caused #653.)


Following Laracasts!!


So King can run on nginx plus?


The reader will need something like this as well, otherwise i get this warning from recent versions of exiftool: `FileName encoding not specified.  Use "-charset FileName=CHARSET"`

I hope this does not break old versions of exiftool.


Why ````libgnutls-28.dll````? GNUTLS-3.4.0 have ````libgnutls-30.dll````


It is, I use TDB 335.58 and I import all the world sql's only this one products the error, its a fact, anyway, thanks for helping me.


Needs the logger too, not print.


Seems you have old packages....


This gives a brief idea of the time required to build, scan and insert all elements of the file into the HASH table.


We just like signing things, it's not like it's a policy. It can be handy
to know who did something without needing to dig through the blame/file
history, though.

- Colby

On Sat, Apr 11, 2015 at 3:28 AM, Colin Wallace <notifications@github.com>
wrote:

> What's the reasoning behind signing your comments with " - [name]"? I've
> come across a significant amount of signed comments while scanning through
> the code base, but I have never seen any other project use this convention.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stepmania/stepmania/commit/cf3c967a51721bc83d35ca11f61f67be5051cc09#commitcomment-10677485>
> .
>


Takes full backup of mysql database, compresses and emails it.
The final compressed backup file is named as <database name>_<date-time>.gz


This is pretty new: [exiftool 9.79 introduced unicode support](http://www.sno.phy.queensu.ca/~phil/exiftool/exiftool_pod.html#windows_unicode_file_names)


i'm confused on the behavior this produces.  if i want to write code in ES6 with promises etc., but target ES5 for the build output, will this code pull in `lib.d.ts` by default instead of `lib.es6.d.ts`?


@mbebenita, this is so, so weird: the test does absolutely nothing whatsoever that could cause intermittent failures, but I had to disable it because it passed ~ one out of three times.


@ibuclaw Any idea how to port this to gcc-4.9? The `expand_thunk` funtion doesn't have a `force_gimple_thunk` parameter.


Well, I installed after building the latest gnutls as instructed. I'll try to remove GNUTLS and reinstall from the build.


What about the following?
```php
protected function _parseDSNString($config, $string){
    $placeholders = array_map(
        function($value){
            return '{~' . $value . '}';
        },
        array_keys($config)
    );
    $binds = array_values($config);

    $dsn = str_replace($placeholders, $binds, $string);
    $dsn = rtrim($dsn);

    return $dsn;
}
```

Name the variables what makes the most sense (e.g. `$config` could be renamed).


I just made a clean install yesterday, I don't get any error.


It's not a good idea to check for something being NULL before using it two lines later.


What is this change about? This line makes no sense at all since the first part is already checking what your newly added part checks.


Can this be just be `exports.compile` = `mote.compile` ?


Wrapped the line and I fixed the version check to compare in string.

```diff
     'conditions': [
-      ['(OS=="win" and MSVS_VERSION[0:4]>="2012") or llvm_version>=33 or gas_version>=223', {
+      ['(OS=="win" and MSVS_VERSION[0:4]>="2012")'
+       ' or llvm_version>="3.3" or gas_version>="2.23"', {
         'openssl_sources_x64_win_masm': [
           '<@(openssl_sources_asm_latest_x64_win_masm)',
           '<@(openssl_sources_common_x64_win_masm)',
````


May I ask you to put `or` on a previous line? This is how we do it for js...


Sure I do.


It might be more reliable to use [ent](https://www.npmjs.com/package/ent) for this. Though if you need to escape just these specific cases I guess the current solution is enough.


Jj, vrátím je tam (a přidám je do svých testů)


Fix minifyCSS() for Quoted URLs in Media.php and only pass to replaceByAbsoluteURL() if data:, http: and https: not specified.


А почему не подходит?


Ok, mas pravdu, ze to, aby to hazelo subclassResponsibility je lepsi, takze to vratim.
No a v tech podtridach, ktere nemohou byt source nebo target tu metodu vubec implementovat nebudeme jako doted (coz se nelibi code criticu, ze delame instanci tridy, ktera ma neimplementovanou abstraktni metodu) nebo to implementujeme tim at to pise chyby ve stylu ze tohle nemelo bejt nikdy zavolany (coz by znamenalo kopirovat to samy napric celym dynacasem v desitkach pripadu)?


If it was a GNU replacement you're missing 900 lines of boilerplate.


You put @mail not @gmail.


:+1: sólo una cosa más, @rauldiazpoblete, prueba a embeber dos fotos en una misma página, para asegurarnos de que funciona. gracias


Sure, np. Just saying. :)


Ok, noted.


```diff
-shouldn\'n
+shouldn\'t
```


Why dot?


I don't think the | completion works otherwise, does it?


Can you update the `install-again/before` state, so that this integration spec actually ensures, that the files aren't touch if there are no changes? That's important, because it could otherwise unnecessarily fault clang's prefix header compiler caches, because there was a modification on file system level to a global available header, which drastically increases the built time.


SO NICE! I'LL ADD MORE ko_KR!


> right clicking on the timer, then hitting Esc makes the timer toggle to remaining time

Will fix that, need to index menu items from 1.

> right clicking and moving the mouse anywhere leaves the hand cursor. Is that normal/can it be improved?

I probably could revert normal pointer when menu shows.


@HmHmmHm More translations for the PocketMine core itself are coming :)
I'll move everything soon to Crowdin, and I'll add your translations there.


What's the behavior now for events that repeat forever?
I'm asking, because I consider using the library for https://github.com/real-or-random/dokuwiki-plugin-icalevents .


Good to see you fiddling a bit. ;)

Disclaimer: All that will be said below is a matter of taste and personal experience:
Since the models names are hardcoded, there is no reason to add too much cleverness in determining their name IMHO.
Indeed, the code is good but you said it also, IMO, you're overdoing it a bit.
In that specific case, I'd simply use something like, since it's a bit more readable and easily changeable:
```c++
std::string meshName = Random::Int(1, 2) > 1 ? "MeshName2" : "MeshName1";
```
That said, that won't kill a cat.
Your change about removing the duplicate calls for x and y is well thought, btw.


To jo, ale mne jde v tomhle pripade removeModelElement spis o to, ze subclassResponsibility rika "mel bys tady mit implementovanou metodu removeModelElement", coz mi nepripada, ze je pravda. Takova metoda podle me nedava vubec zadny smysl jinde nez u diagramControlleru a u par vyjimek v podobe kontejneru, a to i kdyby proste jen vracela obycejny error jako sem to udelal ja.
Nemit tu metodu vubec, tj. aby to davalo chybu does not understood, mi prijde lepsi.


How can I made change in preference file to apply new font family and size? I build a version 0.8.6dev with this pull requests. And then modified `~/.config/robomongo/robomongo.json` file by adding "textFontFamily" and "textFontPointSize", but the font is not changed (neither family nor point size):

![deepinscreenshot20150411210443](https://cloud.githubusercontent.com/assets/216930/7101244/87353d5c-e092-11e4-8edb-0e402eb2b47e.png)


Jest do tego szablon pliku CSV?


Alright. Fix coming up.


Did you add the external reference to the Device API as mentioned in the relevant Issue? There is t any way to represent that directly as part of the pull request.

Also, as a tip for future testing, you can do a trial run of the merge locally so master doesn't have a series of accept and reverts. It allows developers to more easily use git's bisect capability for tracking down bugs. If you want, I can completely erase the accept/revert sequence as well to fix that as well.

Jason Ziglar

Sent from my iPhone

> On Apr 10, 2015, at 11:13 PM, nathodius <notifications@github.com> wrote:
>
> This merge didn't produce a successful build
>
> —
> Reply to this email directly or view it on GitHub.
>


thanks  :heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


I get the error when creating an uptodate DB (presumably including this one)..

```
Creating new databases tc_world_test001, tc_characters_test001
World...
ERROR 1062 (23000) at line 4876: Duplicate entry '52283' for key 'PRIMARY'
Characters...
Importing template characters
Done
```

So, not sure what's going on there.


dereferencing _settings does not consume much time. There cannot be a measurable speedup by this particular change.


Hi Oskar-j:

I've just joined to github because I'm very interested to install and run this hyip-similation, but I don't know how to do it. I've downloaded the zip, but I don't know what do I need to run it.

I'll appreciate any helpful advice.

Paco.


TeamCity Azmq :: Build (Windows) [Build 113](https://78.46.71.115/viewLog.html?buildId=457&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 87](https://78.46.71.115/viewLog.html?buildId=456&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 71](https://78.46.71.115/viewLog.html?buildId=458&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 87](https://78.46.71.115/viewLog.html?buildId=456&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:10


TeamCity Azmq :: Build (Windows) [Build 113](https://78.46.71.115/viewLog.html?buildId=457&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:19


TeamCity Azmq :: Build (OSX) [Build 71](https://78.46.71.115/viewLog.html?buildId=458&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:22


This may be redundant, actually. If the code can be put in project_start, wouldn't that be cleaner?


beautiful! :heart_eyes_cat:


Sagt man das so, Aspekte eines Gottes? Ist mir noich nciht untergekommen... Gemeint sind ja unterschiedliche Bevölkerungen die denselben Gott unter einem anderen Namen anbeten. Da aspect auch als Erscheinung übersetzt werden kann, schien mir das einleuchtend.

Ich find halt Aspekt klingt nach einem Bereich des Lebens, in dem der Gott eine Rolle spielt und ist deswegen missverständlich.


Why do you need such special case processing?


Why do you use UnknownError everywhere?


... Thanks Jorrit! xD


I am adding another comment


I know its odd, but I just did a clean install as well, just to check and re-check twice, this error occurs no matter what.


Oder  "...fand das Schwert in der Großen Halle der Festung, auf dem Thron liegend."  Mir ist's egal. Machst du das oder soll ich es ändern?


Related issue: #48


@shoghicp YAP:P I understand!


Accidentally pushed to the wrong remote? (this is upstream).


Cool!


Thanks. Will be helpful for migration also. Thank you.


The `child_process.exec()` was already using the quote wrap for Windows platform.

Without the above fix some commands already containing quotes could not be executed with `cmdSync()`, for example:

```js
var cmd = '"' + process.execPath + '" package cmdSync_native.js "test" -native';
console.log(jxcore.utils.cmdSync(cmd).out);
// 'c:\\jxcore\\jxcore\\Release\\jx.exe" package cmdSync_native.js "test' is not recognized
// as an internal or external command, operable program or batch file.

var cmd =  '"' + process.execPath + '" -e "console.log(1);"';
console.log(jxcore.utils.cmdSync(cmd).out);
// ''c:\jxcore\jxcore\Release\jx.exe" -e "console.log' is not recognized
// as an internal or external command, operable program or batch file.
```


I don't know, it should IMO. The dot here is confusing - it's not a dot completion, it's `position` (and `usage` above).


This revision is clean.


added pagination script


I'd actually go with "Rendering".


We can't support | in that position because it's a valid expression without the dot. We could change the documentation here to use the concrete byte offset instead, but that's always so finicky and breaks when line breaks change. I guess we should simply mention that real completion would use the offset just after "split".


/CC @kasper93 for next batch


H-m. Do we have one more SI Kafka Support project ?


This commit is not necessary please drop it


That might be indeed good, as we would want to avoid invalid locales to be used.


red is not a good color in this case, cause red may refer to Warning or Cautions in urgent situation, so how about change it to other color?


Looks like this is `utils` package candidate


Probably... I did notice that the function argument signature of `mote` doesn't take `options`:

https://github.com/satchmorun/mote/blob/master/mote.js#L617

Thoughts?


Good idea.


>Here is what you need to share:
- All the files under **.idea** directory in the project root **except** the **workspace.xml** and **tasks.xml** files which store user specific settings

Excerpt from [IntelliJ Support Knowledge Base](https://intellij-support.jetbrains.com/entries/23393067).
I think this mean that you should only ignore those two files?


11 April 2015


`abstract`? No ?


Sure :unamused:


check that you commented here
https://github.com/dorimanx/Dorimanx-LG-G2-D802-Kernel/commit/2f4294c640f4ff24167dddcf40e7ac649b101263#diff-96ded33bf0461d243584af76967f231dR278

or it's will be selected again on build.


okay, green seems good ? :)


Could you test this new configuration with latest git?

```
UserHacks_align_sprite_X = 1
UserHacks_round_sprite_offset = 2
UserHacks_stretch_sprite = 0
```


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


`/SpringIntegrationMetadata` ?
From other side I'm sure it must be as ctor arg and `final`.
See `MongoDbMetadataStore` for example


I think I commented it out because it made it harder to test the flexbox things and forgot to add it back. :stuck_out_tongue: However, wouldn't a better solution be to default the time to, say, 05:00 (like with tag filtering) if no time is set?


Don't see reason to have them here. Just `initialization on definition`:
```
private final List<MetadataStoreListener> listeners = new CopyOnWriteArrayList<>()
```
etc.


`Assert.notNull(client)`


I am not sure, how about test more color, I prefer grey or pink?
Not only this line, this three red color are all too urgent.


Hi,

Any chance of merging into the 0.8.0 branch? I'm using 0.8.0-dev because of some dependencies incompatibilities with other modules on 0.7.x


Thank you my friend, but it wasn't necessary for me, I didn't have any options that select automatically  CONFIG_MSM_RPM_LOG and CONFIG_MSM_RPM_STATS_LOG.... :)


Looks dangerous. `SmartLifecycle.start()` instead of `afterPropertiesSet()` ?


Actually it was a good idea.
This allows us to fix it against a known working version, instead of running into issues with each fetch.  Now I can update the engine when it gets something we need and bring the mod up to date in a single step.


It should be `c.l`


type missmatch


The style of `else` clauses is inconsistent, even with the automatic styling. Sometimes it is `}else {`, and othertimes `} else {`. The space between the `else` and the next opening bracket seems to be enforced, but the space before the `else` is not.


type mismatch (use std::size_t or typename::size_type


Is `cache.close()` idempotent? I mean will it work if we stop/start during runtime?


#bug
hardcoded string constant multiple usages


method_missing (arBmind) - i386-Windows-8.1-VisualStudio-12: OK (336 of 336 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


ok, bad example, will fix it


"raindrops on roses"


what about the other contacts from the list? it looks like we only check the first contact, was this the intention?


`kafka` haunts you :smile:


Doesn't `CuratorFramework` provide something like `atomic session` to avoid such a `synchronized` ?


Seeing the code below, this variable can be `final` as well.


lol


That's all. Looking forward for the PR!


I am curious why the locking via pidfile was removed. Right now multiple umdls are running on the backend and a locking like this seems to be a good idea.


*chapter


<string name="prefDialogTitle_suggestedContactsMode_summary_all">Alle vorgeschlagenen Kontakte sind wählbar</string>


Ok. I understand the issue. The game uses a different code path. Not sure I can fix it. I will check others game first.


+  <string name="clear_suggested_contacts_dialog_message">Alle zuvor gespeicherten, vorgeschlagenen Kontakte werden entfernt.</string>


Close https://github.com/polomarcus/wake-me-up/issues/14


Nope, they're the same order


@ewirch
Hi Edu,

komischerweise hat es sich bei mir nicht komiplieren lassen, so wie es war. Ich hatte die Meldung, dass er den Typ 'prog_uchar' nicht kennt. Das wird an Änderungen am avr-gcc liegen, dass dieses typedef jetzt fehlt.
Die Änderung auf const byte sollte aber nichts an der Funktion von lightmashine ändern.
Mit Arduino IDE Version 1.6.3, also der neusten.

Gruß,
Nico (abz?)


double add = 0;
        for (int k = beg; k < beg + part; k++)
        {
            add += (4. / ((4 * k + 2.)*(4 * k + 3.)*(4 * k + 4.)) - 4. / ((4 * k + 4.)*(4 * k + 5.)*(4 * k + 6.)));
        }
        unique_lock<mutex> _lock(_mx);
        pi = pi + add;
        _lock.unlock();


Please drop python2_6 here.


It looks more gentle


@MacDada ta zmiana sprawiła mi dużo kłopotów, dużo czasu spędziłem na debuggowaniu - w kilku miejscach były błędy i gra się crashowała. Funkcję start_game() podzieliłem na dwie, dodałem funkcję main_game() która zawiera główną pętlę gry. Ustawienia gry ustawia się teraz za każdym razem przed włączaniu gry (jeśli wybierze się opcję "nowa gra").


Small copy paste error


So I don't have a libgnutls-28 in my msys/mingw system. Cmake nevertheless keeps asking for it... What I have is libgnutlsxx-28.dll, but I renamed it to see if it makes any difference...
So what could stop cmake from asking for libgnutls-28.dll? Rebuild cmake?


Thanks for the PR. The challenge is actually around "stock prices" though, so the folder title is correct!


this issue should be fixed now


The standalone emacs-24.5-x86 for windows, not the MinGW version.


WAW


If you want to mix our builded packages with your own you need control dll's version yourself. I can't help much here because I don't know what package versions do you have and what of them is your own


packages in our repository working


> Since the models names are hardcoded, there is no reason to add too much cleverness in determining their name IMHO.

Agreed.

> In that specific case, I'd simply use something like, since it's a bit more readable and easily changeable:
```std::string meshName = Random::Int(1, 2) > 1 ? "MeshName2" : "MeshName1";```

Sounds good for the WeaponShield meshes. For the 4 TrainingDummy meshes I guess I'll change the current code with if / else if / else to a switch on Random::Int(0,3), since there is now 1 model more to take into account and if / else if / else if / else seems a bit redundant :)

BTW is there a particular reason to Random::Int(0, 2*n-1) instead of Random::Int(0, n-1) to determine the mesh number in the current code? e.g. Random::Int(0,5) here instead of Random::Int(0,2).

>  Your change about removing the duplicate calls for x and y is well thought, btw.

Yes I'll rebase and keep that part.


If we set the ASSET_HOST then this could be kept as ASSET_HOST


+  <string name="prefDialogTitle_suggestedContactsMode_summary_all">Alle vorgeschlagenen Kontakte sind wählbar</string>


that's odd, seem to work completely fine here… from the installation step:
```
libtool: install: /usr/bin/install -c .libs/overview.so /tmp/_install_gtk2/lib/geany/overview.so
```
(and that's not a joke, it really is installed there :)

All I did was using `PKG_CONFIG_PATH` so it finds my Geany installed in */tmp/_install_gtk2/*.

Didn't you provide a `--with-geany-libdir` when you configured by any chance?  (you can check the configure line in *config.status* -- and here's a hacky line to extract it: `grep -F "set X" config.status | head -n1`)


I had left those trailing spaces on purpose to show that stuff needs to be added there :-)
But it doesn't hurt to remove them anyway.

In the models/ part the extension should be changed to .mesh instead of .png, and WeaponShield* should be before White.png in the texutres/ section, and before Wizard.* in the models/ section ;-)


Hello,

That looks like a small issue with replacing loop variables.

In second case it's propably the "_" character, I wasnt tested variables replacing with "_" characters.


Great! @balmas , a desideratum for the long-term might be to be able to give two tags to one word or expression, such as "fog" and "darkness". They don't always co-occur, which is why I didn't nest them, but they often do. I'm wondering if this is possible already with a work-around, for instance by tagging one word as "fog", and then the surrounding expression with "darkness"?


yeah, I’ll remove this


This is the commit ( 71dc999 ) from which we made build 1015, which was mentioned on Twitter:

JeanSeb on Twitter: "Hi @dataverseorg, that's kind of difficult to delete a dataset! Even deleted files are still listed (v4.0 b1015)" - https://twitter.com/jeansebback/status/586448919850127360

@scolapasta and I were actively working in #1784 and for this exact commit I noticed and reported very strange behavior and exceptions: https://github.com/IQSS/dataverse/issues/1784#issuecomment-90717960

Many commits later on that issue I was satisfied that my tests were passing at least. As of this writing the issue is in QA.

To summarize, the DeleteDataFileCommand was very much in flux as of build 1015. I would be very curious to hear if the problem reported via Twitter can be reproduced on a server with more recent code. https://apitest.dataverse.org would be a fine place to test since it has the latest (though it has different build numbers). As of this writing it's running e0213b4 .


upd


Up for discussion but there should be a way to configure how long indefinite recurring events should repeat for.

I use the following and set them to stop after 2 years from the day the parser is ran but others might want longer/shorter.

```php
//...
else {
    $two_years_time = date_create('now');
    $two_years_time->modify('2 year');
    $two_years_time->setTime(23, 59, 59); // End of the day
    $two_years_time = date_format($two_years_time, 'Ymd\THis');
    $until = $this->iCalDateToUnixTimestamp($two_years_time);
}
```

Along with this there is also what to do with past events; for example if you consider an indefinite recurring weekly event which started in 2010 and you use my snippet it would effectively be parsed as 7x 52 events which is a quite a lot if you're not interested in the past ones.

Not sure how best to cover this in order to satisfy everyone.


Hello,

Looks like a small issue regarding Tpl::resolveTemplatePath()
Sorry, I didn't test it on Windows platform. The target of my all projects is Unix operating systems at first place.


:tada:


I imagine this issue with paths would be present on unix as well.


Good catch haha  all fixed now


One step closer to the state machine ^^


Removing Unicode support?!! WTF


Consistency with the other code is more important. If we ever convert this to use their conventions, I'll do it in bulk.


I did a push yesterday which removes the uses of scoped_ptr. You should refresh this.


cannot


Something odd happened in this commit. The indentation is off from https://github.com/bjori/mongo-c-driver/commit/c51ce6a7778c35e85e0d8c9d854626d90846c320 and your original 1.2.x merge commit (https://github.com/mongodb/mongo-c-driver/commit/b8422e4c11b0d15b32a078404e20073551c1ffb0), and Hannes' authorship is missing (I don't believe this is a merge commit, as it only has one parent).


so add


Done. https://github.com/kasper93/mpc-hc/commit/0df781ae2b3b9fd07c5b33ebb76f0afd694b5c60


This is just a compiler optimisation and increases code size.


Oops, sorry about this. I should have caught that.

Also I think the VS formatter went a little nuts and removed all those extra whitespaces. Does that actually cause problems in the linux build or is it just your coding style?


I found the whole configuring rather annoying but I accept jscs as linter. Should I remove this?


OK, thanks!


Yup, confirmed working now! Thanks @Kittnz !


@rhb-anme did Xcode 6.2 fix the issue?


> Is it really simple to get the one? For example, typographic-styles is not relevant.
we can introduce `textr-friendly` or even `textr` keyword


Je ne suis plus trop sûr du fonctionnement de git, j'ai fait un commit, c'est suffisant pour que tu aies accès à mon travail?


I dunno! I'll have a squiz now.


It should have just worked!


> Also, list of the plugins makes users easy to start with textr' cause it's removing the confuse.

You are right, but what’s wrong with removing plugins list from readme? You will not be able to update readme everytime to add every new textr plugin. That’s it was unmaintanable => pretty useless, that’s why I decided to remove it and give simple link to npm


Instead of a string, did you consider using the built in booleans? (True, False)


What do you think about it?


@Underground78: I just had an idea. Could we parallelize this loop? This should speed up the process significantly.


Do you have the project to propose uBlock0 on AMO like fork from uBlock please ?


It's not nice to have deprecated APIs from the start. Since this has never been released, could you just remove it?


I now have to emulate your hard-coded directory structure with symlinks.


@Mumfrey Can we get that in a dev only docs then? Because it's still useful to have fully documented somewhere.


All I can think is backport it.  Default initialise the parameter to ``false`` so you don't have to update every use of it.


The 3g2 tag should also be listed, right?
https://github.com/UniversalMediaServer/UniversalMediaServer/blob/master/src/main/java/net/pms/configuration/FormatConfiguration.java#L42


@slipcor :(


All I can think is backport it. Default initialise the parameter to ``false`` so you don't have to update every use of it.


Yes, @bjori opened a pull request for 1.2, which I merged, and I made the same fix here on master.


... with bad indentation.


Whoops :).


fixed


flCluster is still commented out in flags.go


Fixed in https://github.com/mongodb/mongo-c-driver/commit/0bd7850609990e21be2c796b1cb2da1adcf416f1.


In theory yes it's possible but the script isn't that slow and I'm not sure it's really worth it.

If we want to keep coherent output on stdout/stderr (which we should because errors are reported here), we would have to sync the prints and it would basically kill the extra perf we would get (which would be limited by the concurrent disk accesses anyway).


I have tested this with tsc `Version 1.4.1.0`. Imported ts modules are still emitted, sadly. Users will have three options: update tsc to latest, override tsc_args, or bear with generated files.

Otherwise, a workaround has already been tried. https://github.com/scrooloose/syntastic/pull/957

Thanks for your version-aware patches. But I just think this is another issue for TypeScript community which is beyond Syntastic's scope


What's the purpose of having this function?


done


@catsby, it works like a charm now. Thank you!


I don't why there have such error, who can help me. I'm a student from ECNU and now is intern in a advertise company.
[vlionosuser@localhost kaggle-avazu-master]$ ./run.sh x
size = x
make: Entering directory `/home/vlionosuser/chao/kaggle-avazu-master/base/mark/mark1'
g++ -Wall -Wconversion -O3 -fPIC -std=c++0x -march=native -fopenmp  -o mark1 src/train.cpp src/common.cpp src/timer.cpp
In file included from src/train.cpp:6:
src/common.h:1: warning: unknown option after ?pragma GCC diagnostic?kind
In file included from src/common.cpp:5:
src/common.h:1: warning: unknown option after ?pragma GCC diagnostic?kind
src/common.cpp: In function 鈛int32_t<unnamed>::get_nr_line(const std::string&)?
src/common.cpp:17: error: 鈔ullptr?was not declared in this scope
src/common.cpp: In function 鈛int32_t<unnamed>::get_nr_field(const std::string&)?
src/common.cpp:32: error: 鈔ullptr?was not declared in this scope
src/common.cpp: In function 釶roblem read_problem(std::string)?
src/common.cpp:60: error: 鈔ullptr?was not declared in this scope
src/common.cpp: In function 鈌loat predict(const Problem&, Model&, const std::string&)?
src/common.cpp:104: error: 鈔ullptr?was not declared in this scope
make: *** [mark1] Error 1
make: Leaving directory `/home/vlionosuser/chao/kaggle-avazu-master/base/mark/mark1'
Traceback (most recent call last):
  File "./util/gen_data.py", line 5, in <module>
    from common import *
  File "/home/vlionosuser/chao/kaggle-avazu-master/base/util/common.py", line 1
    ../converter/common.py
    ^
SyntaxError: invalid syntax
Traceback (most recent call last):
  File "./run.py", line 18, in <module>
    subprocess.call(cmd.split())
  File "/usr/local/lib/python3.4/subprocess.py", line 537, in call
    with Popen(*popenargs, **kwargs) as p:
  File "/usr/local/lib/python3.4/subprocess.py", line 859, in __init__
    restore_signals, start_new_session)
  File "/usr/local/lib/python3.4/subprocess.py", line 1457, in _execute_child
    raise child_exception_type(errno_num, err_msg)
FileNotFoundError: [Errno 2] No such file or directory: '.util/parallelizer.py'


It's copied and paste. Haven't had time to review it thoroughly, that's why the lack of a PR.


> I guess `scintilla_get_type()` needs to be exported too, since we export/expose the header/GtkWidget for Scintilla, and it's required to subclass the ScintillaObject like my Overview plugin and others […]

It doesn't have to be, but it indeed make some sense to add it (note that it was *not* part of the API before, and so wasn't available on Windows nor before 1.22).

>  (@b4n's Git plugin IIRC?)

Nope, it uses `editor_create_widget()`.

> Anyone care if I do?

Please, go ahead.  Indeed there are legitimate use cases for this, like yours of subclassing, or possibly using the `SCINTILLA`/`SCINTILLA_CLASS`/`IS_SCINTILLA` macros.


I believe so. But need to check.


What's dangerous?


But I don't particularly object to adding Lifecycle in this particular context.


Is this correct? The documentation at https://phaser.io/docs/2.3/PIXI.Graphics.html#lineColor says that lineColor is of type string.


Something in this commit breaks compiling the closure library project.

```
google/closure-library/closure/goog/array/array.js:109: ERROR - assignment to property indexOf of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), T, number=): number
goog.array.indexOf = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:151: ERROR - assignment to property lastIndexOf of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<T>|{length: number}), T, (null|number)=): number
goog.array.lastIndexOf = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:197: ERROR - assignment to property forEach of goog.array
found   : function (?, ?, ?): undefined
required: function ((Array<T>|null|{length: number}), (function (this:S, T, number, ?): ?|null), S=): ?
goog.array.forEach = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:260: ERROR - assignment to property filter of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), (function (this:S, T, number, ?): boolean|null), S=): Array<T>
goog.array.filter = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:301: ERROR - assignment to property map of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<VALUE>|null|{length: number}), function (this:THIS, VALUE, number, ?): RESULT, THIS=): Array<RESULT>
goog.array.map = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:346: ERROR - assignment to property reduce of goog.array
found   : function (?, ?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), function (this:S, R, T, number, ?): R, ?, S=): R
goog.array.reduce = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:391: ERROR - assignment to property reduceRight of goog.array
found   : function (?, ?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), (function (this:S, R, T, number, ?): R|null), ?, S=): R
goog.array.reduceRight = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:427: ERROR - assignment to property some of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), (function (this:S, T, number, ?): boolean|null), S=): boolean
goog.array.some = goog.NATIVE_ARRAY_PROTOTYPES &&
^

google/closure-library/closure/goog/array/array.js:464: ERROR - assignment to property every of goog.array
found   : function (?, ?, ?): ?
required: function ((Array<T>|null|{length: number}), (function (this:S, T, number, ?): boolean|null), S=): boolean
goog.array.every = goog.NATIVE_ARRAY_PROTOTYPES &&
^

9 error(s), 0 warning(s), 98.5% typed
```


Comment doesn't apply: this serializes access to the updateMap. The subsequent node creation is in fact atomic wrt Zookeeper.


This commit seems to cause more idle drain on my configuration. Nothing else changed, just reverted and its back to normal. My normal idle drain is about 0.4-0.6%/h and with this commit added its 0.6-1.0%/h

I don't use mpdecision, running mako_hotplug from franco with all cores online (like mpdec).


Thanks. Don't mind the details (finals, initialization of fields, etc.), this wasn't ever intended to be made a PR in the current state.


Fixed. Pushing out a new release of the agent.


> I have tested this with tsc `Version 1.4.1.0`. Imported ts modules are still emitted, sadly.

I can't reproduce the problem on FreeBSD, OpenBSD, and Linux.  Like I said, it does happen on Windows, cf. #1155.  The patch above is not supposed to address #1155.


Is ok to add this?


Should this really be a dev dependency, or do you plan to have your application depend on semver?


Que bueno!


No, I guess this goes to 'extensions'. Docs need to change.


I sign my comments so that when someone is still confused after the explaining comment, they know who to talk to.
The blame info can be wrong or confusing, sometimes someone changes the indentation of a section, which puts their name in the blame for the section, but they didn't write the comments in it.  So people trying to get more of an explanation have to dig through several revisions or go to the wrong person with their question.  Signing comments makes it easy to know exactly who to go to.
I think signing comments is a good policy to have, and the more I fix broken or misguided code from ages past, the more I find myself explaining why I chose a certain method in the comments.


Thank you, so awesome!


Also wenn ich mich nicht täusche, dann kommt die Meldung immer ganz oben auf dem Screen, wo die einzelnen Schritte angezeigt werden und zwar immer dann, wenn man keine neue Klassenfähigkeit auswählen durfte/musste (also so ungefähr jede zweites Stufe). Bestimmte Klassenfähigkeiten werden ab einer bestimmten Stufe freigeschaltet, man kann aber auch welche auswählen, die schon bei vorangegangenen Stufenaufstiegen freigeschaltet wurden. Ist also imo so in Ordnung und eigentlich sogar schöner als das englische class unlocks, finde ich. Kann aber auchs ein, das ich da was verwechsle.


I don't believe that semver has an `increment` method, which would be undefined. The valid functions are: https://www.npmjs.com/package/semver#functions


Do you really need to pollute your global with semver? Global variable usage should be used with caution and used judiciously.


When you are writing unit tests, the test runner maintains a try catch around your test for you. If your test throws an unexpected exception than the test will fail. So you don't need to do the try catch management yourself.


Hi @mkwork,

with 6dceab583c8ad1d5de3755bc7f67d6fe1f7be8f4, I now implemented four erase functions:

```cpp
iterator erase(iterator pos);
iterator erase(iterator first, iterator last);
const_iterator erase(const_iterator pos);
const_iterator erase(const_iterator first, const_iterator last);
```

They should work as expected.

All the best
Niels


It's certainly not a required policy to explain everything with comments, coding would take too long if it was.  But if you think someone might want to ask you for an explanation some months or years down the line, an explanation might save some time.


In ES6 `var` is no more. Use `let` or `const` instead. See: http://babeljs.io/docs/learn-es6/#let-const


Can I have a look on your test file? It should contain external module like `import externalModule = require('./external.js')`.


Why did you add a newline and tab in your describe suite name?


Yeah, I'd appreciate it if you erased that sequence. Thanks Jason.


Or read the instructions on how this is meant to be used.


When you have a test that you haven't implemented yet mocha provides the `xit` test definition. So you could replace this with `xit('invalid update type');`. When you run mocha it would tell you that the test is a placeholder and output it in a different color.


Just as a note that I tested installing/removing all the drivers in a live media and everything worked fine.


Shit, you're RIGHT.


Don't worry: I've just joked :smile:


Im not sure what most of the other things are supposed to do. so if you
could describe some of the textures and how they work to me i could make
them look better.

On Fri, Apr 10, 2015 at 7:34 PM, Sandra <notifications@github.com> wrote:

> Definitely, those textures are beautiful. OpenAutomation and
> AwesomeSauceCore are the things thatre most popular so focus on those two.
>
> On Sat, 11 Apr 2015 2:36 am AceK7 <notifications@github.com> wrote:
>
> > Would you like some more help with other textures?
> >
> > On Fri, Apr 10, 2015 at 9:06 AM Sandra <notifications@github.com> wrote:
> >
> > > Made new textures the textures. Thanks @AceK7
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > > https://github.com/AwesomeSauceMods/OpenAutomation/commit/
> > > 6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/AwesomeSauceMods/OpenAutomation/commit/6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28#commitcomment-10669718
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AwesomeSauceMods/OpenAutomation/commit/6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28#commitcomment-10675997>
> .
>


@NathanMalo C'est pas une façon automatisée de faire des tests d'afficher du texte. Compare avec le résultat que tu attends avec des assertions !


Fixing #1


[nit] should not it be `sibling is self`?


Lägg till din förklaring (1) som meddelande till denna commit?
> Det gick inte att skapa en tom kontoplan (find(nil) returnerar RecordNotFound).


By initializing this class here you are essentially using a singleton. Which begs the question of whether this needs to be a class if you are only using one function of the class in static scope.


Fixes #34.


For the final PR I'm considering getting rid of the updateMap altogether, and work directly on the node, but that means setting up watches manually. Will see.


Github doesn't support markdown tables?


I agree that this is a bug and needs to be fixed.  My thought is that this should be caught as early as possible since DTSTART is a bogus value (which violates a SHOULD in RFC5545).  We can either sanity check DTSTART at the top of icalrecur_iterator_new() or be a little friendlier (which would allow 19410229 to work) and do something like this (noting that this doesn't fix the problem when compiled with ICU, where checking DTSTART early would):

diff --git a/src/libical/icalrecur.c b/src/libical/icalrecur.c
index 53a62ea..fc2a8e4 100644
--- a/src/libical/icalrecur.c
+++ b/src/libical/icalrecur.c
@@ -2054,8 +2054,8 @@ icalrecur_iterator* icalrecur_iterator_new(struct icalrecu
         struct icaltimetype last = occurrence_as_icaltime(impl, 0);
        icalerror_clear_errno();

-        /* Fail after hitting the year 20000 if no expanded days match */
-       while (last.year < 20000) {
+        /* Fail after exceeding MAX_TIME_T_YEAR if no expanded days match */
+       while (last.year <= MAX_TIME_T_YEAR) {
             expand_year_days(impl, last.year);
             if( icalerrno != ICAL_NO_ERROR) {
                 icalerror_set_errno(ICAL_MALFORMEDDATA_ERROR);
@@ -2105,6 +2105,10 @@ icalrecur_iterator* icalrecur_iterator_new(struct icalrec

     impl->last = occurrence_as_icaltime(impl, 1);

+    if (impl->last.year > MAX_TIME_T_YEAR ) {
+       free(impl);
+       return 0;
+    }

     return impl;
 }


I like this set of unit tests! You may want to also consider testing invalid inputs. Don't forget when writing tests in general to test both happy and sad paths through your code.


Thank you for explaining it to me. Very interesting


this should be removed


that one too


Hi!

Thank you for contributing to ClearOS.

Does this address this issue?
http://tracker.clearfoundation.com/view.php?id=2211

Best regards,

M ;-)


Not really -- I will just ensure the manually installable Firefox version works fine. I am mostly back working on uMatrix for now.


Looks like markdown screws up displaying a diff


That's not right, since the element with which you're trying to align has a margin specified in `em`. In general, `em` is a better unit for designing scalable interfaces.


Okay ça marche !


The css file must be named the same as the theme to be activated by the Kohana theme engine. If the theme folder name is 'terra', the css file must be 'terra.css'. If the theme name is 'ushahidi-terra', the css file must be 'ushahidi-terra.css'.


mm


This is JavaScript and you should use `===`. Make sure you are regularly linting your files, the linter would warn you of such.


remove?


Add Bahasa Indonesia Language!
Thank you :D


method_missing (arBmind) - Win64-Windows-8.1-VisualStudio-12: OK (336 of 336 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg)


Perfecto, luego vi que tenía razón de existir por eso te puse que no me hicieras caso, no lo había visto nunca. Grasies crack!

Enviado desde mi iPhone

> El 10/4/2015, a las 16:51, GpoNsU <notifications@github.com> escribió:
>
> Mira, aquí:
>
> https://github.com/brigade/scss-lint/blob/master/lib/scss_lint/linter/README.md#placeholderinextend
>
> Como ves, se recomienda usar placeholders y no clases cuando hay que hacer @extend.
>
> —
> Reply to this email directly or view it on GitHub.
>


Still not fixed - issue still presents itself as per https://github.com/chrippa/livestreamer/issues/797


Removing now


why don't we just get rid of these memsets since the new packet class constructor 0 fills the data ?


You could try change your local pixi.d.ts to lineColor:any  and then try both "#ffffff" and 0xffffff and see if both work.  If so we can change it to `string | number` on the dev branch.  I do not have a easy test available to try it.


After looking at this some more, I think the only easy way to fix this when using ICU is to reject a bogus DTSTART immediately, otherwise the code can get very complicated.  DTSTART is supposed to be the first instance of the recurrence set and should be a sane value anyways.  Meaning if you want to spit out leap days, DTSTART needs to be a valid leap day, not something like 19410229 or 20150229.  And asking for the 32nd of any month should fail immediately.


Thank you!

But nice for Disable Patch,Any Mod/script to multi player (server)!


> which would be limited by the concurrent disk accesses anyway

...said Underground78 looking at his 15 year old HDD with great care. ;p

> If we want to keep coherent output on stdout/stderr

I pesonally doesn't find our output to be extremally usefull. Only when exceptions happens and this is damn easy to handle without blocking.

And I'm saying form my experience. On my PC sync is heavily bottlenecked by CPU. And I'm not saying it is slow it is just not as fast as it could be. I made changes for my personally entertainment ;) Note Get() call it will get any exceptions and such.

```
PS C:\dev\mpc-hc\src\mpc-hc\mpcresources> Measure-Command { ./sync.bat }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 8
Milliseconds      : 49
Ticks             : 80498232
TotalDays         : 9,316925E-05
TotalHours        : 0,002236062
TotalMinutes      : 0,13416372
TotalSeconds      : 8,0498232
TotalMilliseconds : 8049,8232

PS C:\dev\mpc-hc\src\mpc-hc\mpcresources> Measure-Command { ./sync.bat }

Days              : 0
Hours             : 0
Minutes           : 0
Seconds           : 2
Milliseconds      : 419
Ticks             : 24194393
TotalDays         : 2,80027696759259E-05
TotalHours        : 0,000672066472222222
TotalMinutes      : 0,0403239883333333
TotalSeconds      : 2,4194393
TotalMilliseconds : 2419,4393
```

``` diff
diff --git a/src/mpc-hc/mpcresources/mpc-hc.ms_MY.rc b/src/mpc-hc/mpcresources/mpc-hc.ms_MY.rc
index 681c3e6..aa2d7e0 100644
Binary files a/src/mpc-hc/mpcresources/mpc-hc.ms_MY.rc and b/src/mpc-hc/mpcresources/mpc-hc.ms_MY.rc differ
diff --git a/src/mpc-hc/mpcresources/sync.bat b/src/mpc-hc/mpcresources/sync.bat
index 5c18dd6..91cf177 100755
--- a/src/mpc-hc/mpcresources/sync.bat
+++ b/src/mpc-hc/mpcresources/sync.bat
@@ -26,6 +26,5 @@ IF %ERRORLEVEL% NEQ 0 GOTO END
 python.exe sync.py

 :END
-PAUSE
 ENDLOCAL
 EXIT /B
diff --git a/src/mpc-hc/mpcresources/sync.py b/src/mpc-hc/mpcresources/sync.py
index cf45af7..22e7819 100644
--- a/src/mpc-hc/mpcresources/sync.py
+++ b/src/mpc-hc/mpcresources/sync.py
@@ -19,21 +19,28 @@ import sys
 import os
 import fnmatch

+from multiprocessing import Pool
+
 from UpdatePOT import *
 from UpdatePO import *
 from UpdateRC import *

 if __name__ == '__main__':
+    pool = Pool()
+
     print 'Updating POT file'
     UpdatePOT()
     print '----------------------'

+    results = []
     for file in os.listdir('.'):
         if fnmatch.fnmatch(file, '*.rc'):
-            print file
             file = os.path.splitext(file)[0]
-            print '--> Updating PO files'
-            UpdatePO(file)
-            print '--> Updating RC file'
-            UpdateRC(file, False)
-            print '----------------------'
+            results.append(pool.apply_async(UpdatePO, [file]));
+            results.append(pool.apply_async(UpdateRC, [file, False]));
+
+    pool.close()
+    pool.join()
+
+    for ret in results:
+         ret.get()
```


mikilvægt


When using nodestime=600 you should double the time control and so set 30+0.05


Sorry for the spaces removal, it felt right like that. I'll do the other adaptations. :)


Sleppa því að senda orð á vanteiknara?


Indeed. :)


in testing I was running the report from geowave-test, so this makes sense...nice catch.  I think this worked on travis, no?


This should be `/fixtures/` , and remove the `*`


> It is not the case in this commit but MSVS_VERSION == 2012 would fail when VS2012Express is used.

That's correct but `MSVS_VERSION >= '2012'` would evaluate to `True`.


Example_7.5.c
	三路划分排序算法，他的思想是蒋待排序文件划分为三部分，第一部分是，比划分小的元素，第二部分是和划分元素相等的元素，第三部分是比划分元素大的元素，在排序过程中，扫描左子文件中，与划分元素相等的元素，放到最左面，扫描右子文件，与划分元素相等的元素放在最右边。如图：


This all took way less code than I imagine. Nice worth @laurocaetano and @josevalim !


I have see that


A redux of the awesomeite would be very nice. It is basically a magical
purple metal with a bunch of other metals mixed in. The pure form is the
same without the metals visible. Feel like giving those textures a revamp?
By the way, you can just replace the existing textures when you redo them,
git keeps a history so I can roll back if I don't like it.

On Sat, 11 Apr 2015 11:39 pm AceK7 <notifications@github.com> wrote:

> Im not sure what most of the other things are supposed to do. so if you
> could describe some of the textures and how they work to me i could make
> them look better.
>
> On Fri, Apr 10, 2015 at 7:34 PM, Sandra <notifications@github.com> wrote:
>
> > Definitely, those textures are beautiful. OpenAutomation and
> > AwesomeSauceCore are the things thatre most popular so focus on those
> two.
> >
> > On Sat, 11 Apr 2015 2:36 am AceK7 <notifications@github.com> wrote:
> >
> > > Would you like some more help with other textures?
> > >
> > > On Fri, Apr 10, 2015 at 9:06 AM Sandra <notifications@github.com>
> wrote:
> > >
> > > > Made new textures the textures. Thanks @AceK7
> > > >
> > > > ---
> > > > Reply to this email directly or view it on GitHub:
> > > > https://github.com/AwesomeSauceMods/OpenAutomation/commit/
> > > > 6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/AwesomeSauceMods/OpenAutomation/commit/6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28#commitcomment-10669718
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/AwesomeSauceMods/OpenAutomation/commit/6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28#commitcomment-10675997
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AwesomeSauceMods/OpenAutomation/commit/6431d01a0d83b53038b8e5d7a8bc7f6e2cc98b28#commitcomment-10677989>
> .
>


@bebraw According to w3c:s feed validation that shouldn't be a problem. There were a couple of other things missing though. With the last commit my test feed passes the validation though.


Related issue: #48


It looks like you're using tabs instead of spaces. Please change your formatting settings to spaces instead of tabs for XML and enable "show white spaces" for tabs and trailing spaces in eclipse.


Related issue: #49


There's a number of extra line spaces. It's fine to space out things that make sense, but try to avoid it where it doesn't.


if 64-bit git is found, free the buffer and return success immediately, do not try the 32-bit path.


I'll remove this on Monday, thanks!


I did but initially it didn't work! I've realised since that boolean values in Python are `True` and `False` right? Not `true` and `false`, which is what I tried.

I'll update the code


It seems that eclipse has changed my spaces into tabs when you formatted the code, see my suggestion in the pom.xml


This also applies to the `HKEY_CURRENT_USER, REG_MSYSGIT_INSTALL_LOCAL` case?!


I don't want an Ord constraint on empty or on singleton :-1:

Coherence is useful but this is not an acceptable solution. It doesn't actually guarantee that Sets are created with coherent classes, it artificially limits the usefulness of Set, reduces parametricity and won't work for things past Set. We need an actual solution.


These should need to be final. We are only going to assign them once. Also, per java's style please rename these to ALL_CAPS_WITH_UNDERSCORES. Constants (static final) are formatted that way.


@shabanovd They were added to disambiguate the existing messages printed to `System.out`. Previously you would just see messages like 'shutting down the database...' which is quite worrying. These put them in context when working in embedded mode.


Yes, once found, return immediately, do not attempt next


See http://www.valadoc.org/#!api=gtk+-3.0/Gtk.Window.set_transient_for


@kbrock I wrote a little test script and I don't see much of a difference with 100,000 objects in the array.  I wouldn't expect anywhere near that number in any of these arrays.  Here's the test:

```ruby
class Foo
  attr_accessor :id
  def initialize(int)
    @id = int
  end
end

require 'memory_profiler'
report = MemoryProfiler.report do
  (0..100000).collect { |i| Foo.new(i) }.compact!
end
report.pretty_print
```

Results with compact:
```
Total allocated 100003
Total retained 3

allocated memory by gem
-----------------------------------
   5821680  other

allocated memory by file
-----------------------------------
   5821680  compact_test.rb

allocated memory by location
-----------------------------------
   5821680  compact_test.rb:10

allocated objects by gem
-----------------------------------
    100003  other

allocated objects by file
-----------------------------------
    100003  compact_test.rb

allocated objects by location
-----------------------------------
    100003  compact_test.rb:10

retained memory by gem
-----------------------------------
       120  other

retained memory by file
-----------------------------------
       120  compact_test.rb

retained memory by location
-----------------------------------
       120  compact_test.rb:10

retained objects by gem
-----------------------------------
         3  other

retained objects by file
-----------------------------------
         3  compact_test.rb

retained objects by location
-----------------------------------
         3  compact_test.rb:10

Allocated String Report
-----------------------------------

Retained String Report
-----------------------------------
```

Results with compact!:
```
Total allocated 100002
Total retained 3

allocated memory by gem
-----------------------------------
   5021632  other

allocated memory by file
-----------------------------------
   5021632  compact_test.rb

allocated memory by location
-----------------------------------
   5021632  compact_test.rb:10

allocated objects by gem
-----------------------------------
    100002  other

allocated objects by file
-----------------------------------
    100002  compact_test.rb

allocated objects by location
-----------------------------------
    100002  compact_test.rb:10

retained memory by gem
-----------------------------------
       120  other

retained memory by file
-----------------------------------
       120  compact_test.rb

retained memory by location
-----------------------------------
       120  compact_test.rb:10

retained objects by gem
-----------------------------------
         3  other

retained objects by file
-----------------------------------
         3  compact_test.rb

retained objects by location
-----------------------------------
         3  compact_test.rb:10

Allocated String Report
-----------------------------------

Retained String Report
-----------------------------------
```


This is kind of the point of this afaik. Either the image can be found by the assetmanager and a proper TextureKey with the path is used or the texture is embedded in the j3o, e.g. for generated textures, textures that are packed in the blend file etc.


What the? Don't tell me you plan to create one method for every packet...


awesome, thanks for the write up!


@bcbroussard what's the long term fix for this issue?


You don't need to (read: I prefer that you wouldn't) add additional javadoc to an overridden method unless it truly adds value. The IDE and javadoc compiler will display the interface's javadoc for overridden elements. :)


I am unsure about if its a good idea just to add a "logo" after the sitetitle? I know that the logo but wrong capitalization could be a problem? Maybe make it configurable?


removed


I think we don't need one class for each packet anymore, since the main purpose of a class is to create a structure of multiple fields, and we don't need that in this case (since we already have `InternalResponse` with that purpose)


Any time a variable at any scope will be assigned exactly once, it needs to be marked final. This helps prevent programming mistakes as well as potentially give hints to the compiler to optimize this logic.


이제는 PC 버킷까지...ㅂㄷ


Its just a pretty cool question for a prospective employer to ask.  And I've seen much worse code than this...


@webian please see my note


This could improve synchronization, the application still needs to get informed when changes are made, though.


初めて知りました。便利ですね。
console.logも同様にしたいですね。


@ash47 Ash, I think reincarnation can get off this list.
It is not a global ability, and it is counterable with many tactics + straight hands, so no necessity to ban it outright.
After all, it is fun, like a WKing refresher.


Begini saja pak:
```
               '--debootstrapopts', '--debbuildopts -I -i -j9', '--keyring=%s' % self.keyring]

```


I know ... but then I did revert this commit ...


Well, I did not benchmark every single change change one by one but rather the local branch that I had.
Shall I revert the changes on _settings->experimental and _settings->inconclusive ?


@DeSerg
Не хватает комментариев к классу и методам. Можешь посмотреть, как я комментировал другие помощники представления.


@livingincode done with my review. checkpoint complete.


do i need any type of fix? Thanks for this :D


Sorry, but :-1: from me as well.


I don't think so ;)  We have several subdirs inside `fixtures/`. Hence the `*` wildcard:

![test_fixtures_dirs](https://cloud.githubusercontent.com/assets/73419/7101715/989b7f96-e067-11e4-9a25-890df1b09292.png)


@danicampora now long filenames only cost you 2k :)


I’ve just discovered one aspect where I didn’t quite think this through: `see` is a `SequencedAnnotation`, meaning that
```ceylon
see (`class A`)
see (`class B`)
void f1() {}
see (`class A`, `class B`)
void f2() {}
```
should be equivalent. (Currently, `ceylon doc` seems to only read the first annotation, but never mind that right now.) However, under the system outlined in the documentation, you would get

> `f1`
> See also:
> - `A`
> - `B`
>
> `f2`
> See also:
> - `A`, `B`

I need to consider whether we can “merge” multiple `see` annotations into one. (Preferably without a special case for `see`, but perhaps in general for annotations with one sequential argument.)


can use guard?

> Further, if–let now also supports a single leading boolean condition along with optional binding let clauses. For example:
```
if someValue > 42 && someOtherThing < 19,          let a = getOptionalThing() where a > someValue {
}
```

https://developer.apple.com/library/ios/releasenotes/DeveloperTools/RN-Xcode/Chapters/Introduction.html#//apple_ref/doc/uid/TP40001051-CH1


missing doctype http://www.w3schools.com/tags/tag_DOCTYPE.asp


The MediaWiki coding convention is to use tabs rather than spaces


be aware that html is not xml. In XML i.e. MathML all tags must be closed. For example <br/> would be the equivalent XHTML representation


console.tableとかも感動しました


@offlinehacker: How do you use this when the dpl4hydra.sh script doesn't work? Do you get password lists elsewhere?


Can you do that with a PHP loop?


I think it's right to handle the error at this level (because it could be you the one causing it - as it was my case). However, I don't feel it's the best way to go since here 2 different implementation patterns are being used: callbacks + events. I'd rather have everything unified to make it more standard. Maybe the mq library makes this hard, but I'd expect to have something like:
```
broker.subscribe('s1', optional_config)
  .on('message', handler)
  .on('error', errorHandler);
```
Because a new message in rabbit could also be consider an event easily.

Alternatively, this also could be done:
```
broker.subscribe('s1', optional_config, handler, errorHandler);
```
This second interface standardization example would allow you to do things like:
broker.subscribe.apply(null, myparams), which at the moment is not possible.


Tidak bisa seperti itu pak, saat build paket error
E: Unknown option [--keyring=/home/dotovr/pabrik/irgsh-node/data/blankon-archive-keyring.gpg] was specified

Jadi saat build paket, tidak perlu lagi ngecek manual keyring, sudah ada dalam base.tar.gz pak


Tab spaces.


@DeSerg
Достаточно сохранять просто номер (индекс) нужной формы: 0, 1 или 2.
А в конце уже возвращать return sprintf($wordForm[$i], $number);


I don't just make these things up.


Проба


What if `person` is `null`? A `NullPointerException` will be thrown here. If we're going to throw an NPE (`NullPointerException`) because `person` was `null`, let's throw it at the beginning of the method with a useful error message like, "The parameter person is null."

The reason for doing this is that we fail fast before we do a bunch of work for no reason. We're also able to add better context in the stack trace as to why we failed. And the developer debugging doesn't have to trace through a bunch of code to find the NPE.

I recommend that we use Guava's Preconditions class to do this. See [this page](http://docs.guava-libraries.googlecode.com/git-history/release/javadoc/com/google/common/base/Preconditions.html#checkNotNull(T))


proc?


This one's my fault. I had the method signature wrong. I have pushed the update to master. So if you rebase your code, you should see the correct method signature.


> Or read the instructions on how this is meant to be used.

I don't want to use it your way as I want to work with current bleed on it, not a fixed revision (however I see the benefits for you there) of a second OpenRA git clone.


In general this should probably be moved out of TShock.CS and put somewhere else.


@seebi
Do you agree to revert this?

@bukzor
Could you provide a PR that provides this compatibility for older tmuxen?


Why did you add the UID argument to this method? Just call `person.getUID()` to get the UID.


Quality takes time - well, or something like that ^^


Be aware that the $letter could be anything.
Can you try to hack your own server using "curl -d 'letterDrop=...' http://yourserver.com

http://www.mediawiki.org/wiki/Security_for_developers#Cross-site_scripting


Hard coded because reasons?


Why do you need to do this?
At least a comment would be helpful.


What kind of format tool are you using? or just by hand?


compressed is no more used, compil failed due to unused variable


Now, is this kernel actually boot or functional? Haha


In this case just by hand, but I use the 'AutoPEP8' package for Sublime Text quite a lot.


Have a look at http://php.net/manual/de/function.preg-replace.php and see if you can simplify your code


Why do you call it?   `sublime.set_timeout(init, 200)` do the some thing when sublime starts.


personsByUID is actually a constant? I thought it was a dynamic array (well, a hash map, but we'll keep adding stuff to it in any case...)


Yeah, the default formatter uses tabs and somehow I didn't catch that. Fixing it now.


Haha I've no idea I thought the same thing, but it was like that when you sent the pull request.


Why is it not a config option for each behavior?


(ノ≥∇≤)ノ


`'''` seems to warning in some linter


fix


I totally agree it. In this case, it's really enough. Fixed.


I am +1 based on comments by @guersam in https://github.com/debasishg/scala-redis-nb/issues/109#issuecomment-76605766 .. will merge if no one has any objection.


thanks


I've updated the method signature to this. If the person doesn't exist, return an empty `Optional` of person. I'd recommend making this easy and just using `Optional.ofNullable` in conjunction with the result of the `get` operation of personsByUID.


Oops, this is fixed. Thank you :-)


If this magic number was documented that would be cool.


Public?


On Sat, Apr 11, 2015 at 10:39 AM, Daniele Medri
<notifications@github.com> wrote:
>
> initialize_srand() need some spaces on #ifdef-#endif (were tabs?)

I don't understand what are you referring to, could you elaborate?

>
> —
> Reply to this email directly or view it on GitHub.


In python 2, `def plugin_loaded():` maybe not work. so call it manually.


Ugh for some reason I thought that was a valid convention, but [I'm wrong](https://www.python.org/dev/peps/pep-0257/#multi-line-docstrings). I'll change them all back, thanks.


I'm not sure that this really solves the problem, even in the short term. For example, on my current save, my next Steamworks costs ~335k steel.


@makash Instead of the iframe, I'd suggest please get a plaintext copy of the default email body from the website, and add it in a textarea.
@karthikb351 Is the content in response.html the latest?


To implement this, you'll probably need another map with the person's metadata in it. So every time a person is updated or stored, you would set its metadata's values and store into that map.


In sublime 3, you can just delete it.


AFAICT this formula should not be using a plist/launchd at all.  All the user needs to do is put `use-standard-socket` inside `~/.gnupg/gpg-agent.conf`.  Not even MacGPG uses the launchd trick anymore.


Ok, great!


Maybe you are right but help me with the logic.
Doing osd pool stats give this output:
pool cephfs_data id 1
  -768/9 objects degraded (-8533.333%)
  recovery io 18846 B/s, 1 objects/s
  client io 15356 B/s wr, 2 op/s

pool cephfs_metadata id 2
  -1/0 objects degraded (-inf%)

Looking at PGMap.cc it includes PGMap.h which includes osd_types.h.
At line 1218:
int64_t num_objects_degraded

Now looking through the code num_objects_degraded is in all kinds of places.
Somtimes it's uint64_t somtimes it int64_t.

So it's either the conversion from uint64_t to Double that cause the
negative value to appear or it's not uint64_t and int64_t instead.

Am I missing something?

On Sat, Apr 11, 2015 at 6:13 AM, Danny Al-Gaaf <notifications@github.com>
wrote:

> What is this change about? This line makes no sense at all since the first
> part is already checking what your newly added part checks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/almightybeeij/ceph/commit/7b7c85671207dc99d76115f817e03811915ce1e8#commitcomment-10677578>
> .
>


This commit was ages ago before I learned about the sub protocol system.

On Saturday, April 11, 2015, PEMapModder <notifications@github.com> wrote:

> What the? Don't tell me you plan to create one method for every packet...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/c638ec03464e0beaf5e627ce7c09c1005a594511#commitcomment-10678123>
> .
>

--


Unfortunately this is now happening in several gems. Same with the duplicate `rspec` ... what's the deal with `:test` does it *also* load the `:development environment?


Avoid unnecessary prefix. Java doesn't have out parameters, so there's no need to specify that an argument is an 'in' value or anything like that.


So you are suggesting a big packet translator class?

On Saturday, April 11, 2015, PEMapModder <notifications@github.com> wrote:

> I think we don't need one class for each packet anymore, since the main
> purpose of a class is to create a structure of multiple fields, and we
> don't need that in this case (since we already have InternalResponse with
> that purpose)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/99bc16557c4909d8a57ed6f66b597cb5fd381398#commitcomment-10678133>
> .
>

--


Please remove this - only include generic ignores that everybody needs.


This binary does not support python2, just delete `is_python3 = sys.version_info[0] > 2` and `init()`.


Scope for the library needs to remain _provided_. The library is to be included into other Hadoop projects and does not pull all of its other dependencies.


Same here.


@DeSerg
Всего в результате проверяется 3 условия, и вторая форма $wordForm[2] выбирается в двух разных местах. Можно обойтись проверкой всего двух условий.


*bugfix :P


And another.


good


Document your setters. Good documentation for setters is that it just repeats the getter's documentation. Avoid starting off your documentation with the word set. So in this case, something like this:

```java
/**
 * The optional prefix of the {@code Name} such as Dr., Mr., Mrs., etc...
 * @param inPrefix The name prefix.
 */
```


SSL 中间人重放攻击


Sorry


here's a comment


that's exactly what I meant. Great!


Yeah. Do you think it's worth supporting Python 2? It would just be another set of binaries, not that much effort.


See comment above in the mock implementation.


Not sure :P


I copy from emmet and not think much about it.


The code still exists, I am not sure if I should remove it.


If I rebase (this is on my master) and push --force, this pull request would be destroyed, right? I think I should just merge in your changes and put a new commit with my changes on top of this commit.


I will release a new minor version of pyvisa next week so we can refer to a specific version.


There's a lot that is right in this pull request. It looks like the style is mostly right and you understood what I was wanting. Awesome job!

I've commented on a number of issues that I found which is about what I expected from someone trying to transition to Java. I'm less worried about formatting today since we don't have a lot of time, so please correct the coding and style issues and make another pull request. I haven't pointed out every infraction, but just the first time I've noticed one. So if I make a suggestion in one line of code, please be sure to fix similar issues in other places.

I've also updated some of the interfaces where I've noticed that the code wasn't clear enough or the signature was wrong. Please rebase your code with my changes before making another pull request.

Again, great job. I'm looking forward to seeing an updated pull request.

Thanks!

JT


The texture can be loaded by the assetmanager, it has a proper texturekey it is not generated.

-> This is simply a bug to to not properly having the texturekey set on the restored from cache textures.

The storage system behaves as it should of course it needs to embedd non key textures, but it should not be without a key in the first place if it is loaded from a real texture yes?


@miteshashar not sure why put it in a text area? I am including another file, so whatever is updated in response.html will show up here.

@karthikb351 there was an image which was erroring out, so i commented that. Nothing else showing in the diff.


If this really does try to handle a piece of unicode, we should consider having it in uPy code too ;-)


kkj'


I already have taken a look at it. The problem in fact that in one case int64_t is used in osd_types.h. I have a patch to change it all to use uint64_t all the time, but wait to get all checks trough.

Anyway. Your patch should only be "if (delta_sum.stats.sum.num_objects_degraded > 0)" to make sense.


It certainly would be for better. I'll push a fix shortly.


NIT: customScriptExtensionVersion


I don't know if it's translating unicode code points or some extended ascii page.  All I did was translate the code and verified that it gave the same output for the same input :)


@codebrainz doesn't that use `exec_path` uninitialized? If I'm not mistaken you'd have to set `argv[0] = exec_path` right before `utils_spawn_async()`.

> Not sure if it's worth the commit noise for such a trivial/cosmetic change.

I don't think it is, unless you really prefer it, in which case feel free to play with it (so long as the replacement works :)


What's strange is, there are plenty of Google projects that use the compiler at head, and depend on goog.array, but did not break. Yet when compiling Closure by itself, you get these errors.


Убери пустые строки


Выровняй


Вынеси это в отдельный класс
.photoIcon,.likesIcon,.messageIcon{
...
}


Сдвинь на один таб


Перенеси <img /> на новую строку


@makash reduces a HTTP request to the server. All data delivered in one request.


Extra comments


You can, I will once I get access to a computer. But you will be sleeping
at that time :/.

On Saturday, April 11, 2015, PEMapModder <notifications@github.com> wrote:

> The code still exists, I am not sure if I should remove it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/c638ec03464e0beaf5e627ce7c09c1005a594511#commitcomment-10678310>
> .
>

--


this path refers to a directory in the Variations Template theme.  Probably better to refer to a directory in the variation directory


GREEK SMALL LETTER ALPHA' (U+03B1)
CYRILLIC SMALL LETTER A' (U+0430)

I doubt this handles all Unicode scripts (I read how re2 engine optimized unicode lookup tables, and it's still tens Ks), but if it handles bunch of basic plane ones, it's already next big step for uPy unicode.


Due to 01df670a98cc243e5e9042cc761dd85e623a51fb


Looks perfect to me.

Can you do a follow-up commit to rename scrobble.py to lastfm.py?


Yes, i used it once for recovering bitcoin wallet password and i used my
password generator.
On Apr 11, 2015 4:31 PM, "Bjørn Forsman" <notifications@github.com> wrote:

> @offlinehacker <https://github.com/offlinehacker>: How do you use this
> when the dpl4hydra.sh script doesn't work? Do you get password lists
> elsewhere?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/1a87e651e9ea964b6fa1428619be349d3a1dacd6#commitcomment-10678176>
> .
>


I used "git cherry-pick" way to do that and I didn't test it. Please notice if you see any problems.


Yes but everytime response.html changes, this file will need to be updated. Won't it?


close #52


I suggest refactring your recipe to make use of the API I just added for resolving internal links, that way it is less likely to break in the future: https://github.com/kovidgoyal/calibre/commit/1a2814494e1f5aa6e79a593b8bd5ccca6fdea099


And https://github.com/kovidgoyal/calibre/commit/16efc963e3bef0561b83db9513eb35357eefc698


Interesting. I haven't noticed anything odd but I haven't very closely observed it.

I will check it out,  thanks.


Gonna do:

[Description("Enables log splitting upon file size.")]
public bool SplitLogUponSize = true;

[Description("Amount of megabytes to split logs upon.")]
public int LogSplitSize = 100;

[Description("Splits log after each passing day.")]
public bool SplitLogUponDay = true;

Those config names are fine? Coz can't think of any better right now.


You mean the part where it checks for existing files and adds to FileCount for file numbering, and names the log file?
Not sure where else I should put it.  I saw the try catch where it sets the log file name, so I thought I could put it there, since I define filename there too.
So yeah, where should it be moved?


Maybe its something I've added, who knows. I've already reverted some sched stuff which i never used  - intels patchset for a more fair scheduling https://github.com/hellsgod/hells-Core-N6/commit/fb0a35cab9955f25536353330efa7786c5b72e57 - Its for CFS anyway, so not needed with BFS.


Ok, I can rebase on top of your stuff on another branch. That may get messy with my master, but I can sort out that mess later. I'll try to get you something by noon.


Note that this is only for OSX, \ is correct for Windows


I didnt hard code it on purpose, I  just wanted yyyy-MM-dd-0 for log splitting default.
Or I could leave out file numbering and just do yyyy-MM-dd_HH-mm-ss as its tshock default.
Opinions?


To be able to clean the event message queue when "leaving" either the game or editor in the case the player starts a new game/edition without leaving the app.


thats LogSplitSize * 1mb.  Thought its reasonable to do megabytes.(just reminded me to not allow LogSplitSize to be 0.)
Where do I document it tho?


Yeahh.. *whoooshh* ->  Private :)


Both work.


TenPlus1 fixed this issue one hour after your fix Mg, can you revert your changes, i will merge our code with TenPlus1... :/


I'm sorry for all the trouble with my additions.. was a littlebit too confident of its simplicity, so that i made it in the git-editor. won`t happen again.


Why not this?
```
switch(Random::Int(1, 4))
```


As @Bertram25 said, it's a matter of taste. I think the changes were good and pretty readable. The biggest drawback, IMHO, being that it would not make straight forward so search for "MeshName2" and seing where it is used


i see this in logs after long high temp cpu 100% test.

<6>[  932.901706 / 04-11 18:20:50.374] msm_thermal:do_freq_control Unable to update policy for cpu:2
<6>[  932.901769 / 04-11 18:20:50.374] msm_thermal:do_freq_control Unable to update policy for cpu:3
<6>[  933.152397 / 04-11 18:20:50.624] msm_thermal:do_freq_control Unable to update policy for cpu:2
<6>[  933.152438 / 04-11 18:20:50.624] msm_thermal:do_freq_control Unable to update policy for cpu:3
<6>[  933.401342 / 04-11 18:20:50.874] msm_thermal:do_freq_control Unable to update policy for cpu:2
<6>[  933.401402 / 04-11 18:20:50.874] msm_thermal:do_freq_control Unable to update policy for cpu:3
<6>[  933.651219 / 04-11 18:20:51.124] msm_thermal:do_freq_control Unable to update policy for cpu:1
<6>[  933.651281 / 04-11 18:20:51.124] msm_thermal:do_freq_control Unable to update policy for cpu:2
<6>[  933.651328 / 04-11 18:20:51.124] msm_thermal:do_freq_control Unable to update policy for cpu:3

what do you think?
driver work just fine, temp is controlled by freq reduce as should. freq is restored to max after temp is reduced.
so why i see this?


It's certainly a nice thing not messing with a bunch of huge patches for fairness and new scheduling policies,  then tuning sysctl to try and get better cpu performance/interactivity. BFS seems to just do everything better.  😃


Thank you very much, povmaniaco!! :-)

I've included your fix and some other changes in a Windows 64bit experimental version, just in case you want to take a look. Please see: http://www.yafaray.org/community/forum/viewtopic.php?p=31038#p31038


Use `sqrt(.5) / 2.` instead of `.5` as the length and width of the detector, to get the correct detection area.
x columns are indexed and should (in theory) be faster when using `read_where`, so use `x` for `read_where`, `y` for `compress`.


No. It's the == of comparablemixin only looks at the compareattrs


Vi skal ikke hente portnummer fra en delt config først som sist? Iallefall en variabel?


Yep, I also thought about that. Thats why I began to revert some stuff. Its just a local WIP repo to mess with stuff :smile:


Because I didn't know I could do that :-D
I'll add a patch.


1591_AttributesInArguments (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


I wondered about that, thanks for that info @chdiza -- I unloaded gpg-agent and added use-standard-socket to gpg-agent.conf and everything is working as I expect now.


i am thinking that its can be related to max freq lock on 2.5ghz and allowed max freq is 2.8ghz
so driver tries to set MAX allowed freq when temp is low. but cant do that. as freq is locked to 2.5ghz
by freq_limit driver.

can be that? or it's tries to set freq that is not even in the table of freqs...?


Indeed, that's also not what I had in mind when talking about a machine state. I was more thinking about calling deactivate when the mode had the "focus" and deactivate when it is lost. For such a use, doing what you do in the deactivate function in ~GameEditorModeBase would be the same


Yeah.  I added it in, squashed it, ran it, and looked at the first result:
https://travis-ci.org/ngageoint/geowave/jobs/58040829#L2616

Didn't work there, so I reverted it back.  Doh.

Should have looked a bit harder - it seems the Accumulo 1.5.x versions tend to have errors there also; I'm blaming it on the munge profile.


Good stuff @tahersajjad Let's update the latest on here every week.


The indentation is still incorrect, lol.

In Visual Studio, you can highlight that section of code and go to `Edit --> Advanced --> Format Selection`, and it will automatically fix it.

Alternatively, go to `Edit --> Find and Replace --> Quick Replace`, and replace all spaces with tabs in that file _(you'll need to copy+paste a 'tab' character, since the 'tab' key moves between edit-boxes)_.

To automatically have it do the right thing in the future, `Tools --> Options --> Text Editor --> C# --> Tabs`, set `indenting = smart` and `tab = 4, insert spaces`.  These should be the default options..


Yesh! Lagde en issue på det. Problemet er at om porten er opptatt tar browsersync automatisk å bytter port : / Så egentlig ganske lite kontroll på hvilken port som blir brukt.. Så må finne en smooth måte å gjøre det på. evt. window.locaiton.port osv..


this line causes encode not to work at all


Fair enough. I did not want to change int64_t to uint64_t since I did not
know if there was any code that expected a signed integer. You obviously
have had more time reading the code base so I will trust your judgment.
Since uint64_t will fix the bug I will go ahead and rescind my patch.

On Sat, Apr 11, 2015 at 10:05 AM, Danny Al-Gaaf <notifications@github.com>
wrote:

> I already have taken a look at it. The problem in fact that in one case
> int64_t is used in osd_types.h. I have a patch to change it all to use
> uint64_t all the time, but wait to get all checks trough.
>
> Anyway. Your patch should only be "if
> (delta_sum.stats.sum.num_objects_degraded > 0)" to make sense.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/almightybeeij/ceph/commit/7b7c85671207dc99d76115f817e03811915ce1e8#commitcomment-10678332>
> .
>


1591_AttributesInArguments (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


Are we sure we want to remove perl? When I run `apt-get remove perl` on my Ubuntu system, I get:
````
The following packages will be REMOVED:
  account-plugin-salut anacron apache2 apache2-bin at autoconf automake
  avahi-daemon avahi-utils brltty build-essential cups-browsed cups-bsd
  dblatex debhelper debmirror devscripts dh-autoreconf doc-base docbook
  docbook-dsssl docbook-utils docbook-xml dpkg-dev dpkg-repack
  foomatic-db-engine gem2deb git git-core gitk gobject-introspection
  gstreamer0.10-plugins-base-apps hardening-includes icoutils
  init-system-helpers intltool intltool-debian jadetex kde-runtime
  kde-runtime-data kde-style-oxygen kde-window-manager
  kde-window-manager-common kdelibs5-data kdelibs5-plugins kdoctools
  kernel-wedge kubuntu-debug-installer latex-beamer latex-xcolor
  libalgorithm-c3-perl libalgorithm-diff-perl libalgorithm-diff-xs-perl
  libalgorithm-merge-perl libany-moose-perl libapache2-mod-auth-openid
  libapache2-mod-auth-radius libapache2-mod-php5 libarchive-extract-perl
  libarchive-zip-perl libauthen-sasl-perl libautodie-perl
  libb-hooks-endofscope-perl libb-hooks-op-check-perl
  libbareword-filehandles-perl libcairo-gobject-perl libcairo-perl
  libclass-accessor-perl libclass-c3-perl libclass-c3-xs-perl
  libclass-load-perl libclass-load-xs-perl libclass-method-modifiers-perl
  libclass-tiny-perl libclass-xsaccessor-perl libclone-perl
  libcommon-sense-perl libconfig-general-perl libcrypt-passwdmd5-perl
  libdata-optlist-perl libdata-perl-perl libdata-record-perl libdbd-mysql-perl
  libdbi-perl libdevel-caller-perl libdevel-globaldestruction-perl
  libdevel-lexalias-perl libdevel-partialdump-perl libdevel-symdump-perl
  libdigest-hmac-perl libdpkg-perl libemail-valid-perl libencode-locale-perl
  liberror-perl libeval-closure-perl libexporter-lite-perl
  libextutils-depends-perl libextutils-pkgconfig-perl libfile-basedir-perl
  libfile-copy-recursive-perl libfile-desktopentry-perl libfile-fcntllock-perl
  libfile-homedir-perl libfile-listing-perl libfile-mimeinfo-perl
  libfile-spec-perl libfile-which-perl libfont-afm-perl
  libgetopt-long-descriptive-perl libgirepository1.0-dev
  libglib-object-introspection-perl libglib-perl libgnupg-interface-perl
  libgtk2-perl libgtk3-perl libguestfs-dev libguestfs-perl libguestfs-tools
  libguestfs0 libhtml-form-perl libhtml-format-perl libhtml-parser-perl
  libhtml-tagset-perl libhtml-template-perl libhtml-tree-perl
  libhttp-cookies-perl libhttp-daemon-perl libhttp-date-perl
  libhttp-message-perl libhttp-negotiate-perl libimport-into-perl
  libindirect-perl libintl-perl libio-html-perl libio-pty-perl
  libio-socket-inet6-perl libio-socket-ssl-perl libio-string-perl
  libio-stringy-perl libipc-run-perl libipc-system-simple-perl libjson-perl
  libjson-xs-perl liblexical-sealrequirehints-perl liblist-moreutils-perl
  liblockfile-simple-perl liblog-agent-perl liblog-message-simple-perl
  liblwp-mediatypes-perl liblwp-protocol-https-perl libmail-sendmail-perl
  libmailtools-perl libmodule-implementation-perl libmodule-pluggable-perl
  libmodule-runtime-perl libmoo-perl libmoose-perl libmoox-handlesvia-perl
  libmoox-late-perl libmoox-options-perl libmouse-perl libmro-compat-perl
  libmultidimensional-perl libnamespace-autoclean-perl libnamespace-clean-perl
  libnet-daemon-perl libnet-dbus-glib-perl libnet-dbus-perl libnet-dns-perl
  libnet-domain-tld-perl libnet-http-perl libnet-inet6glue-perl libnet-ip-perl
  libnet-smtp-ssl-perl libnet-ssleay-perl libnss-mdns
  libpackage-deprecationmanager-perl libpackage-stash-perl
  libpackage-stash-xs-perl libpadwalker-perl libpango-perl
  libparams-classify-perl libparams-util-perl libparams-validate-perl
  libparse-debianchangelog-perl libpath-class-perl libpath-tiny-perl
  libperlio-gzip-perl libplrpc-perl libpod-coverage-perl libpod-latex-perl
  libregexp-common-perl librole-tiny-perl libsgmls-perl libsocket6-perl
  libstrictures-perl libstring-shellquote-perl libsub-exporter-perl
  libsub-exporter-progressive-perl libsub-identify-perl libsub-install-perl
  libsub-name-perl libsub-uplevel-perl libswitch-perl
  libsys-hostname-long-perl libsys-virt-perl libtask-weaken-perl
  libterm-readkey-perl libterm-ui-perl libtest-pod-coverage-perl
  libtest-pod-perl libtext-levenshtein-perl libtext-soundex-perl
  libtie-cache-perl libtie-ixhash-perl libtime-duration-parse-perl
  libtime-duration-perl libtimedate-perl libtry-tiny-perl libtype-tiny-perl
  libtypes-path-tiny-perl libunicode-utf8-perl liburi-perl
  libvariable-magic-perl libwin-hivex-perl libwww-perl libwww-robotrules-perl
  libxml-parser-perl libxml-twig-perl libxml-xpath-perl libyaml-tiny-perl lilo
  lintian makedumpfile memcached mysql-client mysql-client-5.5 mysql-server
  mysql-server-5.5 openssh-server parcimonie patchutils perl perl-modules pgf
  po-debconf prosper ps2eps qapt-batch quilt rsyslog samba sane-utils
  sgml-data sgmlspl telepathy-salut texlive texlive-base texlive-bibtex-extra
  texlive-binaries texlive-extra-utils texlive-font-utils
  texlive-fonts-recommended texlive-fonts-recommended-doc
  texlive-generic-recommended texlive-latex-base texlive-latex-base-doc
  texlive-latex-extra texlive-latex-extra-doc texlive-latex-recommended
  texlive-latex-recommended-doc texlive-luatex texlive-math-extra
  texlive-pictures texlive-pictures-doc texlive-pstricks texlive-pstricks-doc
  tipa ubiquity ubiquity-frontend-kde ubuntu-desktop ubuntu-minimal
  update-inetd xmlto
````


Don't worry my friend about that, it is normal............ when cpu is not online that function can't set freq, and it is right :) Maybe we can just avoid log spam, changing pr_info to pr_debug function :)


The old one did too. I assume they are there to remove information that was
written


Can we have this stuff PRed upstream? Thanks!


Can we have this stuff PRed upstream please! Thanks.


It appears not to work for me.

![admeshgui](https://cloud.githubusercontent.com/assets/2401856/7101994/59382326-e072-11e4-86f0-69ceac5b882f.png)


On Wed, Apr 08, 2015 at 09:10:24PM -0700, Thomas Sibley wrote:
> > Sounds good. Can I make any updates to my commit message to be
> > more convincing?
>
> I think it's fine just to leave it.  I guess it might be useful to
> say something along the lines of what you said below: "not worth
> explaining the difference."

In the UW workshop I did feel like we needed some more context than
“for this lesson they're the same”, so I've rerolled the commit into
61c0a6c3 (04-join.md: Don't explain distinction between WHERE and ON).
Now I say that there is a distinction for outer joins with a link to
some outer-join docs for folks who care, but I say that they are
interchangeable for this lesson, and I don't go into any details on
what the distinction is.


Wow!! good job Damien!


@vinkla Confirmed. :(


Нет необходимости разделять строку конкатенацией.
Всё будет хорошо работать в одной паре кавычек


Looks like the path changed: https://github.com/Homebrew/homebrew-versions/blob/0879dd202b8f23033d507ddea31fbf971cdc808f/bash-completion2.rb#L55-L58 Pushing a fix.


Попробуй ввести url:
index.php?page=something_else
Будет пытаться подключить файл, которого нету - ошибки.


1d6664ec420ff19dae837def525f1b7af1cef8f6 Thanks @vinkla for the heads up!


can you please check if the following patch helpes? https://github.com/ceph/ceph/commit/f185d4326843ae319d4046d3e7f9f0c9d6e4b5c5


No. The new code is not worse than the old, so there would be no benefit in reverting.


On 15-04-11 08:06 AM, Colomban Wendling wrote:
> @codebrainz doesn't that use `exec_path` uninitialized? If I'm not mistaken you'd have to set `argv[0] = exec_path` right before `utils_spawn_async()`.
>

Indeed.

>> Not sure if it's worth the commit noise for such a trivial/cosmetic change.
>
> I don't think it is, unless you really prefer it, in which case feel free to play with it (so long as the replacement works :)
>

I'll just leave it alone, it's fine how it is now.


Sorry, my mistake.
I've restored the tests.


See b1e823d


I thought about this some more yesterday. The reason why this problem exists, is because we're messing with the name instead of copying it from the old template.

Do we really want to display the real attribute name instead of the OID? In every other case we try and make the server output resemble the configuration files. We do that because it's useful for users to be able to follow the request flow through the config files by matching lines of debug to lines of the configuration file. It seems to have been a conscious design decision in other places, so why is this case special?


The reference to the `HashMap` is a constant. In other words, we're not going to new off another instance of a `HashMap` each time. The `HashMap` itself is mutable, but the reference to it is not.


Also, since this is a mock, go ahead and implement a `public static` method to reset the Mock. In other words, it should clear the backing `HashMap`s.


в коменте написанно, что код проверяет на введены ли данные или нет, но где тут в коде проверка на "typed correctly" ?