$1? Also, we aren't passing all necessary cmake options here. Also there is an issue with readline library, it doesn't get bundled in the pkg. And we are building pkg here, not dmg.

Maybe we should provide shell scripts with our sources, like build_tarantool_dmg.sh, and avoid documenting gory details here?


I agree, that would make it much more obvious


OK


@SubJunk I'm not sure what I think is a good solution yet, I'm still studying the DLNA standard. But from what I can see this far, thumbnails should be offered in defined sized that's all square in addition to their "real" unpadded size. Those renderers that cannot handle displaying the images in their real aspect/size, will then pick the square ones if I understand it correctly.

As to blocking 6.0.0, yes and no. Ideally I would wish that 6.0.0 wasn't merged until these things were properly sorted out, and that we could release 5.5.1 or 5.6.0. But that's a bit late now.

I do think we should do a release, there have been several fixes and a huge amount of translations (we now have several complete languages that were merely there in 5.5.0).


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13169](https://teamcity.mesosphere.io/viewLog.html?buildId=115428&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:07:54


Topic 4/Exercise 1/Create Table Assign.txt

sql scripts should be stored in .sql files, using the right extensions allows tools to parse and display the contents with syntax coloring.


@hvelarde need some help here.. if I enable the update_featureimage event it freezes when I click on save button at RF test


can you argue ?


Running as root is unnecessary.


Remember to always use English.


This shouldnt be needed, the delegate forwarder should handle it.


@webyom Yes sure, that would be cleaner.


More fluent, here is the comment from the javadoc:
_Performs a Thread.sleep using this time unit. This is a convenience method that converts time arguments into the form required by the Thread.sleep method._


Это похоже на список, как думаешь?:)


Hi @squaremo, would it be possible to get this released to npm? I'd love to have a warning-free `npm install`! :dancer:


This won't work if a new student enrolls


This can go on one like


Thank you @unho!


local variable declarations for i and dev are missing. (I'm to blame for the i one)

`local varname dev i`
`varname="$1"`


Я бы сделал две колонки
![2016-01-28 03-18-36 barbershop-item-1000 jpg 1000x968](https://cloud.githubusercontent.com/assets/8250032/12641967/0cec6512-c5dd-11e5-840c-ce9e8cae17de.png)


I think it's not more readable because you ask a TimeUnit to have a thread to sleep ...


Это вроде не параграф)


и это тоже)


I try : but actually : don't have save button show : http://250.sondages.pro/index.php/10000 (allow save is Y) . But relmind : for security/accessibility/code calrity : we need action button : We do an action on the DB : then we need a CRSF control. And doing an action is a button (or input type submit, but button is really more cooooool).


Список ссылок?


Please resolve this merge conflict before next commit.


Лучше список


Can we add it back?
![capture d ecran 2016-01-28 a 11 37 12](https://cloud.githubusercontent.com/assets/3165635/12642022/82f90c80-c5b3-11e5-8fbe-56cc82e0306f.png)


Аналогично предыдущему. список


I've updated the changelog with all of the PR's that were merged and issues that were closed since the 2.0.0 release.

@kingosticks, you've been busy.


I can reproduce this issue
1. Byte compile swift-mode
2. Quit Emacs
3. Launch Emacs and load byte-compiled file
4. Load flycheck


@gabrielamatias hyphen should be `–` not -


This is great. Stick the contextual help (?) button on this bar too.


лучше вынести input из label, т.к. потом стилизовать будет проще.
```
<input id="input">
<label for="input>Label</label>
```


Аналогично


Yeah; we have a couple of options I think:

1. create a 'NullClient' in the initialiser that responds to all messages with a 'please connect first' exception
2. raise an error here if client is `nil` and/or `connected? == false`
3. automatically create the client if one of these methods is called

I prefer 1 or 2 I think...


Soll hier wirklich "15 S." erzeugt werden, statt "S. 15", wenn nur die **PageNumber** angegeben ist?


Right.  I didn't read that far.  (-:


Topic 4/Exercise 7/Java JDBC-Database Class.java.txt

This code is copy&pasted from http://www.vogella.com/tutorials/MySQLJava/article.html
Have you compiled and tried this code at least?
If you're interested in a code review please code it yourself.


Can we use `padding` instead?


it looks trivial but actually it is not, as it was tricky to come up with a clean solution like this :-)


es gibt keine crs_utils ohne RefId. Die Ref_id wird jedes mal neu über die ObjectUtils gezogen.
Die Prüfung auf $this->crs_utils sollte ausreichen.

if ($this->crs_utils && $_GET["ref_id"] && $this->crs_utils->getRefId() == $_GET["ref_id"]) {


Typo! Looks like you misspelled "canola"!


It's not generally safe to memcpy from hardware - you're casting around the type error trying to stop you.

You can't guarantee the access size - it might do a 64-bit access that the peripheral doesn't support.


Список?


Список


Аналогично предыдущему


right


아직  코드는 안 적은거야?


files of system


I would also replace the `=>` by a `function` to make it work with IE and Safari.


Hi Dean,

Thanks for this excellent package, it certainly looks like it's going to make my life easier :-)
For your information, I have had some issues due to python 3.5.0. I ran your textract-master through python's 2to3 tool, and changed pdfminer into pdfminer3k in the dependencies. I easy_installed with python 3.5.0 and it works as a charm on docx and pdf files (only tested those). I would be glad to contribute this, but I am a total nitwit when it comes to github. If this is helpful to you, just point me in the right direction on how I can help out.

Thanks again for writing this.
Bernie.


Hi @wdecker @dimpase @fcayre @lawrennd @marcinofulus @pdehaye @ClementPernet @defeo
A little reminder to fill in the Data Management Plan. Thanks in advance for your help!

FYI @nthiery


Proguard changes this line to -

`private List<Object> lions = new ArrayList<Object>();`


May I suggest using protocol-relative URL, i.e. `//fonts.googleapis.com/`, for loading Google Fonts to accommodate HTTPS setups?


Think simple ! :-)
Totally agree with you.

BTW, I take this opportunity on storeTemperature() method for example to talk about a point. Remember that a service method like this should be called by devices only and not from a backend service. I mean ... authentication and profiling features are needed.


uint16_t MAC[3]


1. Separate package(e.g. flycheck-swift)
2. Add flycheck to dependency list as requirement and load it
3. Wrap it with `dont-compile`, however it was deprecated macro since Emacs 24.4


https://github.com/liqd/thentos/blob/e44d138377e60ded961abe9d5a7bab1bd4119b68/docs/dev-howtos.md#source-comment-keywords

I think you want `FIXME`?


I'd clarify this slightly as:
The digitizer and trigger you want to use can be specified (in WCSim.mac), as well as properties of the digitzer and trigger (in daq.mac).


The digitizer and trigger and dark noise are now...


Done.


All of the digitizer- and trigger-specific options related to the DAQ are now included in a separate daq.mac file. This file gets called by WCSim.mac. The choice of digitizer and trigger is made in WCSim.mac


I think we should only have one newline after headers, to make it easier to be consistent. This is already inconsistent in this single commit.


We could be more explicit here, and assert that `vizjson3[:datasource].has_key?('template_name').should be_false`, but it's not a big deal I guess.


Converting to hashes is very good! +1

(I thought it was not possible with ghc in lhs files.)


Instead of 'sendSMS' on the installation object, set 'pushEnabled' on the user object to true or false accordingly. Go ahead and still save the installation object so the localeIdentifier is recorded.


should we generate the man files under ```man/``` as we previously did under ```doc/```?

Since I was already playing around with ```xsltproc``` to do the proper thing, I attach the patch I've used.

```
diff --git a/Makefile-man.am b/Makefile-man.am
index 23057aa..92d6fef 100644
--- a/Makefile-man.am
+++ b/Makefile-man.am
@@ -17,9 +17,14 @@
 # Free Software Foundation, Inc., 59 Temple Place - Suite 330,
 # Boston, MA 02111-1307, USA.

-man1_MANS = ostree.1 ostree-admin-cleanup.1 ostree-admin-config-diff.1 ostree-admin-deploy.1 ostree-admin-init-fs.1 ostree-admin-instutil.1 ostree-admin-os-init.1 ostree-admin-status.1 ostree-admin-set-origin.1 ostree-admin-switch.1 ostree-admin-undeploy.1 ostree-admin-upgrade.1 ostree-admin.1 ostree-cat.1 ostree-checkout.1 ostree-checksum.1 ostree-commit.1 ostree-gpg-sign.1 ostree-config.1 ostree-diff.1 ostree-fsck.1 ostree-init.1 ostree-log.1 ostree-ls.1 ostree-prune.1 ostree-pull-local.1 ostree-pull.1 ostree-refs.1 ostree-remote.1 ostree-reset.1 ostree-rev-parse.1 ostree-show.1 ostree-summary.1 ostree-static-delta.1 ostree-trivial-httpd.1
+man1_files = ostree.1 ostree-admin-cleanup.1 ostree-admin-config-diff.1 ostree-admin-deploy.1 ostree-admin-init-fs.1 ostree-admin-instutil.1 ostree-admin-os-init.1 ostree-admin-status.1 ostree-admin-set-origin.1 ostree-admin-switch.1 ostree-admin-undeploy.1 ostree-admin-upgrade.1 ostree-admin.1 ostree-cat.1 ostree-checkout.1 ostree-checksum.1 ostree-commit.1 ostree-gpg-sign.1 ostree-config.1 ostree-diff.1 ostree-fsck.1 ostree-init.1 ostree-log.1 ostree-ls.1 ostree-prune.1 ostree-pull-local.1 ostree-pull.1 ostree-refs.1 ostree-remote.1 ostree-reset.1 ostree-rev-parse.1 ostree-show.1 ostree-summary.1 ostree-static-delta.1 ostree-trivial-httpd.1

-man5_MANS = ostree.repo.5 ostree.repo-config.5
+man5_files = ostree.repo.5 ostree.repo-config.5
+
+man1_MANS = $(addprefix man/,$(man1_files))
+man5_MANS = $(addprefix man/,$(man5_files))
+
+EXTRA_DIST += $(man1_MANS) $(man5_MANS)

 if ENABLE_XSLTPROC

@@ -31,14 +36,16 @@ XSLTPROC_FLAGS = \
         --stringparam man.authors.section.enabled 0 \
         --stringparam man.copyright.section.enabled 0

+XSLT_STYLESHEET = http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
+
 XSLTPROC_MAN = \
-       $(XSLTPROC) $(XSLTPROC_FLAGS) http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
+       $(XSLTPROC) $(XSLTPROC_FLAGS)

-%.1: man/%.xml
-       $(AM_V_GEN) $(XSLTPROC_MAN) $<
+%.1: %.xml
+       $(AM_V_GEN) $(XSLTPROC_MAN) --output $@ $(XSLT_STYLESHEET) $<

-%.5: man/%.xml
-       $(AM_V_GEN) $(XSLTPROC_MAN) $<
+%.5: %.xml
+       $(AM_V_GEN) $(XSLTPROC_MAN) --output $@ $(XSLT_STYLESHEET) $<

 CLEANFILES += \
        $(man1_MANS) \
```


TeamCity Civic Assembly :: Build backend + frontend [Build 33](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=527&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 22 (2 new), passed: 1718, ignored: 7 Build time: 00:20:39
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1453977853799: <no details avaliable>

Venue tests: Hike trail parking lots type: Should Delete an Hike trail parking lots: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

##### there are 12 more failed tests, see build details
```


Sorry, but I don't know which of your code is used or not... If it isn't used, remove it on your own please.


I think the differentiation is not needed


@rppendya Just an FYI: you should flag that you'll need to revert all the places where you made this change when we sync back to the main downstream OMPI repo. As we discussed over there, this change to the opal_list destructor could not be accepted in OMPI as it broke too many things. So we have to stick with OPAL_LIST_RELEASE/DESTRUCT.


sfm::bundler::Matching::compute() takes a const ViewportList& as the first argument but a non-const reference is required in order to call the methods that compute the cascade hashes. The decision could be made inside compute() but then the reference has to be non-const. Looking at the size of compute(), I decided to move the cascade hash computation into its own function in order not to "pollute" compute() with a non-const reference.


Isn’t `which` unix specific? ([See this SO question](http://stackoverflow.com/questions/304319/is-there-an-equivalent-of-which-on-the-windows-command-line))


I think `ValidatedValueUnwrapper` is actually quite good, or just  `ValueExtractor`. However, the former seems to me more descriptive.


Another solution would have been to add the `PrivilegedIP` principal to the clearance check in `lookupUser_`.  I like yours better, I think.


Тут только для iOS, в андроиде только картинка.


IIRC, [plone.autoform](https://pypi.python.org/pypi/plone.autoform) is the one that is deprecated; @davisagli told me once to use plone.directives.form instead


@greut classe ! Ca me rappelle ça : https://github.com/capistrano/capistrano/issues/413 ?


Алекс, у нас много разных полей, которые по-разному называются в Британии и США, но т.к. у нас нет отдельно локали en_GB, то мы ориентируемся на американский английский.


If you add [`node-which`](https://github.com/npm/node-which) as a dependency this should work cross-platform.


Мне кажется, тут очевидно, что индекс почтовый, а не какой-либо еще.


My favorite Code School path is (HTML & CSS).


> It might actually be better just to issue an error here and forbid these
> additions entirely, but that's a discussion for another day.

Or perhaps to use a `sumfun` representation! ;)


I understand the logic of what I think the code should do, but not what the code actually seems to do!


good job, me!


thank you @JonMcPartland. That's much clearer.


Что это за строчки (эта и следующая)? Если whatsnew, то было бы неплохо добавить соответствующий префикс, как сейчас: whats_new_*


length indices are swapped, repair.


does the returned id have to be globally unique? it's always handled in PlatformManager and stored in database with reference to the platform. Is it sufficient to return a unique string on platform scope?


Префикс "mwm_" не нужен


I feel kinda stalked @yuchi :joy:

I'm just trying to figure out a build error we're having, so these were just meant for me :wink:


Let's take the case where we have specified an L1 in the filter.
In this case L1 is not None
So we return ALL L1s which are children of the same L0...!!?!?!?

If we specify an L2 then we get the same effect: ALL L2s of ALL L1s


После "для того" запятая не нужна


Вероятно, "Для того" вообще тут не нужен.


...there was no mention of a `getPosition` method for the `Circle` class in the docs, so it didn't actually centre when you clicked on it.

In any case, I think we probably need to use Markers, as Circles define areas rather than points, but that can wait until we have a decision on what to use for mapping


И тут аналогично


1. Мы экономим место в гитхабе? Строк немного больше получается, да - но они все переводились не на пустом месте, Игорь отдавал их и подразумевается что где-то они используются. Не добавить их сейчас равносильно ещё одному кругу добавления строчек через некоторое время.
Предлагаю не отвлекать сейчас столько сил на определение “нужны-не нужны строки”. Уверен, что в редакторе все новые строки так или иначе понадобятся.
Плюс, если сейчас их не добавить - совершенно не факт, что у Наташи эти строки где-то сохранятся. Вполне может быть, что как раз если не добавить их сейчас - они могут быть утеряны к следующему разу, когда они действительно понадобятся.
> On Jan 28, 2016, at 1:05 PM, Alex Zolotarev <notifications@github.com> wrote:
>
> Я против добавления не используемых строк. Они никем не ревьюятся и забивают место. А к следующему релизу Наташа заново их переводит, потому что все забыли, что они уже были вставлены в код.
> Где конкретно запланирована эта строка в нашем UI? В каком кейзе в будущем мы ее будем вставлять?
> —
> Reply to this email directly or view it on GitHub <https://github.com/therearesomewhocallmetim/omim/commit/49e8ada922b6628e35c80a68e4a5862143879a6e#commitcomment-15728483>.
>


+Android 只显示的TextView，显示不了的字体缩小显示 ！


@jpgarcia Published in version 1.2.0 and not 1.1.1 due to a new feature addition.


Опечатка


@justinbmeyer I did this while working on https://github.com/stealjs/steal/issues/438

My PR in steal to set `@less-engine` is still open https://github.com/stealjs/steal/pull/507


What are these two files for?


@Robotia Pretty much the next time you'll build this, it'll be build 9 so after you make a release, I recommend creating a commit right after uploading that release increasing the **nextBuildNumber** by one, same with the last number on the end of **version**.


new failure will cause crash...


I thought of doing this as well but what if someone has a folder with a ":" in it? I guess it is extremely unlikely but I was just trying to make sure it was full proof.


meiner....


great! thank you dude!


Scheduled is wrong name here, it suggest a user may provide a schedule for imports which is not true. I'd rather rename this to recurring/periodical/etc.


@j3nsch wenn wie lt. OPUS4-Doku Kap. 13.2 das Feld PageNumber wirklich die **Anzahl der Seiten des Dokuments** enthält, dann, dachte ich, wäre das das richtige Format für die Anzeige... wenn da was anderes drinsteht, natürlich eher nicht.


Agreed, see https://github.com/neo4j/neo4j/commit/0d9b045faa8376bf6db6eca9c06f021824e4a2db


Again, not scheduled, but periodical/recurring.


Fixed #15


"mwm_" - это вообще ни о чём не говорящий префикс. Предлагаю его не использовать.


и возвращает функция не число, а строку


а еще у тебя тут есть параметр


You should definitely read this `secret` in from another file :see_no_evil:


だめですよ。太ってますから、修正してください。


typo, it's "pagination" not "pagenation" (also replace in the rest of the change)


Hi, I'll merge this into ongoing development soon.


这个真得好好查一下资料


prefs_map_autodownload_title. Так как, вероятно, будет ещё и *_subtitle


TeamCity pull requests :: netty [Build 1128](https://garage.netty.io/teamcity/viewLog.html?buildId=5143&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:36:22


no need for a constant here I think, it's not reused so no real benefit


You're 100% right, changed!


same as above, no real need for a constant


This may be a typo :)


good call


There's no large difference between `wiki` and `wikis`. It really just depends on which version is expected. In Ruby, when we see a plural variable name, we assume it's an array of that type of item. When we see a singular name, we assume it's one instance.


Merged.


Yes, thanks!


Unique scope of the platform


Had a go at it but it's not worth it IMO. The structure of SingleValueExtractors versus ArrayExtractors makes it into an unnecessarily complex abstraction.

I was able to reduce duplication though. Now only `extractLong` and `extractBoolean` have the whitespace handling. I'd prefer not to increase abstraction/reduce readability to remove a single duplication. And the tests will prevent any future changes from "forgetting this functionality".


This can throw  a security exception because you don't have permissions


Note: this will evolve in a parallel PR and the approach will be different (see #93), also fixing the bug. (bug has also been fixed in Spring Data Commons btw)


to include ereport(), for nice outputs


間違ってます。とむくんでわなくびびたんです。


Can you make an example?  How does the secret thentos information make it from the user back to the service?  What I think should be allowed in here are user name (not persona name), and that's probably it for now.

We can leave templating out for now, but it would be interesting to have this point in the haddock in case we want to add it later.


Is the “more” really needed in “more_custom_fields"?


Agreed.


two “ “ in a row :-)


prima prova


TeamCity pull requests :: netty [Build 1129](https://garage.netty.io/teamcity/viewLog.html?buildId=5144&buildTypeId=pulls_netty&guest=1) is now running


maybe extract the Sort logic, which can also be applied to the case where `accessor.getSort()` returns something (that should be covered as well).

**Don't directly use the property from Sort**, since it could be mapped to a different name in the JSON. You should use `com.couchbase.client.java.query.dsl.Sort[] cbSorts = N1qlUtils.createSort(sort, converter);` as is done in `N1qlQueryCreator`.


Don’t really love this name, but can’t figure out a better one.


Yea, I ignored the `:` in path case on purpose. The regexp would catch all kinds of invalid IPs as well. :-) I also did it because `localhost:123` _should_ be working, too. (Actually it does when you start `nvim` with `NVIM_LISTEN_ADDRESS=127.0.0.1:123`. So this is basically a Neovim bug.)


as said above, also cover the case where no `Pageable` but a `Sort` is passed to the method. Mutualize code between this case and the `Pageable`'s Sort.


oops, will fix it tomorrow. however, seems `new` never fails here, even when config file doesn't exist.


ah gotcha.


Ne peut-on pas utiliser quelque chose comme ``div > div`` pour éviter d'utiliser les sélecteurs CSS3 ? Je ne sais pas où en est le support... ?


@carlaschroder Skype call?


idem


Hmmm, are you sure our unbase64mem() can actually read the line-broken base64mem() strings we now generate?


You probably didn't intend to share this with the public...


おｋ


== 0


assignTrainingToUser


:+1: for traits.


probably need two hooks?
One with encryption/decryption before plugin get packet, and another plugin get packet before any encryption


Multiline text inputs on iOS contain a vertical scrollView. When you type, iOS automatically increases the height of the scrollView content, and it's that height that we send to JS in the change event.

In my autoresizing example, when JS receives the change event, it tells the component to update its height to match the height of the scrollView content. This keeps the height of the scrollview in sync with the height of its content.

The shadow node backing the textinput is completely unaware of the scrollView content height, it only manages the height of the component itself, so there's no conflict. Changing the height of the component doesn't affect the scrollView content height, and changing the height of the content doesn't affect the component height (except after the JS round trip, when my code tells it to update).

If there was any lag, the height of the content might briefly be taller than the scrollview, in which case you would be able to scroll the view vertically until it caught up, but there' no reason why that should make it flicker.


_assignTrainingToUser(), todo:should aslo be posted to database._


I guess that there should be `$format`, not `$ format`...?


This one should certainly be called `:department`, not `:update` — it's an ActiveRecord object, not some strange sort of action class.


+1 (but with @slemeur  question : edit commands icon should be colored or grayed?)


`id: object` certainly works (for now at least), but doesn't feel right.
`id: object.id` is so much cleaner.


maybe extract the mac… after the == to a method? Would probably be a bit more easy to read,


with MFIFO both get wrapped so it's totally fine :)

But for standard SPR transfers only the SPR needs wrapping as most other destinations will be straight, unless of course it's the edge of memory, but I don't think we check that anywhere as it's rare/unlikely


We already have a call for this in dnssec_keytag() in resolved-dns-dnssec.c.

(also for revoked keys it's kind important to also show the original keytag, i.e. the one where the revoke bit is off)


Why not a full name, `:department`?
Code would be easier to read through then.


Since this isn't SQL-based anymore it has become case-sensitive, which is a breaking change.


I do not really have a preference where should it go - in theory QgsGeometryUtils and QgsInternalGeometryEngine could be just one class - with assorted geometry functionality.

I would prefer not to expose it via QgsGeometry - that interface is already way too long - but I do not have a strong opinion.


empty line


Typo in `accesioated`


TeamCity pull requests :: netty [Build 1130](https://garage.netty.io/teamcity/viewLog.html?buildId=5145&buildTypeId=pulls_netty&guest=1) is now running


lol c'est le compilé :)


Please also add this new type to dns_resource_record_hash_func()


Again, let's call in `:instructor` everywhere, not `to_be_deleted`


non. http://caniuse.com/#search=%3Anot

Les utilisateurs IE8 ne verront pas le séparateur entre les messages. Je peux vivre avec.


I think size, subject ,date and headers should be still tested


Minor: perhaps we could define `Guide#content_owner_title`


There is a still a race condition here.  There is no sensible way to do `is_locked()` with `flock()`, this method should be deprecated and any use of it found within the `CAF` library removed.  It serves no purpose.


oups ;)


c'est le fichier compilé, répondu plus haut


not needed for this test?


Ok ;)


There are extra buffer after the memory in the memory map. It makes sense, start of the memory is the kernel. It is likely a bad idea to read/write from/to it.


Don't `unlink` the lock file, there's another race condition here, and there's no need to remove the file.  The existence of the file is meaningless.  It is the state of the lock on the file that has meaning.


wrong format for exceptions.  should be:
```python
try:
    # code to try
except Exception as e:
    # exception handling
```
and if you aren't using the message then there's no reason to capture it as `e` either.


Let's simply not do keytag calculation for RSAMD5. IANA says its "deprecated" and "recommended not to implement it", and that's what we should really.


This `department.to_param` just doesn't feel right. Are you sure that's exactly what this endpoint expects?


Cool!


OK.


Let's sort this out during our call.


Perfect just what i suggested in #38 :+1:


Shouldn't it be _**an** invalid request_?


Best suited for an irl conversation.


Do you have a test for this change? It seems important.


J'avais ajouté une class .next mais comme c'était possible avec du CSS pur j'ai pris cette option


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13170](https://teamcity.mesosphere.io/viewLog.html?buildId=115448&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 215 Build time: 00:04:14


@jasiekpor Commented out code. Please remove.


Technically the `type` isn't necessary any more, unless you were planing on using that other scripting language the browser provide...


umm
your timezones are wrong
for example

      'Australia/Sydney' => '(UTC+10:00) Sydney, Canberra, Brisbane',

but sydney and brisbane are different timezones because Brisbane does not observe DST

and

    'Australia/Hobart' => '(UTC+10:00) Hobart',

is strange because Hobart uses same timezone as sydney and has same DST rule

also No Melbourne so I cannot support this

when I am not drinking I am happy to help fix but i think the list you got is probably rubbish and a new one needs to be found


@gabizou it's [only logged if debug-verbose logging is enabled](https://github.com/SpongePowered/Mixin/blob/initphase/src/main/java/org/spongepowered/asm/mixin/transformer/MixinTargetContext.java#L404)


Je comprends la nécessité des 20 lignes qui suivent, mais... pff ouch quoi ;-)
Est-ce qu'on ne pourrait pas faire une partie du traitement en JS avant d'envoyer les données à Nunjucks?
Par exemple, cela semble naturel d'avoir un groupby("date") fait en JS. On pourrait ensuite avoir une structure author: messages_list préparée en JS (simple à faire, de surcroit) et balancer tout ça à la template, qui n'aurait qu'à itérer comme il faut ;-)


I like canola


From configure.ac in ../tools/depends:
``AC_ARG_WITH([toolchain],
  [AS_HELP_STRING([--with-toolchain],
  [specify path to toolchain. Required for android. **Defaults to** xcode root for darwin, **/usr for linux**])],
  [use_toolchain=$withval])``

Maybe you have something useful in ``/usr``?


Эта строка, по словам @AlexanderMatveenko, используется и в What's New и в диалоге приложения. Предлагаю оставить как есть.


Я только убрал префиксы в своем реквесте #1679


Hurra ! Thanks to @rdoursenaud @eldy ;))))))))


how do you mean? SPR does to/from memory, the other DMA's of course only do from memory (except IPU which writes out to memory) but we don't check for mirror boundaries on any of that.


I **am** stalking all of you guys!

(Jokes apart if this harasses you please tell me, I just want to be helpful)


18 kaarten eruithalen


Thanks for this, just one error of your own `vanilla traditional implementationimplementations`!


18 kaarten eruit halen,


Опечатка, удАлить


Префиксы - очень удобная вещь, чтобы быстро находить строки. Давайте их использовать по максимуму.


oh and you have included some very obscure timezones like Eucla, 368 people live in Eucla and its not actually a timezone
lord howe island has almost the same number of people

still no Melbourne


Should we do it in a controller/helper?

Perhaps we can order/sort by title too?


wow incredible!


No I have nothing in ``/usr/local/bcm-gcc``. I suspect this should be removed. It is adding a dependency on a second toolchain that doesn't appear to be required. Can you confirm if it makes any difference with that line removed.


Returns only baseline should be
return Y-B which would be the baseline corrected spectrum


The autotools scripts were probably generated with an old version of zproject. I'll fix this if it's still an issue (getting around to my emails slowly).


hahaha... no problem at all, Pier!! In fact, maybe you could help :wink:

I'm looking at this [install script](https://github.com/liferay/liferay-theme-tasks/blob/master/scripts/install.js)

Looks like the inner [npm install](https://github.com/liferay/liferay-theme-tasks/blob/master/scripts/install.js#L47) is not working now properly for us (is not installing anything).

If I run `npm install` using my system node and npm (5.5 and 3), everything works.

In our build system, though, we grab a local copy of node and npm inside the module we're building and run something like `node lib/node_modules/npm/bin/npm-cli.js install` which seems to fail now :cry:

Any idea of what's going on?


Nice, that's a lot better (using st2 for determining version that is) :+1:


`try_lock()` can report a number of different errors, which are only really meaningful on the last attempt.  In the previous code, the message `lockfile is already held, try $tries out of $retries` was logged for intermediate attempts to get the lock.


@MikaalSky I know! Sometimes it is a little unfair how those dead, decaying bodies get to be so skinny! I wish that we both could be that skinny!


this is why I prefer not to use 'using namespace std;' b/c it's hard to know where functions come from


it is my pleasure if you accept the merge request.


printWiFiStatus();  was not declare in the scope.  Error  occurred  during the compilation of program.


This breaks installation. crowdtruth/divedashboard does not exist.


I was trying to figure out how this worked (some comments would be nice)

//A1 = pi*d/4
//A2 = (x*h) + pi*h^2/4

//solve for x where A1==A2 gives x = -(pi (h^2-d^2))/(4 h)
//W = x+h
//W = h -(pi (h^2-d^2))/(4 h)

float width = height-(PI * (height*height-nozzle_diameter*nozzle_diameter))/(4.0*height);

(This is the same result as your formula but slightly simpler)


what updates are there to a task? pybossa so far only links to an experiment in worker-service and has no other information on it. does pybossa need to show title and description of an experiment or is this sent by the worker-service?


While readability is improved, it's probably less secure.
Constants cannot be changed during runtime, while a variable can (or ultimately be deleted).

It's a toss-up.


Can `Wiki.joins` be replaced with `scope.joins` on line 39?


Дим, не совсем поняла. какого заголовка? Ты имеешь ввиду тег к тегу p?


Would it not be nicer to have one method that allows explicitly passing in `content_owner: nil` to build an edition without a content owner?


Я сделаю просто autodownload, так логично и просто. И использовать можно будет как угодно и где угодно.


Can non-premium users own wikis? If yes then review the `where` argument on line 40.


@JustAkan 램디스크는 lz4 로 압축해도 정상 부팅 되는데, dtb 이미지는 lz4 로 압축하면 arm 때와 다르게 자체적으로 풀리는 구조가 아니라 부팅이 안되네요.

램디스크만 lz4 로 해볼 생각입니다.
LG 에서 개발한 압축 푸는 속도가 제일 빠른 포맷입니다.


Thats not loking good


OK, sent a PR to fix this.


exists


Why is this dead?


In force mode, we continue regardless, however we don't record the fact if the `LOCK_EX` failed.  Therefore we will error out when `unlock()` is subsequently called which will attempt to `LOCK_UN`.  We are also attempting to use `$lf` in force mode even if `FileHandle->new()` failed to get a file handle.

If `FileHandle->new()` fails, then `try_lock()` should fail too, regardless of whether this is force mode or not.

If `flock()` fails in force mode, don't update the PID in the lock file itself as this would be misleading, and don't record the file handle in `$self->{LOCK_FH}` (but still set `$self->{LOCK_SET}` to `1`).  Then in `unlock()`, if `LOCK_SET` is `1` but `LOCK_FH` is not set, then don't attempt a `LOCK_UN` because it will definitely fail.


The first sentence of the commit log is a bit rough:

> /showIncludes was not added to and empty list if includes was stored in the manifest.

'Was not added to and'? I guess 'list if includes' should be 'list of includes'.

Code-wise, I think the patch could have been simplified to just

    realCompilerCmdLine = ['/showIncludes'] + cmdLine if stripIncludes else cmdLine

...but it seems to me that even this is not necessary: in my experiments it appears that `/showIncludes` can be specified multiple times without changing the effect, i.e. maybe the patch is just:

    returnCode, compilerOutput, compilerStderr = invokeRealCompiler(compiler, ['/showIncludes'] + cmdLine, captureOutput=True)

In any case, I think this trick of unconditionally adding `/showIncludes` warrants a commit: the effect and the cause (getting a more expressive manifest) are fairly far apart in the code. At least to me, the effect is not obvious from reading the code.


Could be named `defaultAdminCode`


Yes, can I help with something?


@tchaikov OK, would you like me to do this or you are going to fix it by yourself :-)


По-моему, в ключе должен быть контекст. И переиспользовать эту строчку нельзя, так как даже сейчас её нужно изменить на что-то более информативное типа "Загружать карты автоматически".


There's a bit of inconsistency in naming. `HiLoSettings` and the param is `HiloDef`, the `L` is mixed throughout the code, I'm unsure if that's intentional or not.


`string split -a` - string is a multi-purpose tool with subcommands.


very  well


Is the `Increment` just for flexibility? I can't think of a scenario where you would want to increment > than 1 at a time.


Adding sencond line of code


I know submit method is no good, is some kind of pseudo code :), but i need your suggestion.


Thanks, this method really helped me with issue with not rotated images after dropping!


@xiexingguo please let me fix it. seems it was me who screwed your previous fix =(


fixed at https://github.com/tchaikov/ceph/commit/ca7cfbc4dd7199fdea564db742873409a99a05ae


don't forget to  close #622 after you remove all of them , nice work.


@oana-inel , this is related to the package we added a few days ago?

Package crowdtruth/divedashboard still exists on packagist:

https://packagist.org/packages/crowdtruth/divedashboard

But the github repo it used to point to doesn't seem to exist any more.

https://github.com/CrowdTruth/DIVEDashboard (404 - page not found)

Has it changed name? I found this other repo:

https://github.com/CrowdTruth/Crowdtruth-DIVEDashboard

But it seems empty. Let me know if you want me to change the repo where crowdtruth/divedashboard points to.


By convention we put license.xml into the root directory of the project.

We also generate models in src (cd src && gsl whatever)

So, the correct syntax is <include "../license.xml" />

I'll fix this in zproto.


Thats correct. But in order to pull in Symfony <3.0, I have to explicitly specify it in my project's composer.json, which forces me to know about Silex's dependencies.


Should now be fixed in the latest commit on dev.


Sorry, there's no error in zproto.


Пример удалять не надо было, просто дописать что надо проставлять ключи в массиве models


@tchaikov Never mind and suit yourself. Then I'll drop this change:-)


THis doesn't look right. ttys aren't the only character devices...


Мусор скопирован и нерабочий пример в итоге.


Yes, I removed it :)
I integrated the code in the framework. I'll solve this issue soon, I just
forgot to update the composer.

Oana

On Thu, Jan 28, 2016 at 12:55 PM, c-martinez <notifications@github.com>
wrote:

> @oana-inel <https://github.com/oana-inel> , this is related to the
> package we added a few days ago?
>
> Package crowdtruth/divedashboard still exists on packagist:
>
> https://packagist.org/packages/crowdtruth/divedashboard
>
> But the github repo it used to point to doesn't seem to exist any more.
>
> https://github.com/CrowdTruth/DIVEDashboard (404 - page not found)
>
> Has it changed name? I found this other repo:
>
> https://github.com/CrowdTruth/Crowdtruth-DIVEDashboard
>
> But it seems empty. Let me know if you want me to change the repo where
> crowdtruth/divedashboard points to.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CrowdTruth/CrowdTruth/commit/cb3800bd68024f21a894f7551f08b1eca1de4c90#commitcomment-15730727>
> .
>


@phil-davis I believe, the disconnect is that we have two deployment paths.
a) Deploy through a download repository that is registered with the package manager on a users computer.
b) Deploy updates through the Client Updater Server.

The statement 'If you host a branded download repository for your users, then you don’t need...' clearly refers to a) and implies that b) should be disabled in this case. Not sure if that implication holds water.

One way how to disable b) is to configure a very old version in the Client Updater Server, so that it will never offer updates. Somewhat hackish, if you ask me :-)

@carlaschroder sorry for tearing your innocent words into pieces, without giving clear advice. Maybe @danimo can please double check the when and how of disabling the Client Updater Server.


Should we remove from packagist then?


Probably fail.


:( on aimait pas les icônes? Je trouvais cool, aussi ça les différenciait des autres links.. :(


sorry mate, I'm off the internet and my laptop for couple of weeks again :(
work call

On Thu, Jan 28, 2016 at 10:18 AM, Josh <notifications@github.com> wrote:

> Never mind. I thought of a better solution to fix this :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gsstudios/initramfs3/commit/b1479eabfb6d241d252eb0842e997ea20a93cc88#commitcomment-15723521>
> .
>

--
Thanks & Regards
Ruturaj


actually the toolchain should be removed from PATH instead. this is a rpi specific "hack"


What's the working directory of that exec?


Копипаста из проекта, надо брать примеры из старой документации. Сложно разобраться что за поля, слишком частный(перегруженный) пример.


“我们明白这可能令人感到混淆，因此我们正在努力地清楚说明这些关系。”
我们知道这样的比特币生态会令人感到费解，所以正努力将这些关系说明的更加清楚。


Make sure to add task number in commit message next time. ;) #2


For sure !
But power from rideFilePoints is not necessary with 1sec recording interval like WPrime data and It seems I missed the smoothedArray for this purpose.


Make sure to add task number in commit message next time. #4


Let me ask @csierra...


Why do you need the quotes?


I missed this commit. Just pulled the latest version to the server. No more Rick Astley.


thanks for explanation. good spot on signature


good point, sure

On 28 January 2016 at 08:09, zzts <notifications@github.com> wrote:

> Would you consider removing the TSL2591_ prefixes on all the variables?
> It's not necessary and just adds extra typing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zzts/python-tsl2591/commit/629d740216b20c31c7bea99ec95a39ab94b56ccb#commitcomment-15725247>
> .
>


Надо ещё добавить пример где будет обозначено как формируются url-ы для моделей/urls и как выглядит разбивка при большом количестве ссылок (прям скопировать с живого).


you are right


I don’t really understand what that script does. Actually I get the **what** not the **why** :confused:


IDE was suggesting it.


Typos: sure, thanks

Let's discuss the rest on the main thread, to keep it visible when I rewrite commits.


RewriteRule on line 9 should be doing this.


Looks good and works on dev


Thank you!


There doesn't seem to be a way. Using a ServiceTrackerCustomizer seems the best option as you have done. Thanks!!


It would be nice to see an example.


Je suis pas trop pour, ça voudrait dire utiliser moment.js en dehors de nunjucks. Au final pas sur que ça apporte grand chose (on précompile quand même le template). Par contre je peux essayer de simplifier tout ça et d'utiliser {%- -%} .


the cost of testing individual headers is big enough to keep a full object equality


これは"their own transaction"なので、意図的に「その自身のトランザクション」と訳した可能性もありそうですが、最後の"started"が訳出されていないなど、原文とややニュアンスが違うように思うので、訳文自体を見直します。


This was a bug fix - the documentation already states that the attachTo option for a step can either be a string or an object - recent changes meant that passing an object resulted in no attachment to an element.

attachTo: What element the step should be attached to on the page. It can either be a string of the form "element on", **or an object with those properties**. For example: ".some #element left", or {element: '.some #element', on: 'left'}. If you use the object syntax, element can also be a DOM element. If you don't specify an attachTo the element will appear in the middle of the screen.


Edit: {% for message in results|groupByAuthor %} ça t'irait? on peut alors coder en plain JS


It works now, straight from clone. thanks!


You have an error, `strings.contains` should be `strings.Contains`


no jasne, ale ja potrebuju vedet ty casti aplikace. to si to mam vymyslet? co kdyz to vymyslim nejak jinak nez ostatni?
ja to potrebuju exaktne specifikovat, ja su uplne blbej:)))


please suppress unused warning and write where it used


@ajmath that must be it but I've no idea how I managed to pull that off. In either event we should have the buildImage-upToDate functionality merged today.


ici: {% if loop.last %}


Required as we get a 404 if we duplicate fields in the request - tested.


elif


I’m stupid. Never known there’s a standard `install` script, always used `postinstall`.


ici on utilise plus la variable on peut dégager ce bloc


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


Sorry,this is my mistake.I must restore original one.


on peut bouger les conditins user_last ici, plus besoin de al variable


@AlexandreDecan J'ai pris des notes. A suivre ce soir.


Je crois pouvoir réduire pas mal


Ya, it is hard to predict, somewhere from the function where you drag a bubble. The variable is a member so it is not set from that call.


Moved instantiation as otherwise we cant set Fields in the constructor..
Same way scope is defined in OAuthOptions
```
public IList<string> Scope { get; } = new List<string>();
```


@BebZ I wouldn't mind but I don't think so. I find Dolibarr full of very-specific modules and this would be one of them… Also, due to Dolibarr using old versions of external libraries, if a bug appears it won't be fixed until next release (and be grateful), while if in an external module will be fixed in a few days.


@darkeyes84 You can fix it by changing it here
https://github.com/BlissRoms/platform_vendor_bliss/blob/mm6.0/config/common.mk#L102 to init.bliss.rc


--engine-install-url cannot be used with the hyperv driver, use --hyperv-boot2docker-url instead


--engine-install-url cannot be used with the virtualbox driver, use --virtualbox-boot2docker-url instead


--engine-install-url cannot be used with the vmwarefusion driver, use --vmwarefusion-boot2docker-url instead


@rootical look at this solution. This is correct? When I used init function I didn't have new instance for name values and when I added new human object all previous objects had the same name.


--engine-install-url cannot be used with the vmwarevsphere driver, use --vmwarevsphere-boot2docker-url instead
 Add a line note


hahaha... not sure, I'm just waiting for @rframpton to wake up, he's the one that will know what he's doing :joy:

The working dir for the inner npm install looks good to me:

`[..]/modules/themes/user-theme/node_modules/liferay-theme-tasks`


@SubJunk Do not forget to pull from Crowdin, it's really nice to see all the new translations:

We have gotten very substantial contributions in Arabic, German (100%), Hungarian, Italian, Japanese, Portuguese, Brazilian Portuguese (100%), Swedish and both traditional and simplified Chinese.
```diff
 ar.name=Arabic
-ar.progress.translated=19
+ar.progress.translated=41
 da.name=Danish
-da.progress.translated=40
+da.progress.translated=45
 de.name=German
-de.progress.translated=30
+de.progress.translated=100
 hu.name=Hungarian
-hu.progress.translated=0
+hu.progress.translated=52
 it.name=Italian
-it.progress.translated=49
+it.progress.translated=65
 ja.name=Japanese
-ja.progress.translated=18
+ja.progress.translated=60
 nl.name=Dutch
-nl.progress.translated=93
+nl.progress.translated=94
 pt.name=Portuguese
-pt.progress.translated=19
+pt.progress.translated=61
 pt-BR.name=Portuguese, Brazilian
-pt-BR.progress.translated=20
+pt-BR.progress.translated=100
 ro.name=Romanian
-ro.progress.translated=16
+ro.progress.translated=22
 ru.name=Russian
-ru.progress.translated=26
+ru.progress.translated=27
 sr.name=Serbian (Cyrillic)
-sr.progress.translated=0
+sr.progress.translated=3
 sv.name=Swedish
-sv.progress.translated=25
+sv.progress.translated=52
 zh-Hans.name=Chinese Simplified
-zh-Hans.progress.translated=24
+zh-Hans.progress.translated=65
 zh-Hant.name=Chinese Traditional
-zh-Hant.progress.translated=25
+zh-Hant.progress.translated=65
```


Fix accordingly


attach 'baseDomain' parameter to all application.


Unused code. Please remove.


for fireball-x/fireball#1362


Unfortunately it seems it does not work as expected (should fix #50).

We need to write some spec test case for that to be explicitly tested


I don't know who did the push, but you are, guys, doing something wrong with Git:

```
 Fetching gh-edison-linux
 remote: Counting objects: 915392, done.
 remote: Compressing objects: 100% (44/44), done.
 remote: Total 915392 (delta 255389), reused 255385 (delta 255385), pack-reused 659963
 Receiving objects: 100% (915392/915392), 198.81 MiB | 1.15 MiB/s, done.
 Resolving deltas: 100% (776562/776562), completed with 31345 local objects.
 From github.com:01org/edison-linux
    ac21b06..4bf19a00 wip-edison-3.19.5 -> gh-edison-linux/wip-edison-3.19.5
```

200 MiB for an update to the same version?! Imagine "happy" user for that…


Ok :)
Je ne suis pas partisan du |groupByAuthor car cela signifie (si je ne me trompe pas) de défnir un filtre (JS) spécifique pour la template. C'est donc une façon déguisée d'ajouter de la logique dans la template, je préfère alors manipuler les données avant de les passer à la template (cela ne me choque pas d'utiliser moment.js en dehors de la template, surtout qu'on veut juste comparer des dates, pas faire de transformation dessus).


Thanks,I locally removed these mistakes, unnecessary comments and restored original gui/filehandling.py in my PC.
And I'm going to rebase/squash this now,
but I'm not sure how to do it in correct github/mypaint manner.
Should I rebase all of my commit and then push it? or is there any other(correct) way?


Muhahahah there's a comments system built into github!

`sudo editor` <3


"PT Sans"


First of all I suggest to output the stdout and stderr through stream piping:

```js
exec('...', (err, stdout, stderr) => {
  stdout.pipe(process.stdout);
  stderr.pipe(process.stderr);
});
```


add current host parameter.
each application's config has a same parameter.


I mean since `fn.assign` has already been created (so it's body should contain (or not contain) appropriate checks based on the current value of `expensiveChecks`), wouldn't it be enough to just copy it over to `expensiveCheckFn` ?
Can the value of `runningChecksEnabled` affect the execution of the `assign` function ?


Exaktně: filename bez přípony

Dobrý? :)


Woops. Turns out eclipse's "Block Selection Mode" doesn't like auto-completion.


TeamCity pull requests :: netty [Build 1129](https://garage.netty.io/teamcity/viewLog.html?buildId=5144&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 01:13:24


...and there was me wondering where all the random letters I was typing in were going.


TeamCity pull requests :: netty [Build 1131](https://garage.netty.io/teamcity/viewLog.html?buildId=5156&buildTypeId=pulls_netty&guest=1) is now running


HI.
In my OnePlus One, skiping song from volume buttons, bluetooth car audio or whit gestures doesen´t work.
Whit others music apps works great, and in older cm13 nigthlies works aswell.
Maybe this change is the problem??


Does this mean that we should also do `expensiveCheckFn.inputs = fn.inputs` ?


ensureは、ご提案どおり、「確実に」を削って「保証されます」としました。
properlyは「適切に」のまま残しました。


Ignore these, I sometimes forget to do a `git pull` before I commit. :smile:


Key was regenerated and encrypted. Documentation how to encrypt keys is located [here](https://docs.travis-ci.com/user/encryption-keys/).


Nice! Do you think we can collapse those two selections into one, with some OR-ing?


はい、それが一般的に使われる訳語だと思います。


I cannot find anywhere in repository. so I captured particle example from processing.org. thanks processing!


Awesome, sounds good.


At some point during the design, I was using the single vs plural nature to allow some constraints to naturally apply to both. For example `@NotNull`. The need faded but I wonder it if it will resurface later.

If we do not differentiate it means that each "single" extractor will have to return an `Iterable<T>` with a single element in it (or null). That's an extra object all the time with likely more for "average" implementations using a collection to build an `Iterable`.


At least it was inside a comment, no harm done.


Yay! After spending many hours struggling with random test failures, I believe I've tracked it to this exception silencing as the main culprit. Thank you.


I was not focusing on names much, I need to collect them for a latter round.


If it wasn't inside a comment I would have noticed the compile error :)


TeamCity pull requests :: netty [Build 1130](https://garage.netty.io/teamcity/viewLog.html?buildId=5145&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4911, ignored: 23 Build time: 01:05:34


Fantastic, thanks!


I hope you generated this block of code. :) Also, I'd use just `return createJsonData();`


Tut cela c'est du cosmétique donc pour moi ça doit etre dans le template. Mais je crois que ce soir je pourrai réduire ce code à peau de chagrin


Меня смущает слово "стягувати". Никогда не слышал его на украинском. Если точно норм, то ладно.


Shouldn't the names mention Cabal? E.g. `CABAL_PKGTEST_*`?


Я так понимаю, это заголовок сообщения о миграции?


Лол


Если заголовок, то эта строка не влезет, так как под заголовок выделяется всего две строки.


need to re-implement xp incrementation at some point in the battle


this is a temporary exception , will go away when ntlm implementation comes in. In next PR mostly.


@stulees I was mucking about with the tests to try to get them to pass and missed this. Not sure how CCI passed with this in place. Hrm. Will fix.


TeamCity pull requests :: netty [Build 1131](https://garage.netty.io/teamcity/viewLog.html?buildId=5156&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:08:33


Я не согласен с таким именованием: https://github.com/therearesomewhocallmetim/omim/commit/352a97cdc5abda72560206786ad192a6d0a32686#commitcomment-15730469


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13171](https://teamcity.mesosphere.io/viewLog.html?buildId=115489&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:08:34


Well, never say never, but Rick will have to take the same route as everybody else ;)


CI Test failures do not appear to be related to this.


Again this is temporary .Going to be removed in next PR. SO resource string for this is not needed.


I think you misunderstand the sentence. To translate it directly: 我們知道這會令比特幣生態系統混淆, but this is not a good way to express in Chinese.


VIelen Dank. Das war ein Denkfehler von mir. Ich habe in dem Moment an die Seitenzahl und nicht die Anzahl der Seiten gedacht. **PageCount** wäre vermutlich der bessere Name für das Feld gewesen.


Тебе достаточно одного тэга вместо двух, https://www.w3.org/wiki/HTML/Elements/h1,_h2,_h3,_h4,_h5,_and_h6


Вернее как-то вроде "Загружать автоматически при приближении".


Installing those components dynamically at installation is somewhat wrong.

Those should be added as dependencies in the generation phase. In fact whatever I do will with npm will fire errors about extraneous (as in *unsaved*) dependencies.


bah maintenant faut le faire à ta sauce .... en gros reproduire ce png en HTML/CSS


Если "Удалить всё", то должно быть "Видалити все". Если "все" - то "всi"


Related to this change, shouldn't `addqi` and `subqi` be cleaned up [here]
(https://github.com/facebook/hhvm/blob/master/hphp/runtime/vm/jit/vasm-xls.cpp#L638) ?


Shouldn't this be `intOrFloat`. Can this have both values?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13172](https://teamcity.mesosphere.io/viewLog.html?buildId=115490&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:04:04


New changes


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


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


Аналогично, см. ниже.


Wow, that was a simple change!


or move it to gradle


test commit


Need to create the database, see 3830fc26.


:warning: NPE (null pointer exception)
Este tipo de if te conviene ponerlos al reves, de la forma "password".equals(campo) ya que en el caso contrario, si campo es null tenes un null pointer exception y en la forma inversa solamente un false.


resolves #57


TeamCity pull requests :: netty [Build 1132](https://garage.netty.io/teamcity/viewLog.html?buildId=5163&buildTypeId=pulls_netty&guest=1) is now running


This is called when the weapon is restocked. I don't see what's wrong with it, then.


I see you do -- sorry.


Unfortunately, it looks, this hash is not exist on benchmark.js repository, so bower install is failed.


No, it's not a bug, it's a feature! I want to make some turns for an opponent to better estimate a current sequence of moes. Currently I make only one turn because of perfomance, but probably in future I'll change this constant.
I suppose that the best thing to do is to get rid of this magic numeric constant, replacing it with named constant


@mathewpeterson, just asking, why do you have this container and not only the PHP for the application?


You forgot to change the argument type in the declaration of the method (l.115), resulting in the examples not compiling anymore.


This `foreach` looks ripe for being in a function of its own, which is then just called in each view. Procedurally looping through the exact same thing in each view looks WET.

```
ChainMail::renderSections($context);
```


Нужно заменить id в коде.


done, .pot files were excluded by .gitignore


Yep, thanks for the report.


Trailing line spaces needed at end of file(s).


Pillo proyecto


Can i pick up this project for the future?
Really love it
xxxx


Temporary exception .Going to be removed in next PR. .


Саша, у нас пять лет использовался единый, хорошо работающий подход. Ты сейчас предлагаешь его быстро налету поменять? Я против.


What about?

```
return _check_msg_version() && _deserialize_message();
```


That is already fixed, see the most recent version of `public IEnumerable<ITemplate> GetAll(params string[] aliases)`


TeamCity pull requests :: netty [Build 1132](https://garage.netty.io/teamcity/viewLog.html?buildId=5163&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:02


What has Kong got to do with [J. Walter Thompson's website](https://www.jwt.com/)?


Your RPG plugin shouldn't know which plugins are using it, the plugins using it should have an optional dependency on your RPG plugin in that case. Same for the services, hardcoding the plugins implementing a certain service in the dependency section is very bad practice imo, this will just break if there are multiple different plugins implementing the same service.

_I'm against adding this, however I think plugin authors should be really careful about when to use it, because it can cause bad plugin habits quite easily._


But have noticed a few existing issues with comparing alias, will fix these now.


None of the cases above. Method is getting referenced in the code common to both unix/windows. but the value in case of unix is returned as 0 , this value is not used and does not impact the code path for unix.
So neither we can remove it nor put the Debug.Fail.


Что такое "единый, хорошо работающий подход"? В том, как сейчас добавляются строчки, нет никакой системы. Я предлагаю добавить некоторые улучшения. Не бросаться исправлять по всему коду, конечно же.


Not sure why it did this (maybe because it's unused and `nil` - https://github.com/bbatsov/ruby-style-guide/issues/237?), but everything seems to work.


Yes it can.


hi, could you please draw the imaginary parts as well ?


Why not removing `profile` parameter all together? Use `this` instead. Alternatively, this method could be transformed to `static`.


Еще раз плюсы отсутствия префиксов:
- строки не дублируются. Там, где надо [save], ты вставляешь save, а не создаешь себе копию [editor_save]
- не нужно ждать, пока кто-то разберется с переводами. Берешь и вставляешь в код готовый.
- не разбухает файл переводов.
- нет "потерянных" строк в проекте.
- старые строки хорошо вычитываются и перепроверяются пользователями. Наши переводчики не всегда понимают контекст и не всегда правильно переводят. Использование старых строк, или добавление в них новых переводов - правильный путь.
- переиспользование строки с префиксом в коде вызывает непонимание (вставь editor_save в редактор закладки и озадачь программиста закладок)


That explains why one of the javadoc changes disappeared somehow yesterday and I had to add it back manually...


In the meanwhile I made liferay/liferay-theme-tasks#12 to account for my complaints.


Is this log message meaningful if we don't log namespace and selector?


Spacing is incorrect.


why do you use a copy of array?


Non, elle giclent :)

Sent from else where.

> On 28 janv. 2016, at 12:59, Walid <notifications@github.com> wrote:
>
> :( on aimait pas les icônes? Je trouvais cool, aussi ça les différenciait des autres links.. :(
>
> —
> Reply to this email directly or view it on GitHub.
>


This should probably say unix-0.5.jar.
Did you make any progress on finding the official sources which we need for the CQ?


Why not using just `signIn` instead?


I mean, you can't mirror the write after the end of memory to the start of memory. Otherwise you will overwrite the kernel. Here a quote from the source.

```
struct EEVM_MemoryAllocMess
{
....
	// Two 1 megabyte (max DMA) buffers for reading and writing to high memory (>32MB).
	// Such accesses are not documented as causing bus errors but as the memory does
	// not exist, reads should continue to return 0 and writes should be discarded.
	// Probably.

	u8 ZeroRead[_1mb];
	u8 ZeroWrite[_1mb];
};
```


There is already a login method, def login (self,loginID, psd, ldap_connection=None):
Do we need another !?


LGTM


I think it's worth to comment that `cpu-usage` is already in millicores (it is, right?)


you could use `pkill` for that.


s/return/returns


that's weird.. I haven't got OG no more to test it, so I have to believe in your words. thanks


Close issue #1


Really good project, lucky ozil guy


since you have an if condition in each moveView method, this could be one single method.


@henriquemoody that way we can have multiple PHP containers for different PHP versions, I would say.


@praveenkumar  you should add a change log


Why have you changed the version here?


It's looking good so far. Is it fully working?


s/custom custom/custom


Если здесь неправильные строки или переводы - давай выкинем. Как должно быть? Какой контекст использования? Может она вообще не нужна?


Yes, this might be a good idea.

On Thu, Jan 28, 2016 at 12:58 PM, c-martinez <notifications@github.com>
wrote:

> Should we remove from packagist then?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CrowdTruth/CrowdTruth/commit/cb3800bd68024f21a894f7551f08b1eca1de4c90#commitcomment-15730796>
> .
>


Not sure how to add inline comments to a file which was renamed without changes.

in ```apidoc/Makefile.am```:

The hunk below should be removed as it causes a circular dependency, I've verified that out of tree builds work.

```
-# Hacks around gtk-doc brokenness for out of tree builds
-ostree-sections.txt: $(srcdir)/ostree-sections.txt
-       cp $< $@
-
```

nitpick: s|bulid|build| in the commit message.


why did you use dispatch_once_t for this static var? can you explain how it works?


You see?.. Here you are using `profile` to call it's method `createJsonData` and pass same `profile` as a parameter.


А давайте вопросы такие решать ДО переводов, а по факту переведённые строки просто добавлять? Тратим уйму времени в никуда.


TeamCity pull requests :: netty [Build 1133](https://garage.netty.io/teamcity/viewLog.html?buildId=5166&buildTypeId=pulls_netty&guest=1) is now running


Это пункт в настроках карты для автоматической загрузки при зуме.


Python Templates/Filter with Dialog/README.md

Die größte des Fensters ist uninteressant.


TeamCity pull requests :: netty [Build 1134](https://garage.netty.io/teamcity/viewLog.html?buildId=5167&buildTypeId=pulls_netty&guest=1) is now running


I didn't know about that. Thank you :D


redefined :cry:


Там много места для длинного перевода? Игорь, как я понял, задумал именно такую строку.


这里居然还有写错的。。。。。。


Why was this removed?


:) it's just a test URL


importing the persistency manager, strongly couples these 2 classes, meaning, we will not be able to use the Graph component anywhere, if we don't have that persistency manager.


Нет, всего одна строчка


Prečo sú .jpg iné ako som poslal cez github. Posielal som
[hlasenie_vyroba_v16.pdf](https://github.com/eurosecom/projekt1/files/108168/hlasenie_vyroba_v16.pdf)


Accepted from RStudio


Единый подход:

Игорь заказывает строки. Наташа смотрит, что у нас уже есть, чтобы не переводить повторно (у нее своя база переведенных строк).
Программисты вставляют в код строки, ревьюят и очевидные ошибки исправляют.

Done.


Значит оставляем как есть.


Guys, I don't even **have** `erlang-syslog-1.0.1` in my deps.
I have `lager_syslog-2.0.3` and `syslog-1.0.3` which I am sure you guys have too.

The issue? un-clean'ed BEAM files in old dependencies. Please `rm -rf deps && git checkout -- deps && make …`.
(That must be it)


could you place this function closer to the call site?


getRasterMetaData ?


:( :( pas d'accord.


there you go, I am going to send you the dictionaries as soon as I have
them clean and neat.

best,
Sira.

On Thu, Jan 28, 2016 at 1:47 PM, Vincent Lostanlen <notifications@github.com
> wrote:

> hi, could you please draw the imaginary parts as well ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lostanlen/scattering.m/commit/598522adaa7f820875dadae7ef36d80106755f76#commitcomment-15731647>
> .
>

--
Sira.


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


Hi, I have added the POG csv SF to the Jet_ collection, so now we have:
Jet_btagCSVL_central
Jet_btagCSVL_up
Jet_btagCSVL_down
and same for CSVM and CSVT.
A standlaone macro is also provided in python/


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


I have never commented on a this repo before, but I just want to say I love you.


maybe the following structure would be better:
```
if () {
return 0
} else if () {
} else {
}
```


You're extending this method but it has literally no test coverage, please add at least one.


Here we should also check that the second task `buildImageAgain` was UP-TO-DATE. Is this even how it works? I thought I saw that in the original PR.


Саша, строки отлично ищутся по содержимому. download_all, save, start_migration - куда проще? Зачем тут префиксы?

> On Jan 28, 2016, at 14:35, Alexander Marchuk <notifications@github.com> wrote:
>
> Префиксы - очень удобная вещь, чтобы быстро находить строки. Давайте их использовать по максимуму.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/therearesomewhocallmetim/omim/commit/352a97cdc5abda72560206786ad192a6d0a32686#commitcomment-15730283>.
>


Hi Kristian,

First I want to thank your contribution to CloudVAMP.

We have to test it but I have read the ONE documentation (http://docs.opennebula.org/4.12/advanced_administration/application_insight/onegate_usage.html) and it says that the header name is "X-ONEGATE-VMID" and not "X_ONEGATE_VMID" as you used in your code.
Have you tested it with this header?


Thanks and we are able to use the same for ejabberd version 14.12

now how do i dynamically add/remove vhosts using this module . suggest us the interface (i/e/ can we use ejabberdctl)

your immediate help on this would be appreciated


I like this form better


package is gone


Je préfère FIXIE


I have inspected DNet `ObjectStoreFile` class and it seems `fileSizeKB` is defined as `long` primitive type so we can keep the implementation as it is and define `contentSizeKB` in `DocumentContentUrl` as required. Probably in different commit to make this change less implicit.


ここら辺ってajax入れると不要になる記述だから、1ブランチで一気にやっちゃっても良かったかもね(この前の厳重注意にビビったと見受けられる　笑)
「1機能」だから、「like機能」で1ブランチでいいよ。
まぁここに追加するか次のブランチにするかは任せるわ！


Imagine this situation: files.length = 1, this row will throw error when variable 'i' will became equal to 1.


did we have a convention of `// Drop 10.10 ...` or something whenever we used `[NSApplication isSomeOS]`?


Ah, I should /cc @ttuegel to know it should be fixed now.


I don't remember exactly. Had something to do with the format that python packages expected the version to be .. or something like that. In any case I recall that the pyethash package is unmaintained since mathew has not given ownership to anyone else.


This isn't very readable. Any chance we can make it an if/else?

Never mind if you don't want to change it


あーちゃんと入れてるんだ。
この前のコミットで指摘したけど気にしなくておっけ！


Actually this log message was there before. See:
https://github.com/kubernetes/kubernetes/blob/master/pkg/controller/podautoscaler/metrics/metrics_client.go#L144


TeamCity pull requests :: netty [Build 1134](https://garage.netty.io/teamcity/viewLog.html?buildId=5167&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Out of memory: unable to create new native thread (new); tests failed: 1 (1 new), passed: 3390, ignored: 6 Build time: 00:09:23
### Failed tests
```
io.netty.buffer.PooledByteBufAllocatorTest.testConcurrentUsage: <no details avaliable>

```


Tests please.


how did you calculation the radius? Why `/16`?


Sorry, for late review. But what about setting a `new ThreadPoolExecutor.CallerRunsPolicy()` to the executor? (`setRejectedExecutionHandler()`) and possibly checking the task for `null`.

`TimerTask` has the problem that it kills the thread on exception. Maybe we should move to `ScheduledThreadPoolExecutor` which doesn't have the problem anymore.


first commit


I'm going to move this together with some other points in a separate issue I'll create later so just comment there once I've created it.


ここら辺長いから、条件判定用のメソッド組んで適応させてあげよう。


シングルね


Done.


スペースね　ちょこちょこ抜けてるから癖付けて


2 x Done


%button はbutton_tagで書こう


了解！　これをメソッド化してあげて！


@vcunat Oh, I guess I had most of these changes in my unmerged branch, but it looks like you picked up a few issues with patches that I missed.


And yes, we should fix all of the logs around here, but in a different pr.


Where is elsewhere :grey_question: A direct public Link on README should be cool...


what's the difference exactly between:
src/**/**.e2e.ts &
src/**/*.e2e.ts
?


The default behaviour does not seem to be documented.


ふと思ったんだけど、毎回@main_thumbnail.image.url(:large) ってするなら、コントローラーで.image_url(:large)までを含めて@main_thumbnailに入れちゃダメなの？


TeamCity pull requests :: netty [Build 1133](https://garage.netty.io/teamcity/viewLog.html?buildId=5166&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:15:10


add a package comment


TeamCity pull requests :: netty [Build 1135](https://garage.netty.io/teamcity/viewLog.html?buildId=5172&buildTypeId=pulls_netty&guest=1) is now running


これもどうせajax入れるから消えちゃうんだよなー　笑


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


%button → button_tag


@stulees All fixed up. Thx.


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


It is a bit strange as PyPi package available has version 0.1.27.


no. you can also simply configure your project to tell composer that it needs to select packages compatible with PHP 5.5.8 instead of using the PHP version from the runtime used to run composer: https://getcomposer.org/doc/06-config.md#platform


Ok.


@fenollp i think this is not needed


В коде в ветке этого айди не нашел. Покажешь, пожалуйста, где он встречается? Возможно он в невмерженном реквесте? Но тогда надо править реквест.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1257](http://teamcity.monogame.net/viewLog.html?buildId=14113&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed to start: 2 Build time: 00:00:00


I know whistle_apps will always be started. It doesn't hurt to specify it though


We only have one instance of FieldResolver and its cache is not static on purpose (so that 2 clients with 2 connectionsettings do not bleed into eachother).

So two calls two resolve calls on two threads can not share a single stack property like this. This lock is superfluous.

We need to seperate the visitor and FieldResolver (with cache) in to two so we can new a visitor each time.


为什么不是直接上一排 if 然后 merge PR 欸…


OK, I'll port this to the python generator.


@copumpkin PR wasn't applying cleanly and fastest was to just manually merged it instead of playing pr-ping-pong.


Oops!  Accidental drug reference or intentional subliminal message?


Yes, you're right. I'll update the comment to mention this


Thank's :)


Я думаю это What's new.


Диалог миграции: MwmActivity.showMigrateDialog().
Когда сделаем WhatsNew, я потом добавлю там тоже.


No line length limit in Go in general. Don't know if your project has preferences, but normally this string should not be broken into two.


I'm not sure why you'd want a web server to pose as a client (through clif). Ideally, if you wanted to connect from a webserver to the mapserver, you'd use a separate socket...


Заголовок диалога миграции выше - migrate_title


Thanks :)


Переписано в 0291657bd37f24709f41616f3f234980a70fc743


OK - I do understand, thanks. So the "Just configure an older..." sentence could be expanded to explain that the older version goes in b) so that a) will always be the thing that happens.
And yes, it does sound a bit hackish to me - isn't there a way to disable b) completely?


Это не заголовок, а кусок текста what's new, как я понимаю.


Ок, всё равно там места немного. Нужно проверить, что на большинстве девайсов длинный текст поместится.


To prevent cut-and-paste errors, prefer:

if got, want := f.ferr.TypeString, f.expected; got != err {
  t.Errorf("TypeString() returned %s, want %s", got, want)
}

And there doesn't seem to be a need for this function at all, actually. Just inline it.


@dkayiwa  No, I don't have any reason for that. I  think removing this test case solves the entire problem since it seems to be repetitive.
I'll remove this test case in next commit. :))  Sorry it is my fault.


does this need to be global? Does the type need to be named, as opposed to anonymous?


Обсудили там, все ок с этим наименованием.


Cool, thanks!
May be, it should be handled by default on the `poise-service` side? (cc: @coderanger)


When `Comment::Associator` is run on its own source, this approximately triples performance.


fix #58


поменяю на все русский вариант


What is `/captures`? Is this necessary?


Is this line necessary? @jozzi05


GitHub Markdown 的话用 `~~划掉~~`。


Отличный пример, когда новые строки переводятся независимо от старых! В старых строках везде используется Завантажити карти


this looks incorrectly formatted. did it go through gofmt?


Kotlin code is in java folder, so this is not needed.


The comments here have bogons and anti-lockout the opposite to what is in the code below.


Саша, молодец, что заметил. Это в очередной раз подтверждает важность просмотра старых переводов, вместо бездумного копипаста новых.


TeamCity pull requests :: netty [Build 1135](https://garage.netty.io/teamcity/viewLog.html?buildId=5172&buildTypeId=pulls_netty&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new)) Build time: 00:09:39


Я поправлю, как нужно.


Hello! I can take over this project.

/Jocke


Заменю, спасибо.


@djsegal thanks for spotting it :heart:


цукнц укцу кнцук


Closes #281


раво апв45734 457 4573457


This import does not seem needed.


Looks so good!  Super excited!


What about cases like...
```
<<x
#{
  foo # bar
}
x
``` ?


Толстые jar-ы  (ServerAccess-all.jar) используются не только в сборке под винду, но и в остальных сборках (макось и линукс). В связи с этим попрошу сделать такие изменения:

1. Предлагаю убрать win32 и win64 из названий новых целей, классов и переменных внутри `build.gradle`. Заменить их, соответственно, на простые 32 и 64. Это просто битность архитектуры.

2. Целям `prepareRelease` и `macosx` в качестве зависимости достаточно одного `shadowJar64`. Обычный `shadowJar` надо убрать, чтобы он не вызывался лишний раз.

3. Скрипт `resources/debpack.sh` надо поправить, чтобы он находил толстый jar в новом месте (`build/libs/64`).


godoc


To match your signature (and aim?) it must be:
```js
var ifEmpty = R.uncurryN(2, R.pipe(R.always, R.ifElse(R.isEmpty, R.__, R.T)));
```


*always* check for errors. even if the action taken is to panic.


We should find reasonable debugger instead of placing information about placement of a msg. That's the role of a typical debugger.


Ok after quite some searching I found it,

"The NOW function can be used in the following versions of MySQL:
MySQL 5.7, MySQL 5.6, MySQL 5.5, MySQL 5.1, MySQL 5.0, MySQL 4.1, MySQL 4.0, MySQL 3.23"

So it feels safe to proceed :)


I love the way the code is written in a mix of Spanish and English is such a beatiful mix <3


if empty, headers should be empty
for date this test allow to specify which date is chosen in this case


ok


oiihoi


Если `platform` не будет содержать слова "win", а только битность архитектуры (32/64), то `include` можно формировать без условного выражения, сразу подставляя архитектуру в строку:

```
files("lib/$platform/swt.jar")
```

(важно, чтобы это были двойные кавычки)


> Текст 1
_Текст 2_
`Текст 3`
[Текст 4](//3gs.su)
@Текст 5
#Текст 6
# #Текст 7


This appears to be the main entry point to the API, yet it's untested. Isn't this the most important thing to test?


This must be enabled on eng build


Т.к. тут мы пишем на Groovy, можно воспользоваться конструкцией `with`:
```
shadowJar.with {
    description = ...
    conventionMapping...
    destinationDir = ...
    ...
```


i would add an empty line here


is this needed as an external part of the API, or could the fixer dedup in NewFixer()?


Можно так:
```
shadowJar.description + " for $platform platform"
```

или
```
"${shadowJar.description} for $platform platform"
```
(существенно, что двойные скобки)


Actually, the return value should be void.


- [ ] Текст 1
- [ ] Текст 2


no idea to be honest, it has been a long time since then. If anyone wants to take it over ask mathew and maybe something can be done.


can you give the cood ?


I know, it is fixed in the next commit... I added this one too fast, sorry about that...


opts -> options


channels should be unidirectional


Импорты лучше все в одном месте прописать, а не раскидывать по файлу :)


Can be the one from commit msg.


End comment with period.


Line breaking is unnatural. Break after "{" and then one per line.


idem elsewhere


gccコンパイルの名残で、ファイルの松尾に開業いれないとgithubでは警告でるから
ファイルの松尾は改行よろ


##This is a markdown file


@K-Guan 阁下是不是忘了什么……
`Last updated: Fri Jan 15 13:03:59 CST 2016`


Just create a helper method waitForAsyncCompletion() with these two lines and call it, following the pattern we have in many tests.


That's just your opinion. When you read a filename your read bytes from the OS. The interpretation of those bytes is done by python3 and subject to error.
Guessit parses filenames, it should be able to handle python 2 str type.
Further reading on the topic: http://click.pocoo.org/5/python3/


Who is Mathew?


the reason for the weird commit message is http://askubuntu.com/questions/434832/why-does-exclamation-mark-within-double-quotes-cause-a-bash-error


red commit


Right handed system.
---------- x
|
|
|
y


But I'm not sure this is needed as the event listener is synchronous.


Add proper spacing in your if statement.


My bad


Don't use new DocumentModelImpl() but session.createDocumentModel()


ok


Thanks. I'm wondering if we can have everything related to MySQL setup in `before_install`.


Why?


This alias is fine to stay because `3.0.x@dev` is still very much in existence as changes happen following the 3.0.0 release to target 3.0.1.


Actually, this API should not be async *at all*. Caller should do:

workerCount := 100
f := NewFixer(workerCount)
blah, err := f.Fix(certs)
if err …

and the concurrency should be hidden by the internals of the Fixer. If caller is inherently async then *it* is the one that's supposed to make it async. As the API is now it's hard to call sync. Go makes it easy to turn sync API into async, but async API will always be async.


This does not seem to work with mongodb 3.0.6 while it works with 3.0.7


Add proper spacing between everything to make it more readable. Also, keep your `else if` statements on the same line as your closing brackets.


Maybe we could added events to image imports, this is quite reasonable place to look for errors as an end user. Can be handled as a followup.


Good question; I thought it was good form to remove that alias and because it might interfere if I wanted to tag a 3.1?


Are we talking about the Mutable.Vector implementation or the Data.Sequence one? Maybe I would add "...in a particular case **using mutable data structures** ..." or something similar, but the Haskell paragraph seems correct to me.


Please make sure that this bug won't be reproduced after removing 'tmp' prefix https://youtrack.jetbrains.com/issue/KT-4507


```python
print( __import__("unicodemoticon").AUTOSTART_DESKTOP_FILE ) > unicodemoticon.desktop
```

And you can drop the Semicolon ;


Fixed, thanks!


this should be `-test`
eg:
`MIDJE_COLORIZE=NONE ./bin/build-all.sh -test`


Actually I think all of them could, and hence all of them should be checked. I recall mentioning this in previous review comments before the rebase as well.


Забыла написать, что сверстан попап обратной связи.


this is superfluous, as it is checked in the while loop


please remote the label, it's very simple


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


can you do pr?


这里不要改吧，放一行就行了。


Could use just "License: [MIT License](https://opensource.org/licenses/MIT)"


句末少了个句号。


Same comment as in front branch


Thanks! Now it works. I'll merge that in so that people reading the blog can see more idiomatic Erlang code.


get


O


this will create many n+1 errors, I would suggest:
1. find if subnet is a foreman subnet
2. find all records in that subnet that are hosts (via interfaces)
3. check if the entry here is in 2.


wont this break data table initialization?


哦。。。我读书少了


Only one way to find out :)


Should remove the reference to array.h too in this project?


It's because `places` isn't used in the method body at all.  To give a hint to future developers that the variable isn't used Rubocop suggests you either rename it to `_` - a special variable name that means, in ruby parlance, something akin to "ignore me" (see: http://po-ru.com/diary/rubys-magic-underscore/ for a bit more) or to prefix it with an underscore to hint that it's not used, but give some context for what it represents (in the off chance you rework the method so that you do use it).


Once the neatened timeline is ready, you can do a "forced push" on the existing branch. Github is smart about updating the associated pull request, and it's simple for me to discard and recreate my local testing branch at any time.

I think the syntax is `git push --force`, though you may need to name the target repository or even the branch name. I tend to use `--dry-run` a lot before pushing for real, just to reassure myself about what will happen ☺


where is version 0.8 ???


LGTM


Well, the comment says the value is unused.  Perhaps it should instead say that [whatever this is for] is not yet supported on Unix, therefore always return zero?


Please fix the indentation here


I'm not sure about this one, as the numbers do not necessary add up, I feel people might be adding up reservation and leases to know the subnet usage, but if you add hosts it might be confusing (as it might be counted twice).
also I'm less in favor of the icon, as its containers icon


これはどういうことでしょうか・・・？
#4


Please fix the indentation for the next three lines


lololol


memset - принимает третий параметр в байтах, поэтому должен быть sizeof(real_size_) * real_size_


Moi j'aime bien fromage, c'est français, c'est cool. Et puis on est pas des
animaux, on dit pas fixie :0
Le 28 janv. 2016 13:02, "Martin Richard" <notifications@github.com> a
écrit :

> Je préfère FIXIE
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/feston/alphabet/commit/05064fe840a6ff3da2460d3db33c11b9bfe8e95f#commitcomment-15731944>
> .
>


Please fix the indentation for the next two lines


Totally agree with the message - just not sure this is the place to put it. In the cases where this has happened, would a warning on the README or a comment in layout have helped?


What 47 means?


please init this with  NULL.


It should now work with $ npm install -g bookiza
:)

- Marvin | Founder | https://bubbl.in
"Hey Hey What Can I Do" - Led Zep

---- On Wed, 27 Jan 2016 10:32:20 -0500 notifications@github.com wrote ---- Thanks. Glad I asked!
—
Reply to this email directly or view it on GitHub.


`@dev`


Please fix the column alignment


is there a sensible way to parameterize this to run on the correct `test` vs. `spec` vs. `feature` folders?


should this be comparing x2 and x3? Trying to test that keys with different units are not equal, correct? x1 and x2 have both different units and different value -- I know that should not be equal, but x2 and x3 seems like a cleaner test.


It might be worth using the stanza from whitehall (shortly to be in govuk-rails-app-template) https://github.com/alphagov/whitehall/blob/ec3932e8194a12b2b681205cffb829a6e82718bf/jenkins.sh#L52-L60


idea is protected constructor in combo with two static methods be able to create an instance for new and persisted tokens


ok @sahildua2305  sir i will keep this in mind !


необходимо сделать initial_size_ const и это строка не нужна!


Oooh. Clever, clever, clever.

Perhaps we should avoid skipping over heredoc nodes?


maybe a better description than "units work funny"? As the test above shows, units *are* compared -- so this comment and units being in `eq_ignores` is confusing.


This change breaks graph_view.php for default admin user. When clicking "graphs" it opens page without loading the rest, shows no graphs and just a simple htlm

![image](https://cloud.githubusercontent.com/assets/16924668/12646094/7b247614-c5e7-11e5-84d1-e4ec2e5f3443.png)


Необходмио скопировать данные из copy, предворительно проверить не равно ли 0.


The PHPDoc incorrectly refers to the first argument as `$buttons`.


Have to find a way to do the same, but with the HashMap. Working on it...


remove space home slice


Will that still lint master? Do we want it to?


Please fix the indentation


так же проверить, if(copy != this)


remove space


is 3.2.5 already available in our official Jarvis addon repo? Latest I'm getting offered is 3.2.3...


Please fix the indentation for the next two lines


Wouldn’t that be the same character soup with more (parentheses) added to the mix? Or did you have something else in mind?


Im glad the hours i spent adding in all the future item and block id's and classes are helpful, they just need to be uncommented when they are added to the game :D


где delete[] buffer_?


Yes it is my opinion, but it's not one formed without thought, nor am I alone in that opinion.  Yes treating everything as unicode has issues, but so does treating everything as any particular type.  Conversion between types when you don't know what types to convert to and from is always going to be an issue.  I could also just as well point to articles where Python 2 strings cause just as many issues.

I don't want to turn this into an argument, but I think an option to enforce a string type would be useful, as it's just that an option.


@timpaul suggested that maybe we should have a banner at the top of the visible page, but that would seem to interfere with user research?


Remove semicolon, please.
Maybe will be better export Fabric integration to class? Like this:

https://github.com/elpassion/simple-github-client-team-door/blob/master/app/src/main/java/pl/elpassion/door/githubclient/application/GithubAplication.kt


- The requested package barryvdh/laravel-cors could not be found in any version, there may be a typo in the package name.


Gute Ergänzungen. Perspektivisch sollten wir dafür vielleicht ein wiki aufbauen und die README.md eher für Basisinformationen nutzen (was ist das hier, was kann das, brauche ich das)...


I do not quite understand. There is no bug report for this, but this peace of code exists that fixes one particular case of zero. So I guess you want me to do the other ones? I do have a working solution for all cases, will write a test an pr that


Instead of doing `DeepEqual` can't we just traverse tags and check if ids changed? I'm aware it might not catch all the metadata changes but do we care about it?


LOL three code reviews for 3 line diff


We have a convention like `// drop10.10`. I think we only talked about what it was for, but not necessarily when it should be used. I think if there’s a call to `isSomeOS`, we can find it. The comment is for the less obvious ones.


If you go by the latest revision of the standard these checks shoud also include a check of Byte 9 (Item Designator) of the key and issue the invalid file warning only if the byte is not 0x0E, i.e. the key is not Class 14 key that are allowed to exist before the Preface.


Yeah, actually once I figured out what it'd be then it would be exactly the same just with more parentheses. Yeah, maybe you can call me a ()() lover


It's the _process_, man.


let's move this to either email or gitter.


I replace "barryvdh/laravel-cors" to "barryvdh/laravel-cors": "0.*" and installed version 0.7.3

on laravel 5.2


Besides this is costly operation and needed only in the else, shouldn't it be there then?


стоит добавить memset( new_buffer, 0x00, sizeof(new_buffer_size) * new_buffer_size );


I want you to do this PR because this is not a proper fix but more a
workaround.

2016-01-28 14:53 GMT+01:00 Fabian Neuschmidt <notifications@github.com>:

> I do not quite understand. There is no bug report for this, but this peace
> of code exists that fixes one particular case of zero. So I guess you want
> me to do the other ones? I do have a working solution for all cases, will
> write a test an pr that
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/231ff5213d31fdf16a28b80c79331032fd638054#commitcomment-15733121>
> .
>


Couldn't we just use `LONG_MAX` that is define in `limits.h` ?


It means the size (in character) of the adapter name and a separator.
An example is: ``\Device\{754FC84C-EFBC-4443-B479-2EFAE01DC7BF};``
Its size is 47.


добавить memset()


We should migrate over all the existing associations to avoid data loss in production.


test readme was added


@DidierBesset why are you checking on syntax whereas you will actually check everything with semantic a few lines later... you can check name of method based on the symbol and check that owner of that method (symbol.owner()) is of the correct type.


необходимо логигу работы, if(i == real_size_) push_back(value);


@sterrym are we sure that the merge_vars method takes any params?


I know that, that's the problem.


Good idea!

I'll take a look at it this weekend, super swamped at work right now.


These kind of tweaks (cleaning up the spacing) make me very happy. :)


1. Создать абстрактный базовый класс Figure с виртуальными методами вычисления площади и периметра.
Создать производные классы: Rectangle (прямоугольник), Circle (круг), Trapezium (трапеция) со своими функциями площади и периметра.
Площадь трапеции S = (а + b) h/2.
Площадь круга S = R^2.


Looks OK to me


you guys should be paying attention in class! I SEE YOU! 😎

On Thu, Jan 28, 2016 at 8:55 AM, Lucas Schuermann <notifications@github.com>
wrote:

> It's the *process*, man.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cs3157/recitations/commit/5b84c52c0fa11e3f3eb0450baf66318215e0fc8f#commitcomment-15733169>
> .
>


This thing raised Javadoc warnings @lauraluiz


Убрать "Html Academy"


Eh… trial and error? It was over a year ago when I first did that. I only made it public recently. :smiley:

I’m open to discussing it, or making it a hidden pref. Or maybe even a slider?


don't you have to do `#define _GNU_SOURCE`  in the cflags ?


If you only document `@deprecated` then warnings appear for missing `@return` and `@param`


I agree with Diaoul. In fact, after reading more stuff about the subject, it is possible and perfectly fine to use native python 2 `str` and still display a treat special characters properly. It can be more complex when manipulating those string using `re` module, `in` operator and things like that.

Blindly using unicode in a python 2 project can also lead too some string hell and strange issues, because many API from python 2 modules and third party dependencies doesn't support them properly.


Try to go down with the amount of parameters here, it's killing me. I can't parse that many parameters without spending more than 30 mins looking back and forth :cry:


I think so.


i think you have to define GNU_SOURCE before the include.


Thanks


Dear Greg,

I'm sorry to bother you. But you seem the right person to ask. For some reason Safari crashs immediatly when I try to change a soundcloud url on any vertexshaderart example. Same happens with my second Mac.

As I'm right know trying to create my own little soundcloud player I have the same problem in my application.
it seems that Safari won't accept any replacements of the src=""-tag in the <audio>element while an other src is already playing. It works very well in Firefox and Chrome. But Safari is killing me.... !
Do you have any Idea how to fix this?

Maybe you could give me a tipp? (christian.losert@icloud.com)
Since we're on the same boat here I'd love to solve this issue...

Best,
Christian


- [ ] この書き方だと、「デフォルト設定」にすることを強く推奨しているように読み取れてしまいます。なので、ここでは、「`<sec:form-login>`\ タグを使用して、フォームを使用したログインに関する設定を行う。詳細は、xxxx参照されたい。」みたいな感じにしてほしい。


There could be a comment referring to the issue number.


One additional info: with "element" I mean the <audio> element, of course.