98318570_RemoveRadianceCheckbox (macumber) - i386-Windows-7-VisualStudio-12: OK (2196 of 2215 tests passed)

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


test


@flavour Please review


@ropilz hi. i get this bad error while trying to build a template using adobe build for android:

error ("plugman-npm install --platform android --project /tmp/gimlet/1648/e9e85f5e-4232-11e5-aaca-22000b0e956b/www_android/project --plugin /home/ec2-user/gimlet/var/plugins/2f48747e-3646-11e5-82ff-22000b609528": Failed to install 'com.medlei.pushplugin':Error: Variable(s) missing: APP_ID, CLIENT_KEY
    at /home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/cordova-lib/src/plugman/install.js:299:23
    at _fulfilled (/home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/q/q.js:787:54)
    at self.promiseDispatch.done (/home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/q/q.js:816:30)
    at Promise.promise.promiseDispatch (/home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/q/q.js:749:13)
    at /home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/q/q.js:557:44
    at flush (/home/ec2-user/.npm/lib/node_modules/plugman-npm/node_modules/q/q.js:108:17)
    at process._tickCallback (node.js:442:13)
Variable(s) missing: APP_ID, CLIENT_KEY
)

please help me.

also, how can I get installation object id?


Most of these are checking for x86, not x86_64, which makes the check below seem somewhat redundant if you could just directly detect only x86_64. And what's up with `APPLE`? Why can't we detect using the CPU string on OS X too?


まあ今くらいの規模ならこれでもいいけど、最終的には `@user_profile` に答えを尋ねるほうがいいのではないかなあ。


Also, I think a generic define for both x86/x86_64 would come in handy.


Can you see if this refreshes the HIT data right after creation, so it's not blank?


This is meant to check for `ARCHITECTURE_X64`, right?


Added this line to throw up the error if something is thrown.


`"不正解! 答えは#{@user_profile.answer_name}です。"` とするのがおすすめです。

ちなみに https://github.com/cookpad/styleguide/blob/master/ruby.ja.md#strings 曰く、
> [SHOULD] 文字列リテラルに対して String#+ を用いて他の文字列を連結してはならない。式展開を使用すること。


Oh. Right. Good catch.


This makes the form display properly using BS3 styles without the need for the clearfix.


Moved this into the worker popover.


good


@kaichen 哦，我不知道这个事情，那可以去掉了


saw approval from shino
at https://github.com/basho/riak_cs/commit/f28ce534f6c94c03184fa92425bafdff1351868b


merging basho/riak_cs/feature/support-stream-listkey-on-inspector = f28ce534 into borshop-integration-1207-feature/support-stream-listkey-on-inspector


basho/riak_cs/feature/support-stream-listkey-on-inspector = f28ce534 merged ok, testing candidate = fc3eaa02


seems ok


@paf31 I could reproduce the error. Yes, definitely it's an issue with the checker, I just removed it from make and the example worked fine.


+For support, we encourage the use of the **GutHub** issues system.

We need to add a link to the issue system for customer to reference. And there's a typo in **GutHub**


Hmm, yeah. We could move the iterations_limit break outside of the conditional (making room for creating an error if that limit is reached), and calculate the relative cost change before / in the loop instead of in the conditional?


This is taken from Dolphin. The general logic flow is: First, detect that we are on x86 (and APPLE being defined means we are on an Apple system, which should be x86); Second: detect 64-bit support based on address width. Feel free to suggest something better, but some quick Googling didn't yield much.


101162692-IddBiquadriaticCurvesObjList-4856 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


You are absolutely right, the comment above even states my original intent. Thanks for reporting it.

Luckily pasting still falls back to text-mode, but I can guess it left you puzzled for the last few months.

Sorry about this, I don't remember what I was thinking.


There's no place in code with a (non-64-bit) x86 define would currently be used, so I didn't bother defining it.


Hi Dan, According to your suggestion, I check the code style by using tools you mentioned. Thanks.


Yes, this is fixed in a later commit


93653360-ResetVarsAtEndOfWarmup (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1275 of 1275 tests passed)

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


インデントの修正とかは一つ前の「文法ミスなど細かな点を修正」に入れたほうがよかったかもですね


@mrinaldhar : I merged this version for now. But theres a much better way to do this even without using `rq workers`.
The idea is to directly connect from the client to the nodejs wrapper at `pipeline.ilmt.iiit.ac.in` over socket.io. The client will maintain two parallel socket.io connections, one to the translators desk server and one to the pipeline server.

Will discuss its implementation in detail when we meet.


In which cases, are `model` and `entitySet` null in `entityInstanceContext`?


@terrafrost So this is an actual bug fix?


8:30 pm fixing merge - no changes


具体看 https://www.ruby-lang.org/en/news/2013/02/24/ruby-2-0-0-p0-is-released/

> The UTF-8 default encoding, which make many magic comments omissible

可以用下面这个去刷掉所有的 Magic Comment

```
find . -type f -name '*.rb' -print0 | xargs -0 sed -i '' '/^#.*coding.*[uU][tT][fF]\-8/d'
```


hashの記法はできるだけ統一したい感


This should be `false` (without the quotes)


この修正はコミット分けたほうがいいかもです


and this implementation warrants the use of background workers, as the overall timetaken by `pipeline.ilmt.iiit.ac.in` to process a request can get to around 10-20 seconds in the worst case (like in a pipeline using fullparser), and that could exceed the internal default timeout of `urllib2`


This should be `True` (without the quotes), or just remove all the `== "true"` part.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1477


Is that more Seam code for us to maintain?  Oh well, if we must.  But please make it clear where each file comes from.

Isn't `zanata:` a bit long?  Can we use `z:` or something?


This isn't actually fixed


On Windows, this is `AMD64`. On Linux it's `x86_64`. Some googling indicates it's also `x86_64` on OS X but it would be better if @archshift could test for sure. So you can just match against those two.


Meaning a non-beta version; we don't have any for this one because it's one of the newest ones.


后排强势围观~~


@albertdev
Thank you for fixing it promptly!
I rarely use block-wise selection, just noticed a problem a few days ago and decided to debug it today.


Just changing the option name sounds fine to me. Such interfaces are harder to change later.


更新


@juarezpaf done ;)


@Novik

`strip_tags()` only protects against the most trivial XSS attacks, like \<script\> tags. This code is still vulnerable to wide variety of more complex attacks like like `strip_tags('<strong onhover="alert(\'XSS\');">Hover this</strong>', '<strong>');` will still succeed.

Torrent comments are not expected to contain html as they're displayed in a large number of non-html torrent clients. If you want to avoid XSS concerns here, you should just replace [all of that logic](https://github.com/Novik/ruTorrent/blob/bd8cfa524318931eeb11811b775bd2a623b1f953/js/webui.js#L2166-L2183) with something simple and safe:

```js
$("#cmt").text(d.comment);
```


101235614-Fix-AFN-Error-Output (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1797 of 1797 tests passed)

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


No this is not seam. This is based on seam's migration project at https://github.com/seam/migration.

```zanata``` was used before therefore I followed it.


101235614-Fix-AFN-Error-Output (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


👆上面那些 pyc 文件能不能 ignore 一下……


O_o. When I used to experiment (time ago) with this class, GetColumnCount hasn't called for me... Looks like I did this without proper attentiveness... Did it fix sorting or crash under Linux?


develop (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1274 of 1274 tests passed)

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


@bcho 初始化的时候忘记添加gitignore了。。马上弄！


Ermm. Your private key probably shouldn't be on github. (sorry I'm stalking your github history lol)


I could be wrong, but I feel like we still need this `_super()`. In my branch, which I know is behind this one now, without this call I couldn't pass in an `elementId`.


On OS X it's `x86_64`.


@uagarwal744

You must always do this using `environment` variables. And not hard code secret keys / passwords here.


ok


Thanks Deni!


typo "any of regions"


PHP with pthreads has issues with static object class properties. Therefore, avoid using them to avoid having errors.
If you are only targeting to get rid of errors, you'll probably have no issues during testing but this will happen when you test your code:
![xkcd.com/292](https://imgs.xkcd.com/comics/goto.png)
Or when you ask someone to fix your code:
![xkcd.com/1513](http://imgs.xkcd.com/comics/code_quality.png)


wtf


@tatosjb confere o que eu fiz para ser o nosso seed, comece lendo o arquivo [index.js](https://github.com/fabioelizandro/tap/blob/2e9e457b34f3ddfde359eec35e50e49e4311139a/server/config/seed/index.js) que é o arquivo que vai ser chamado. Depois confira os arquivos [state.seed.js](https://github.com/fabioelizandro/tap/blob/2e9e457b34f3ddfde359eec35e50e49e4311139a/server/config/seed/state.seed.js) e o arquivo [city.seed.js](https://github.com/fabioelizandro/tap/blob/2e9e457b34f3ddfde359eec35e50e49e4311139a/server/config/seed/city.seed.js) que são os seeders, eles é que vão conter os dados. Os demais objetos a gente pode refinar e publicar elas em um pacote npm


101162692-IddBiquadriaticCurvesObjList-4856 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1797 of 1797 tests passed)

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


@mulby Sorry for discovering this late, but: this doesn't work.

When I run any tasks on the devstack with this in place, I get:
```
  File "/var/lib/analytics-tasks/devstack/venv/local/lib/python2.7/site-packages/luigi/hadoop.py", line 222, in track_process
    proc = subprocess.Popen(arglist, stdout=temp_stdout, stderr=subprocess.PIPE, env=env, close_fds=True)
  File "/usr/lib/python2.7/subprocess.py", line 679, in __init__
    errread, errwrite)
  File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child
    raise child_exception
OSError: [Errno 2] No such file or directory
```

The reason why is a subtly of sudo/bash (preserving the `$HOME` variable):
```
vagrant@precise64:~$ sudo -u hadoop /bin/bash -c 'echo $HOME'
/home/vagrant
```

You can fix this by changing the `sudo` command to `'sudo -Hu {0} /bin/bash -c {1}'` or `'sudo -iu {0} /bin/bash -c {1}'` instead of just `-u`.


You might notice... I don't use an editor with a spell check.


Don't thank me. Thank @swiftSwathSee; the guy who made wrote that edit and submitted the pull request. ahhaa.


Creating application form for the careers, Creating register form form edit view, Redirecting non logged in person to the register form, Redirecting logged in person to account form from register page, Preventing non registered user from submitting the application form,

Application form data are saved to database for the users, Creating login form, Creating new role candidate for registered users, Verifying logged in user
Redirecting logged in users to the account page, Creating the new row action for all user in admin with name application, Displaying the application form at the admin, Fetching the content of user submitted in the application


How does your solution compare to:

1. keeping the current  behavior to initialize the observers before any user-properties are processed
2.  then immediately calling:

```js
for (var prop in this) {
   this.notifyCurrentValue(i);
}
```

(Or some more efficient variation.)

It seems like the same behavior except that the hash passed to `refreshRendering()` contains the original values of the properties, rather than the current values.

PS: Your solution could presumably also be modified to pass the old values to `refreshRendering()`, by calling `refreshRendering(this.constructor.prototype)` rather than `refreshRendering(this)`.


@iksaku :+1: (only for MySQL; also, INSERT OR REPLACE INTO doesn't seem to be implemented in MySQL)

Also, I believe you forgot this:
![xkcd.com/327](https://imgs.xkcd.com/comics/exploits_of_a_mom.png)


```cpp
if (mapid == 654 || mapid == 648)
    return CONTENT_1_60;
```


@roughani I really like that idea. I'll add a contributing section to the README to encourage small commits and thorough commit messages. I think this project is a great candidate for being used in a tutorial. If you want to take on turning the commit log into something a little more user friendly, I'm happy to try and explain the existing stuff in more detail if you need.


Test


I'll do it!  Thanks for letting me know.

Sent from my iPhone

> On Aug 13, 2015, at 1:42 PM, Albert Bori <notifications@github.com> wrote:
>
> @iosdev-republicofapps Can you bump the cocoapods version? Or point me to where I can learn how to do it? I couldn't find anything with a quick google.
>
> —
> Reply to this email directly or view it on GitHub.
>


Shouldn't the optional interface take care of that?


even i use Server::getInstance() because $this->getServer() dosent work for
me
On Aug 14, 2015 12:20 AM, "PEMapModder" <notifications@github.com> wrote:

> PHP with pthreads has issues with static object class properties.
> Therefore, avoid using them to avoid having errors.
> If you are only targeting to get rid of errors, you'll probably have no
> issues during testing but this will happen when you test your code:
> [image: xkcd.com/292]
> <https://camo.githubusercontent.com/b5dece381492c332460d289021254e6b88ae5476/68747470733a2f2f696d67732e786b63642e636f6d2f636f6d6963732f676f746f2e706e67>
> Or when you ask someone to fix your code:
> [image: xkcd.com/1513]
> <https://camo.githubusercontent.com/e815b9b967ffa0d4956fc74248adaa7ea05aa8c6/687474703a2f2f696d67732e786b63642e636f6d2f636f6d6963732f636f64655f7175616c6974792e706e67>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12703953>
> .
>


@valib @SubJunk @skeptical This is causing #616 to fail when I try to build with Maven. Setting it to ```1.7``` fixes the problem. Does the ```maven-antrun-plugin``` version equal the java version? It so, ```1.8``` is wrong for ```master```.


@SavionLegendZzz because you did something wrong. It always works. If it doesn't "work" for you, that means you did not use it (not properly).


@icyflame
If there is any problem, should I send you the files on mail?


@sellout: I obviously need to review to review your code better. `attempt` was kind of the whole point of this function, and I just spent a half hour finding this spot where Task.fail was happening.

The good news is now it's easy to fix.


@uagarwal744

No, there is no problem! :smile:

I was just reviewing your code, and giving you feedback on it.


i did $this->getServer()->broadcastMessage()
On Aug 14, 2015 12:37 AM, "PEMapModder" <notifications@github.com> wrote:

> @SavionLegendZzz <https://github.com/SavionLegendZzz> because you did
> something wrong. It always works. If it doesn't "work" for you, that means
> you did not use it (not properly).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12704045>
> .
>


What is your context?


This is a bug.
The `class_p` here should only be use for the check, but it'll accidentally override previous mapped skookumscript class.


PlantOperationSchemes (kbenne) - i386-Windows-7-VisualStudio-12: OK (2209 of 2235 tests passed)

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


@thibaultCha I hate myself for this solution. This allows for flexible schemas that allow flexible keys not defined in the schema. For example right now the transformation plugin accepts data in the following format:

```
value.add.headers=helloheader:some_value
```

While with this change we can accept:

```
value.add.headers.helloheader=some_value
```

I also need this for the response rate-limiting plugin.


im sorrey it tried find forum but not sure where yet ,but i cant put the script for everyone, its a rare script almost nobody has it. i just need to fix problem with override so the rs client can open. do you have an enmail i can contact u on or anything?  i really need the help without me puting out script out. if u could help me on teamviewer or somthing? would been great


Having not tested this fix... is it possible to use `uint8_t ` vs `char` (for more precise semantics... though, acknowledging the historic/understood definition of `char` might be today's best solution)? Also, does this patch translate well to other platforms? (e.g., does it solve issues, or at least not create new complaints, in GCC?) Thanks!


@PEMapModder What do you mean?
On Aug 14, 2015 12:41 AM, "PEMapModder" <notifications@github.com> wrote:

> What is your context?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12704072>
> .
>


98318570_RemoveRadianceCheckbox (macumber) - Win64-Windows-7-VisualStudio-12: OK (2193 of 2215 tests passed)

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


this was pretty inefficient. fix this in v 1.3


Test add text.


I really didn't know what to do here.  I couldn't find a convincing way of testing this `writetape_free()` function (since it's not in the main runpath; normally it just goes to `writetape_close()`.  I'm 99% certain you're going to tell me to remove these lines, but on the off chance that I hit the 1%, I wanted to include the same clean-up code in here.  It certainly shouldn't do any harm, since `multitape_write_internal *d` is initialized with all 0 so doing the free() should have no effect.


v2ex观光团路过 强势鄙视


typo


Actually yes - can you submit a pull request?


Add abit of work here to bubble up the still open windows, just add another flag to them and reset all set flags and while updating set them. new windows should have the flag set automatically. once done, bubble them up. I don't expect many to close instantly so a simple bubble should work... maybe after the add, move up to the last one that's set or first... should work like that and probably won't be to cost heavy.


You are tracking wombat256.vim in vim/colors but not peachpuff.vim


fast-forwarding develop to borshop-integration-1207-feature/support-stream-listkey-on-inspector = fc3eaa02


Why didn't we call it GetSections?


.Any()?


peachpuff ships with vim by default.


This is working well for me really nice to not have to switch to an ios device to manage followers


Sorry it took me so long. I've made the fix. Anything else?


When I try using GCMReceiverDelegate (line 20)  x-code says that  its an undeclared variable usage. I have followed all the steps provided in the "QuickStart".


Thanks for this - it's close!

We still should use song.id for it though since we are storing the id of the song in the database.  Unfortunately the plug_id passed via the API is actually only unique when it's the same user, playlist and youtube/soundcloud ID).  Two people playing the same exact video will return two different song ids, which kills the usefulness of the identifier.


How is the `php maintenance/update.php` script working at this point when no `LocalSettings.php` file would exist? Wouldn't it just die with something like:
```
A copy of your installation's LocalSettings.php
must exist and be readable in the source directory.
Use --conf to specify it.
```


@HaoK filed a bug for this to get it reviewed - https://github.com/aspnet/Configuration/issues/269


@SavionLegendZzz
> where did you put that code in?


my OneInTheQuiver plugin
On Aug 14, 2015 1:11 AM, "PEMapModder" <notifications@github.com> wrote:

> @SavionLegendZzz <https://github.com/SavionLegendZzz>
>
> where did you put that code in?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12704194>
> .
>


Why had we changed function parameter names? This increases number of changes in function body. Can we make this part less intrusive?


How did you find out which registers to write to? datasheet?


@SavionLegendZzz thank you so much... * reads the code from your mind*


Thanks much


@PEMapModder Oh Sorry I Didnt Know U Wanted Too See It
On Aug 14, 2015 1:15 AM, "PEMapModder" <notifications@github.com> wrote:

@SavionLegendZzz <https://github.com/SavionLegendZzz> thank you so much...
* reads the code from your mind*

—
Reply to this email directly or view it on GitHub
<https://github.com/DarkRealms/HardCorePvP/commit/1aeadb1cf580a74de89cb2ed0af4c7fe8462f604#commitcomment-12704213>
.


I can fix it for u, add my skype live:riwu0730


I'm not sure what "(you)" means in this scenario. The change could include making what "(you)" means clearer.


Ah, I figured it out. 98336958584e22a75668a69553d61e2739f991e3 makes a symbolic link as part of the `Dockerfile`, you do the same thing in the entry file a few lines before this but that is essentially useless.


@JahBrewski ermmmm good call!


It will be removed in next blob package


Of course. But a lot of trackers use an url's in the torrent's comments. This is a very useful and can't be simple removed.


As a side note, I also cleaned up the missing carriage return at the end of the file.  Dunno how that got there, and it wasn't intentional on my end, but made it's way into my commit.


can there be a generic/plugin type?


я собрал название предметов и оценки по ним в 2 массива для удобства


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1793 of 1797 tests passed)

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


Useless


No, it is not called "hideNegativeValuesEnabled".
It is meant for making sure you have the 0 point in your axis even if your values are well above that.
It shouldn't limit the visibility of negative values.

You can use a custom axis minimum/maximum to constrain the axis.
And you can use `startAtZeroEnabled` to make sure that the axis starts at zero for positive values only, or ends at zero for negative values...


:) это я практиковался интерфейсы писать :) в данном случае, интерфейс который отвечает за средний бал по сессии, екзамену


RecBook :)


Old code gave a misleading message, should be based on the arguments port_low, port_high


Ffor consistency only. Two functions are close in function, but had different signatures. Seemed weird.
The renaming isn`t needed for it to function.

14 серпня 2015, 08:13:38, від "Myroslav Opyr" < notifications@github.com >:

Why had we changed function parameter names? This increases number of changes in function body. Can we make this part less intrusive? —
Reply to this email directly or view it on GitHub .


Thanks, it works now.


@huangp Okay, well, wherever it's from, we should say so at the top of the file.

@carlosmunoz I assume you created the `zanata:` tag prefix, any objections to shortening it to `z:`?


@nigeltao Do you have any planed roadmap? I want to know where go/leveldb is heading to and where we can help.


This resolves #698. See comments there for tips on determining the latest OT Taxonomy version using our APIs.


Oh ya, I wouldn't treat riot.py as too much of a startup of file.  I was mostly using it for playing around and testing.


Haven't tested, but `v2.1.1` should be good :grin:


If the port can be configured to anything by any client, this no longer makes any sense. Other old clients will be using HTTPD_PORT, as any configurable client will pass HAS_PORT below. But we should probably document when HAS_PORT was added if we want to break compatibility.


Shouldn't have committed this, it's a temporary measure to deal with the difference between my production and development environments.


Not only german, other languages suffered from too tight buttons/text areas as well :P
Thanks.


Sounds good. Generally I think a user does not want to make any configuration changes on the main project to get an external library running.


It earned us 200 points according to a few sources. The value of the signfastest problem that was created in result of us finding this weakness was only 100 points, which means no other team could get 200 points by solving the problem.


hi @ooq , Thanks for your new sub-task. I have reviewed this source code. LGTM. +1


What's happening with the indentation here?


I refactored this after pairing with [mrhamburger](https://github.com/mrhamburger "Mr Hamburger").  My first pair programming exercise.  Thanks Mr Hamburger!


aaa


If you want doing this kind of clean ups in code, do them in separate commits, not to increase the scope of the essential change.


Pushed a fix! Thanks for finding this Nate - it somehow did not cause any issues with our test code over here.


hmmmm


I wonder if we really need/want a separate class for this, instead of just setStartLoopPoint/setEndLoopPoint or similar.


Yep, my bad. Kept to fix later and skipped. Will add necessary strings to lang files today.


This should be a typo?


This should mitigate issue #5, by implementing timeouts in curl.


Please change state name. Make it cancel & schedule.


It would be nice if this could be pushed to PyPI


Yes, we included them for any future implementation. Currently they are guarded by #ifdef constructs using CONFIG_INCLUDE_LEDK and will not be compiled for current designs.

Hope it is fine. Please let me know if you think we should remove it.


Removed the MigrateCompleteCommand and instead doing work of it in CheckVMOnHostCommand


Añadi una comprobacion para cuando tablas en bd no estan relacionadas. porque en principio no
se borraban los comentarios, Aunque ahora ya si.
En local funcionaba ok y salia aviso de tabla sin asignar, pero en heroku se queda colgado al
entrar en estadisticas y no da error en el heroku log, asi que me plante y solo lo deje como
comentario


fwed


d


Ping @Wout-van-Dongen


```ruby
      def column_names
        klass.attribute_set.map(&:name)
      end
```


No problem.


passed!


Did adding the test vector change performance?
If yes, you should have put it at the end of the list, not into the first position.
Or is there a specific reason why it is in the first position?


the ctor of `bench_interval_data()`, you could use in-class member initializer (http://www.stroustrup.com/C++11FAQ.html#member-init) as i put in last comment (nuked by the new commit). it save you some type and easier to understand.


thêm dấu space sau dấu ,


@aab-odoo  Done.!


@roughani Haha, are you trying to tell me that my commit message wasn't descriptive enough? Should we suggest using multiple -m flags when making commits like below?

`git commit -m "Title..." -m "Description..."`

@jesserosato Are they any intro to backbone and marionette resources that you can link to in the readme to help backbone first-timers like myself get up and running? I was planning to look over this [three part blog series](http://www.smashingmagazine.com/2013/02/introduction-backbone-marionette/) as it appears at the top of a Google search. The only thing that's holding me back somewhat is that the posts are over two years old.


@aab-odoo Okay as you say. Done.!


bỏ hết ngoặc chỗ eq đi Long êy


add new line


Watapon and Infernity Archfiend can activate Special Summon's effect in Damage Step.

Q.ダメージステップ中に「黒き森のウィッチ」の効果で「ワタポン」を手札に加えた場合、手札に加えた「ワタポン」を特殊召喚できますか？
A.ダメージステップ中に手札に加わった場合でも、「ワタポン」の効果は発動して特殊召喚できます。

Q. ダメージステップ中に手札が０枚の状態から「インフェルニティ・デーモン」をドローした場合、ドローした「インフェルニティ・デーモン」の効果を発動できますか？
A.「インフェルニティ・デーモン」をドローした処理で終わっていれば、ダメージステップ中にドローした場合でも、「インフェルニティ・デーモン」の効果を発動して特殊召喚できます。


@xavriley, @samaaron I'm still working on this to build a minimal viable app – with editing/running code, rudimentary settings and a package. There's several things I'd like to do before opening a PR:

- Stream the logs from the server to the frontend app
- Let the user edit basic settings
- Write some installation / building instructions

I'm excited to do PR next week and discuss whether it's a viable approach. :cactus:


(2,2) (3,2) (3,3) (4,2) (4,3) (4,4) ...
so you mean I don't need to check when i and j is the same?


Are you going to remove [hphp/scan-methods](https://github.com/facebook/hhvm/tree/master/hphp/scan-methods) folder, too?


Could you be more specific? That is, what prompted this question?


91977808_LBNL_NewVRFModel (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1799 of 1803 tests passed)

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


Why is this `TargetsExclude`? Wouldn't `TargetsInclude` make more sense?


The intended use  was to be able to choose which verb you want to inflect. In the demo there are 3 different kanjis for "hanasu". You might want to first display them and then let the user inflect a given one. So, yes, both cases are required.


101235614-Fix-AFN-Error-Output (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


Does lighting depend on submodels or entities?


@iksaku @PEMapModder https://github.com/PurePlugins/PurePerms/commit/244d327e0a9f5bcd5c86c8be429d76c04fe351f1


Thank you for the suggestion.  Changed so in bb7de02.


PlantOperationSchemes (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2206 of 2235 tests passed)

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


You can certainly create your own `SpawnType`, just you'll have to expose it properly. Also, please note that this is still a heavy WIP, so the types available may be final.


If we want to do that, it might be easier to just mirror https://github.com/PowerShell/DscResources and then we can control the submodule references inside, rather than cloning each thing individually.


Because we don't know which files we want to include beforehand. What will be passed for is what is in the current working directory minus files in the test/ directory and files that have an extra Main.main function.

In short, we can't know all the files we want to include beforehand but we know those we want to exclude.


Shouldn't this still be a Closure?


Thanks for the inputs. we will make the changes.


Shouldn't there also be an @throws \Throwable here?


See also: http://ryan.mcgeary.org/2011/09/02/do-your-commit-messages-suck-rockymtnruby/


I think https://github.com/urbanslug/wai-devel/issues/1 is a better elaboration of my thoughts here


Thanks for pointing this out. See #21


Great visualization, and you used it perfectly :+1: :100:


So it seems alright to me, just few questions.
- I think BackBone has `_changed` property, shouldn't we be using that to see for pristine/dirty?
- How would this work with conditional logic fields? I am not sure if this affects that but just came to mind.
- Also at some point, we should only include definitions thats been used in test, and remove rest to avoid future confusion.


Extra SPAAACEEEE! Please remove it :)

And also could you change `@return boolean` into @return bool` in a docblock?


`nextInt(max)` gives values from `0` to `max - 1`

By checking `rand <= chance` where chance is `0` to `100` you are making off by one error.

For `chance = 100` resulting drop rate is 100%, it is same for `chance = 99`.

Also setting `chance = 0` will result in 1% drop rate.


I still think that the return type of `CSwordModuleSearch::unindexedModules()` should still be `QList<CSwordModuleInfo *>` a.k.a. `BtModuleList`, the type of `unindexedModules` should be `BtConstModuleList` which also has a `BtConstModuleList(BtConstModuleList const &)` constructor. This way we do not need the ugly cast.


地方


We want to avoid dependency on the kernel headers in target module. Now the target.h includes the led.h for the led enums.

What is your opinion?


Going to make a tiny change in wording here. I learned some new stuff about what `git remote`s are and that's kinda helped :smile_cat:


This fails with "error Undefined type 'T' extend VectorList <T>"
Documentation does not mention anything about this limitation. Should this work ?


If you aren't going to use record_source in the identifier, then don't add it inside this functiom


Awesomeness! Thank you for putting time and thought into this :bow:


Record Source needs a String supertype


What's the reason for this restructure? It doesn't seem to change anything.


Error!!!!


Yes, you should exclude these.


`CompileShader`


properly* destroy interval timer


We should use hyphen-bound text or a role name here


No problem with the shortening of the prefix


Still an 'or' to remove. oe_button classname could have stayed, but it's ok to remove it


Why doesn't `JitCompiler` simply return an instance of `JitShader` instead of having this odd indirection?


OKです。


Today can we have a call to discuss about the Task Unlocking? Thinking of a way to restart the `task` with the executing `block`


not sure, it doesn't look very useful but might take quite some time to remove from all pages....


Work it


Only when you have virtus :)


@shadycady  No ETA.  It will happen when it happens.


неверные слитно надо


This is not a modal so you can revert those changes.


Bah humbug...

Tim 'mithro' Ansell
On 14 Aug 2015 7:58 am, "Henare Degan" <notifications@github.com> wrote:

> That hackfest @mithro organised was a pretty massive contribution aye
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/openaustralia/planningalerts/commit/4bfeff89f93d0dd21c61cbbccd35da2330d525cb
>


this break all images builds ! rcu now missing in all images


Ok my bad, this is the same template as for DebugManager modals.


@aab-odoo Its Done.!


So its ok to update it, but please
- Replace the a by a button
- don't forget classname btn on both buttons (also apply btn-sm on none of them or both of them)


use std::begin(this->vpu.code_ptr)


This should work, but it is not properly implemented in current compiler.
Related issue:  https://github.com/fasterthanlime/rock/issues/830


I'm commenting on a particular line. It looks like original indentation was tabs but you use spaces instead.


use std::end(this->vpu.code_ptr)


Ah, thanks Henare.  :)

On Fri, 14 Aug 2015 at 15:58 Henare Degan <notifications@github.com> wrote:

> Very deservedly add @otherchirps to the list of contributors
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/openaustralia/planningalerts/commit/ed6b029d5aff120bd2f82958cc866e4f9f496a4e
>


91977808_LBNL_NewVRFModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1799 of 1803 tests passed)

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


Esse commit não era basic toggling não.


Just to be sure: if "Digitial Ocean" is replaced as default type in the bill model by "Internet", this commit will then be useless, right ? (Not the case if the default type becomes something else of course)


Thanks... I'm waiting for Karel to approve this and merge when he is ready.

On Thu, Aug 13, 2015 at 8:27 PM, nichongjia <notifications@github.com>
wrote:

> Hi Dan, According to your suggestion, I check the code style by using
> tools you mentioned. Thanks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaldi-asr/kaldi/commit/7afae3f8c6db1a5a7f4737656a646e491a8b85b8#commitcomment-12703728>
> .
>


Are you importing the right file. GCMReceiverDelegate is a public protocol (http://cocoadocs.org/docsets/GoogleCloudMessaging/1.0.3/Protocols/GCMReceiverDelegate.html) and it's defined as a property in GCMConfig.h (http://cocoadocs.org/docsets/GoogleCloudMessaging/1.0.3/Classes/GCMConfig.html#//api/name/receiverDelegate).

I'm presuming you're using a Swift project you would have to define a bridging header for Obj-C interfaces and use that within your Swift files. More info here https://developer.apple.com/library/prerelease/ios/documentation/Swift/Conceptual/BuildingCocoaApps/MixandMatch.html#//apple_ref/doc/uid/TP40014216-CH10-ID126


here we have a call with argument of object.
What happens when it's called with 'new' keyword?  DEFAULTS are not used.


It's been a while since I wrote the led device code, I think that all I did, was to modify the original ws2801 code to work with apa102 leads instead. Regarding the end and start frames, are both added at line 23, because at the time all my testing showed that it really did not matter if the endframe was 4 times 0xFF or 4 times 0x00. However the article you linked should give some thought to see if the endframe should be calculated in another way.


why namespace is there? it is obviously comes within params


Without this I was not able to compress with -c:a mp3, so I guess ffmpeg has no mp3 encoder by it self.


@greenlikeorange I am fix for uglify problem 3ed5d48 with `{mangle:false}` option. :smile:


I copied over the comments I had left here over to the global PR diff, so you can ignore the comments on this commit.


Good work


I would opt to remove the "Sfx" because there really only is one kind of an audio channel so extra qualifications are unnecessary.

I don't see a benefit to having a MusicChannel, because there can only be one song playing at a time. Also,  a logical channel capable of playing sound effects with looping should be sufficiently abstract for also controlling music. I would instead look toward interfaces or abstract base classes to provide a uniform way to control and observe any audio-playable objects.


currentPath will give the server path of only UI website . We need to get the server path of Service hosted. Tried to give as http://wcfServername/uploads/ as you have mentioned but it was showing error 'Path not routed'. Hence hard coded the same temporarily.


basically, entityInstanceContext.NavigationSource as IEdmEntitySet may return null if the navigation source is singleton or contained entity.


@aab-odoo As on generic view I think we should add btn-sm and btn, and I didnt get you first point "Replace the a by a button" , can you explain what are you trying to say.?


Hmm OK. I'll take a look tomorrow. I think I know what the problem is, I
just don't have a G4 so never really tried on getting it to boot.

On Thu, Aug 13, 2015, 10:42 PM Graig <notifications@github.com> wrote:

> Yea, I tried using your repo first, but wouldn't boot past the LG Logo.
> Thought it was a kernel issue, but my kernel booted fine using the hima
> repo.
> On Aug 13, 2015 10:12 PM, "Rashed Abdel-Tawab" <notifications@github.com>
> wrote:
>
> > You know I've got a g4 repo right?
> > http://github.com/lg-devs/android_device_lge_g4-common
> > <https://github.com/lg-devs/android_device_lge_g4-common>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/graig79/android_device_lge_g4-common/commit/181ae486d735e6c27300ed71206b4814f9aec08a#commitcomment-12703309
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/graig79/android_device_lge_g4-common/commit/181ae486d735e6c27300ed71206b4814f9aec08a#commitcomment-12703489>
> .
>
--

Thanks,
Rashed Abdel-Tawab


this should be sandbox1.ala.org.au


Thank you =)


this form can be placed to product page of woocommerce via shortcode [place-order-form]


Perhaps it's better to pull that package in at deploy-time with bower?

On 13 August 2015 at 21:15, bz <notifications@github.com> wrote:

> It helps nothing to make changes here. This is part of a package that
> lives in an other repository.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddycloud/buddycloud-angular-app/commit/0e3a041f9f4b6e45c2aa6f22adc2021ec02d17af#commitcomment-12697142>
> .
>

--
Simon Tennant | CEO Buddycloud <http://buddycloud.com> | +49 17 8545 0880

Book a Buddycloud call <https://calendly.com/simon-tennant/buddycloud-call>


@otherchirps long overdue :)


That's totally wrong, why do u need where the service is hosted and even if you need figure out from where in the iis the ctxt will be there. No hard coding please, that is broken code and I will not allow that.

Thanks
Ranjan
Sent from my iPhone

On Aug 13, 2015, at 11:41 PM, sreejithms01 <notifications@github.com<mailto:notifications@github.com>> wrote:

currentPath will give the server path of only UI website . We need to get the server path of Service hosted. Tried to give as http://wcfServername/uploads/ as you have mentioned but it was showing error 'Path not routed'. Hence hard coded the same temporarily.

—
Reply to this email directly or view it on GitHub<https://github.com/Sbux5401/sbuxready_new/commit/3aa74de6161830bb7f72d388e1d4cfa594aee780#commitcomment-12704728>.


remove ()


I thought it was convention to put the more common case first, though this does make the question clearer.


Discard should become
<button type="button" class="oe_form_button_cancel btn btn-sm btn-default" accesskey="D">Discard</button>


You should not add yourself here. BTW, official starters use that for documentation purposes only (Spring Boot itself does not use this at runtime).


Added a second line


remove()


ця сьома стрічка так і каже http://turboportal.ru/uploads/posts/2013-06/1371595971_my-power.jpg


what is the point? DEFAULTS are skipped when called without 'new'


к2


Well the point of a having an abstract Channel would be to abstract away looping and other such logical management so that it becomes agnostic (as you said). In which case, MusicChannel would be the specialized variant if we did happen to need one.

I think we're arguing the same point for the same benefits but from different perspectives (again) :)


@aab-odoo Okay got it. Done.!


``Discard`` should become
``<button type="button" class="oe_form_button_cancel btn btn-sm btn-default" accesskey="D">Discard</button>``


<h1> Chỉ để đặt tên của trang web thui


Needs ifdef


Shouldn't you use ```url.replaceAll(" ","%20")``` ?


Gee, thanks @gavinking A whole patch dedicated to me :D


Excuse me, This is not a jQuery Countdown.
This is "The Final Countdown" by Edson Hilios. (http://plugins.jquery.com/countdown/)
Already, cdnjs storing a jQuery Countdown 2.0.1 by Keith Wood. (http://plugins.jquery.com/kbw.countdown/)

The version and name is very similar but different.
I referenced jQuery Plugin sites.


comment this out, debug leftover?!


PopulateStores() is needed; commented out only because of performance issue which I described last week


So...whatever floats your boat, of course, but I'm not sure why you're still using Ant to build this?

Seriously, the Maven build instructions have half the number of steps (with one being optional), no need for manual dependency downloads, no need to clone the main FindBugs project, Eclipse is optional, and you get reporting pretty much for free.


Would it be clearer to assign status := params.EntityStatus{blah} and then append to the slice


use `replaceAll` if the first param is a regex, but it's not


I've always thought the convention was to not use negative conditions first but now I look for a reference I can't find one (though [this is close](https://github.com/bbatsov/ruby-style-guide#no-else-with-unless)).

The reason I did it was to make it easier to read and as you say above, it does that.


you might want to use expression such as deptno + salary or some other column name instead of repeating deptno.


I don't like this empty catch...


Elasticsearch


if this fails, i wonder how it deals with having orphaned attachment records in state?


Ok.


it make sense


**Note to Self:** Don't write documentation at 3am before the launch of a reasonably anticipated project.


that's equivalent of previous logic. create an issue?


"Pointer to the variable that stores the current Pica code offset. Used to handle nested code blocks."


ez, modify get\set to apply checks


color_bits can read from
input->get_video_info ();
you can refer from CLBayerPipeImageKernel::prepare_arguments


Need to clean up the code - lot of comments to remove, and css can be streamlined.  Adjacency checking function is lot of cut and paste, hope I can clean it up.  Need to randomize the puzzle and let user know when they've won.


You mean we have no :crown: by default!


Feels kinda academic to expand "JIT" here. :)

"This class implements the shader JIT compiler. It recompiles a Pica shader program into x86_64 code that can be executed on the host machine directly."


What you are seeing is probably the same issue as https://github.com/EricssonResearch/openwebrtc/issues/385 and https://github.com/EricssonResearch/openwebrtc/issues/422


we don't need static members in cl_image_handler.h
and this file is a base class. please don't change it.


default should be false, since most case are not WDR by now


プロトタイプの画像は別テーブルを作り、prototypeモデルhas_manyを指定しましょう。理由としては投稿時の画像は一枚とは限らないからです。
画像の投稿はcarrierwaveを使ってみましょう


@caladri, thanks for your explanation. Currently it worked using this approach.


return $.getJSON


Yes, hard coding should be avoided, that's for sure. We should find an alternative for getting the service hosted path. Do you have any other suggestions for that than the one which you suggested earlier which is giving Path not routed error?


Your right. I've introduced 374a340b12bd9534e82c857b41bdbf0fcb174d36. Much cleaner!


is this required here ?


looks good


What do you think about a format like this?


use one loop instead of 4


use obj notation.


same as above..


3773 .responsive .first.column, .responsive .third.column, {

Komma zuviel


As the constructor is not public, I take it that it is not part of the public API and not intended to be used by any other plugins. I could add a new constructor instead and deprecate the old one, but don't see the benefit.


Hm. Så om jeg setter PLACK_INTRA=false så ... blir PLACK_INTRA=true?


PopulateStores() is needed; commented out only because of performance issue which I described last week


@starrynte Yeah, but please make a branch on the main UCLA Moodle repo for easier integration.


There is actually a real case for the m2release-plugin, where the scm password ends up as a system property as it is passed as a Java system property to the Maven process.
Also, it looks like all environment variables ends up in the system property list as well. Not sure if this is a bug or correct though.


eh, ja. så lenge PLACK_INTRA er satt vil den bli brukt.


We could simply get the end position instead of `len` from above then pass it to `substring()`.


Kanskje heller gjøre dette på linje 17 så det er mer eksplisitt?

`:koha_intra => ENV['PLACK_INTRA'] ? ENV['KOHA_PLACK_PORT'].to_i : 8081,`

Nå ser man ikke så lett at porten endres.


Thank you @swcheon!

cc @Piicksarn


Nice catch! Interestingly that it hasn't been discovered until now.


- Can we preload the Gradle properties from hardcoded locations? If we know that Gradle will look for the properties at i) System.getProperties() 2) in gradle.properties and 3) in ~/.gradle/gradle.properties file we can parse it upfront. It's more of a guess, but I don't know a way to consult the tooling API for this info before it's too late.
- If I use Eclipse I'd expect if I configure my IDE the changes are propagated to the integrated tools. Therefore the configuration in the IDE should have priority over the tool.
- Regarding to mixed proxy settings I see the following use cases:
  - Grade: no properties, Eclipse: no properties
  - Gradle: http, Eclipse: https and vica versa
  - Gradle: http(s) with no password specified, Eclipse: same http(s) server with  password specified and vica versa
  - Gradle and Eclipse specifies a different server for a protocol
I think all use cases above could make a good test case. They can also be resolved by using the _eclipse proxy settings win over gradle.properties_ principle. If you agree I'll list them in the spec.


I'd rather download the `tar.gz` from GitHub tags... Since the workflow with `pypi` is not reliable.


Looks like the `end_indices` field in `column.proto` is not used anymore after this change..?


You change the logic here - access rights on a socket from srw-rw-rw- to srw-------


Denne var kanskje ikke nødvendig likevel?


Не очень похож...


No.. Actually forgot to delete


Twice?


Denne if'en er vel ikke nødvendig?


это, конечно, очень хорошо... Но даже из названия ничего не понятно


Thanks.


Are you sure that we don't need to specify the file the `%attr(-, root, root)` ?


nei, skal fjerne den


if @browser?


Well then


Couple of questions @red-trela


это я понял, это как раз хорошо
а вот это что?
```
private int sessionTheFirstCoursEnglish;
private int sessionTheFirstCoursUkrainian;
+	private int sessionTheFirstCoursBiology;
+	private int sessionTheFirstCoursAnatomy;
+	private int sessionTheFirstCoursLatin;
+	private int sessionTheFirstCoursPhylosophy;
+	private int sessionTheFirstCoursPhysics;
+	private int sessionTheFirstCoursSociology;
```


Your service is hosted in IIS. So try to access the relative path.

If that doesn't work then enable path of all the folders from iis for the project once that is done you can find  accessing the path easily

For more could you search online.

Thanks
Ranjan
Sent from my iPhone

On Aug 14, 2015, at 12:05 AM, sreejithms01 <notifications@github.com<mailto:notifications@github.com>> wrote:

Yes, hard coding should be avoided, that's for sure. We should find an alternative for getting the service hosted path. Do you have any other suggestions for that than the one which you suggested earlier which is giving Path not routed error?

—
Reply to this email directly or view it on GitHub<https://github.com/Sbux5401/sbuxready_new/commit/3aa74de6161830bb7f72d388e1d4cfa594aee780#commitcomment-12704937>.


bedre :)


Kan du dobbeltsjekke den "insecure-registry" på dockers eget repo?


I see it now. Yes the original indentation was tabs, but this would be an error, since the vast majority of the source code uses Visual Studio's default indentation settings; Enforce spaces over tabs, 4 spaces. Which is also what I use. In fact I see that SpriteSelector.cs is very inconsistent about its indentations. I could reformat the file since I'm already in there, or this should possibly be a separate commit or pull request. @ivan-mogilko do you have an opinion on this?


good spot


@swcheon, @PeterDaveHello, i got it! Thank you!


we're silently consuming only jwt or findById exceptions, which is probably ok


Is the file rename really needed? Why not rename all other .glade files then?


BLanks

\

Why?


Nei, er uklart om det faktsikt var avgjørende. Skal vi prøve med 2GB igjen?


Lol , [] is so 2010! I hope we can be friends agn


File name inconsistent


Yep I derped


Why

those

blank

lines?


@mikel Wasn't it supposed to be `{ |f| f.write("#{mail.encoded}\r\n\r\n") }`?


You could use[Setup] attribute on setup method


ok test


这是第二个里面的修改文件


That's exactly what jemalloc is meant to replace. You can certainly write a custom allocator in Java, on top of a big ByteBuffer. But unless you invest a huge amount of engineering effort, it's very likely to suffer from fragmentation and other performance issues.

> either new ones or using MemoryUtil.setupBuffer on an existing ByteBuffer

You can do this with realloc:

```java
ByteBuffer buf = je_malloc(32);
// ...
long ptr = nje_realloc(memAddress(buf), 64);
if ( ptr == NULL ) {
	// handle error
} else {
	// reuse instance
	memSetupBuffer(buf, ptr, 64);
}
```


naming conventions sit there in the corner raped and crying


dispose() is not enough? :D


啊啊啊啊


This does not produce the correct result:
````
$ awk 'BEGIN { print ("1.10.0" < "1.3.1") ? "1" : "0" }'
1
````


this is not


Good point, I haven't heard if anyone were using localised version of git, but I can imagine such situation.

There is ```git branch --(not-)merged``` command. I will add this fix to my PR on weekend.


改为工厂方法


I think it is a mistake and the static can be removed .


those are in `vt.setup` now.


this should be `vt.qemu` (your other PR was accepted)


第二次添加信息


观光团！！


Issue reopened.


:boom:


`const_buffer const&`


lgtm


because it is enabled by default .... and to find this bug :)


Cho em hỏi chút là tại sao các dòng cuối, trước dấu } thì ko cần ; hả anh ?


You're welcome. :+1:


Thanks, folks :)


@digitalextremist Here is what I have done so far to detect the deadlocked Task


Would have been nice to have a 4.1 branch. no?


Alright, I had remove the file. Thanks for pointing this out!

BTW. Is there some official documentation or a walkthrough of creating a 3-rd boot starter?


Logs are used for finding and debugging the server error message details during development. We can remove the logs by switching it off while hosting in Production server.


I think /f motd doesnt show because u replaced it with

/f desc which doesnt work

https://github.com/LazerSoft1/FactionsPro-EDITED/blob/master/src/FactionsPro/FactionCommands.php#L352

Look into it


70 Popular addons part2


Use trace instead you can see the error in event viewer

Thanks
Ranjan
Sent from my iPhone

On Aug 14, 2015, at 12:41 AM, sreejithms01 <notifications@github.com<mailto:notifications@github.com>> wrote:

Logs are used for finding and debugging the server error message details during development. We can remove the logs by switching it off while hosting in Production server.

—
Reply to this email directly or view it on GitHub<https://github.com/Sbux5401/sbuxready_new/commit/9db9c7a3748b7465a069bbea79310f6ad300a7bd#commitcomment-12705310>.


Maybe you can make it more simple:
```
 req.uri = sdscatprintf(req.uri, "%crecursive=true", sort?'&':'?');
```


It is the TestProxyServer, right?


Isn't this setting `use_parent` for all the tasks? Like there is a missign a tab?


Remove unused code.


Ok I'll let you get on with the work. :)

Ultimately, I would like to see sound effects (samples) and songs in Banks, so a Channel would be completely agnostic of what type of content they happen to be playing.


That will make another test case verifying that subsequent build can have different proxy settings.


tes


Ingen tid til git-kørekort, slamkoden skriver ikke sig selv!
Git-politiet fanger mig aldrig!


I thought of starting working on that today, actually. See https://github.com/spring-projects/spring-boot/issues/2537 and https://github.com/spring-projects/spring-boot/issues/2927


I need to see when I can get to the lab. Write tests and continue with everything that is still clear to you until we talk.


Use case:
Installed TYPO3 7.4 without composer.
Does not have a vendor folder on web root atm.
Compatible with master but not 7.4 atm.


I would like this patch to be backward compatible and run on old REST. Can you add an `if 'tm_secondary_input_dataset' in kwargs['task']` please ?


What are you talking about?
This code changes the window title so that the WM recognizes it.

This is exactly what other apps (gedit, terminal, etc) do.


fixes #78


maybe you should make this page plugin first because you'll need to rewrite tests after change


@et65 `stage` only exists if you compile boost by youself  from source.


Nó có cũng được, không cũng không phải lỗi. CSS chấp nhận bạn à


The general format is:
window title (visible in headerbar): _Blah Blah Blah_
window title (visible from GShell and other WMs): _Blah Blah Blah_ - `application_name`


> Compatible with master but not 7.4 atm.

correct! It is impossible to make this code be compatible with master and all 7.x versions.

I will tag a and release a TYPO3 7.4 compatible version


Also added cleanup and signal handler exit stuff.


shizzle... repoman commit -m "bla" is so hardwired ... I keep trying not to forget it though!


<!DOCTYPE html>

	<meta charset="utf-8">
	<meta http-equiv="X-UA-Compatible" content="IE=edge">
	<title></title>


Maybe this makes it clearer? The `map` is required to convert the DOI type to the String type, as the orElse can only return the same type that is currently stored in the optional.

```java
Optional<DOI> doi = DOI.build(fieldText);
Optional<String> doiURL = doi.map(DOI::getUrl);
return doiURL.orElse(someOtherStringInstead);
```


改啥鬼


Archer circle 2 required


If removeAttachments fails, the error bubbles up and causes the worker to restart. Once the worker restarts, it will rebuild the state, which will find the attachment is still Dying, will attempt to detach, and then remove, again. This is why DetachVolumes must be idempotent.


Same thing was done, right?  Only the names were changed to IStarbucks and StarbucksServices for avoiding errors during conversion since the existing project was referring the same.


What is the logic behind this name?
I use something like **pow10** in such cases.
Though technically correct name would be **radixPower** or **basePower**.
And then it would make sense to define const **radix** equal to 10.


Perhaps a *little* clearer, but I prefer this as the append is done in one place. If I were to append below, it would have to be done in two places (i.e. in both the err == nil, and err != nil branches). I was doing that originally and missed a branch.


iti arata ultimu website mereu pentru ca faci each. Nu trebuie sa faci each pentru ca iti inlocuieste mereu continutu in get-websites pana cand e ultimu mereu. (logic)

Trebuie pur si simplu sa faci `$('.get-websites').render websites` exact ca la errors daca te uiti acolo.


rename to object


generalize comment


ここも NOT NULL があった方が良いのでは？


全部に NOT NULL ないと回答履歴として機能しなさそう


Current testset lacks tests with large skew between list sizes.
Also I don't see tests with chaining/cascading overflows (e.x. one overflow causes another).


Move all these to unlocker.rb and always use ||= so it doesn't override intentionally set values


maybe we could get rid of that :)


carrierwaveの使い方を見ていただければわかると思うのですが、投稿画像の情報を格納するカラムが必要だと思います


Maybe this could be removed.


@bcoe fyi, sync method added ;)


Maybe an idea to reduce your build time : Stripe runs three hours of Ruby
tests in under three minutes by splitting the tests over multiple workers :
https://stripe.com/blog/distributed-ruby-testing

2015-08-12 9:56 GMT+02:00 Dieter Adriaenssens <dieter.adriaenssens@gmail.com
>:

> Hi Alex,
>
> What I did to get to builds 7129 :
>
> - set timeframe to month (button on top left corner of dashboard)
> - set some filters in the build job duration chart (chart on the right on
> the second row) :
>   - build matrix : dart, stable
>   - result : passed
>   - build_trigger : push
> - On the chart you will see jump in the buildtimes, if you hover over the
> columns, you can see the corresponding buildnumber
> - With the travis api query (see above), I found the corresponding build
> id on the travis website
>
> For builds 7112 and 7145 :
>
> - the same procedure as above, but with filter on build_trigger =
> pull_request (all other filters are the same)
>
>
>

--
Kind regards,

Dieter Adriaenssens


なるほど。少し違うのですね。
確認します。


Awesome :)


I want to go over this with you on Sunday as part of the code review

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Thu, Aug 13, 2015 at 11:47 AM, grisha-kotler <notifications@github.com>
wrote:

> the index definition changed but we don't need to reindex only update the
> index definition.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grisha-kotler/ravendb/commit/3daf5921a13d076c59f60b3b1cbc3a5643d93b1a#commitcomment-12683770>
> .
>


This is unneeded here.


I agree with you : for now, in some cases we open the file several times if we pick up several streams in it. I will make this improvement.
For the Decoder, you think of a new class in avTranscoder or in loudness library? The avtranscoder::AudioDecoder already provides decoding.


I have added a buffer into the token expiry so that an in-flight token won't expire, see https://github.com/ably/ably-ruby/blob/master/lib/ably/models/token_details.rb#L26-L28 and https://github.com/ably/ably-ruby/blob/master/lib/ably/models/token_details.rb#L88-L93.  I would recommend doing something similar to avoid race conditions and subtle differences in clocks


This needs to be `||=` otherwise you'll be clobbering an intentional setting possibly


一部例外はありますがshoulda-macherで大体のバリデーションのテストがかけるので可能な限り書いてください


モデルとマイグレーションのコミットは分けた方がいいと思いました


I agree: we can organize the application into separate files.


Nice template, i made one of these for Sublime textedit, since i dont use Notepad ++ :)


@sindresorhus awesome, saw the work happening -- I'm going to work on getting this pulled into yargs this weekend. It looks like the polyfills work on almost every platform?


Okay, that is better.
We also want to have the admin/license/forceUpdate endpoint.
And we have that available on the UI as well.

*Hibernating Rhinos Ltd  *

Oren Eini* l CEO l *Mobile: + 972-52-548-6969

Office: +972-4-622-7811 *l *Fax: +972-153-4-622-7811

On Thu, Aug 13, 2015 at 11:33 AM, talweiss1982 <notifications@github.com>
wrote:

> this is fixed in the second commit
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/talweiss1982/ravendb/commit/1bd0a80ce84308da9907e7556ac91eb34902180e#commitcomment-12683479>
> .
>


@benjaminjt try to checkout my fork [http://lordkada.github.io/healthy-gulp-angular](http://lordkada.github.io/healthy-gulp-angular)


@stefan-kolb: And again I learned something about git :) Thanks for the reminder!


By the way, this routine can be simplified to
```cpp
void TestCase(
  std::initializer_list<int> first,
  std::initializer_list<int> second,
  int result)
{
    CHECK_EQUAL(
      ListToNumber(SumListsAsNumbers(CreateList(first), CreateList(second))),
      result);
}

void TestSumListsAsNumbers() {
      TestCase(
        {3, 1, 1},
        {5, 1, 9},
        1028);
}
```


ddddddd


return i jest za wcześnie. Pętla sprawdza pierwszą wartość w B, jeśli jest false kontynuuje zewnetrzna, jesli jest true, zwraca i


dddddddddddddddddddddddddddddddddddddddddddd


请问为什么要加 `Qt.Tool`, 歌词不能一直置顶显示了...


![captura de tela de 2015-08-14 05 07 05](https://cloud.githubusercontent.com/assets/12115990/9269704/7785b6fe-4242-11e5-88d3-3df5c9da1992.png)


Looks ok, thanks!


You should be able to do this without involving Maven,

./gradlew shadowJar
java -jar build/libs/dropwizard-dashboard-fatty.jar

:-)


Not exactly impossible. We can iterate through the two options for the time being and use the first one found. That's common behavior in e.g. Composer `bin` scripts.


Why did you change the code from the asynchronous `fetchAsync` to use the synchronous method `fetch`?
Please note that `fetchAsync` already returns a `Promise` (and don't indent with <kbd>tab</kbd>).

I think that you should be able to revert this the previous:
```javascript
return pdfManager.pdfDocument.xref.fetchAsync(ref).then(clone);
```


probably one is enough :)


LGTM


`old-locale` is available, but it makes no difference. When I specify `-fold-locale` explicitly during configure, then the build fails as follows:

~~~~~
configureFlags: --verbose --prefix=/nix/store/cdipi2m2blv9pkpfbfkdxq1d6awrgjh8-haskell-hledger-lib-dirty --libdir=$prefix/lib/$compiler --libsubdir=$pkgid --with-gcc=gcc --package-db=/tmp/nix-build-haskell-hledger-lib-dirty.drv-0/package.conf.d --ghc-option=-optl=-Wl,-rpath=/nix/store/cdipi2m2blv9pkpfbfkdxq1d6awrgjh8-haskell-hledger-lib-dirty/lib/ghc-7.8.4/hledger-lib-dirty --enable-split-objs --disable-library-profiling --enable-shared --enable-library-vanilla --enable-executable-dynamic --enable-tests -fold-locale
Configuring hledger-lib-0.26.98...
Setup: At least the following dependencies are missing:
time >=1.5 && <1.5 && >=1.5 && <1.5
~~~~~


Yes, it has been tested on OS X, Linux and Windows, but should work on other OSes too as it's just using standard locale environment variables and falls back to the posix `locale` command.


This seems caused by an issue in Hive and may not an issue for BigBench.
We did some experiment based on 1G data size (Hive version 0.14).
The Q12 result will be empty if enabling the optimization 'set hive.optimize.skewjoin=true;'.
The result will be NOT empty if hive.optimize.skewjoin is set to false. Thanks.


release RTV, you don't use it as I can see.


Nit: Is this actually necessary, since this method calls `fetch` ([see below](https://github.com/dwightkelly/pdf.js/commit/2a24f195c5729d7834d1ffba9aeba4bbf99c9fea#diff-3fd0fc66c9db990d996e595419b3f1e1R1348)) which already has this assertion; see https://github.com/mozilla/pdf.js/blob/master/src/core/obj.js#L1228.


Hello,

I made this adjustable so that I could easily run multiple instances on a
single machine for working on the multiple radio rhizome synchronisation
work, and know for sure which instance was being talked to by which Rhizome
HTTP API client.

I'm open to a better solution.

Paul.

On Fri, Aug 14, 2015 at 3:03 PM, Jeremy Lakeman <notifications@github.com>
wrote:

> If the port can be configured to anything by any client, this no longer
> makes any sense. Other old clients will be using HTTPD_PORT, as any
> configurable client will pass HAS_PORT below. But we should probably
> document when HAS_PORT was added if we want to break compatibility.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12704310>
> .
>


Vad händer om x inte är en skalär? Om man får in en vektor? För så lär det oftast vara.


Samma här. Low & high begränsar bara i 1D.


Same here. No need to hold pointer to texture if you don't use it. release it.


that you have to use _NotFound to detect if something was not found points to an issue by itself (not of your code, but of lrucache implementation).

i recently noticed that pop() without a default causes a KeyError for dict.pop, but not for lrucache.pop (because there, the default behaviour is to give back None).


I added that method when I was trying to write a test that used mocking. In order to do mocking I had to provide a getter for the private property. I ended up removing the code, so I will delete this extra method.


Em hiểu ròi, cảm ơn anh !


I don't know if it's case-sensitive.


Returning a non-obvious invalid `CFileItemPtr` looks very odd but it's not really a change of this PR. Just noticed it.


It's more uglifier friendly, since `that`, as a local variable gets compressed, reducing the resulting min.js


I'll document the @return , but the @param is more of an overkill since it's a bit redundant.


Must change previous line to complete the fix


I think you can revert all of these changes, since they really shouldn't be necessary.


"Mas chamadas de funções em JavaScript são custosas em comparação ao um blocos de um `loop` simples".

**Original**: But function calls in JavaScript are costly compared to simple loop bodies.

**Sugestão**: Mas chamadas de funções em JavaScript são custosas comparadas a simples blocos de repetição (loop).


saw approval from shino
at https://github.com/basho/riak_cs/commit/f33c8f35616fb7e99d73a64fd33a98579da94280


merging basho/riak_cs/feature/verify-realtime-sync-complete-on-rt = f33c8f35 into borshop-integration-1216-feature/verify-realtime-sync-complete-on-rt


basho/riak_cs/feature/verify-realtime-sync-complete-on-rt = f33c8f35 merged ok, testing candidate = eecdf6f3


Base64url encoding contains only alphabetics characters, numbers, dash and underscore (I forgot numbers in that regexp). "".split('.') doesn't validate the input.


I thought about it, but there's a need for this method. There are other methods that could take advantage of this method if it was in another class and supported other reference points. I'm going to leave it as is, but in the future when we do some refactoring, we'll support other reference points.


I'm not 100% sure if you need to explicitly set `${result}` to `Py_None`.

I'm not 100% comfortable with this change as it might affect other methods and therefore break backwards compatibility of the API. Did you check if there are other API methods which return a pointer to a `std::vector<>`?


@SauloSilva show cara! Só dá uma olhadinha na frase ali para fazermos o merge :wink:


Missing space before `=`.


Nit: `var promise = doc.getRawObject({ num: 17, gen: 0 });` would work just as well.


1: M100 needs more reviewers
1: M101 signature is missing


41:1: I101 Imported names are in the wrong order. Should be RECORD_KEY_ALIASES, RECORD_PROCESSORS
41:1: I201 Missing newline before sections or imports.
45:1: I100 Imports statements are in the wrong order. from collections should be before from invenio_records.config
151:4: E131 continuation line unaligned for hanging indent
178:54: E261 at least two spaces before inline comment
178:55: E262 inline comment should start with '# '


1: D205 1 blank line required between summary line and description (found 0)
3: L102 copyright year is outdated, expected 2015 but got 2014
27:1: I101 Imported names are in the wrong order. Should be find_packages, setup
28:1: I100 Imports statements are in the wrong order. import os should be before from setuptools
28:1: I201 Missing newline before sections or imports.


Nit: there should be no space between `name` and `:`, e.g.
`expect(data.map.Type).toEqual({ name: 'Page' });`


Comparing pointers with integers can be dangerous because you can accidentally put * before pointer variable and compiler won't issue any warnings.

Instead you should compare pointers with special const **nullptr**.
```cpp
if (inFirst != nullptr)
```
Or use implicit cast to bool which comes from C
```cpp
if (inFirst)
```

Choosing between the last two depends on codestyle that is used in your environment.


TeamCity Subsonic :: Debug [Build 51](http://109.103.226.38:8512/viewLog.html?buildId=35049&buildTypeId=Subsonic_Debug) is now running


I only moved it from above. It's just an arbitrarily large number, iirc this is state in a comment somewhere else.


TeamCity Subsonic :: Debug [Build 51](http://109.103.226.38:8512/viewLog.html?buildId=35049&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:01:59


When this commit will be tagged? I asked this on twitter was ignored.


Do you have any other thoughts on this? I predict this class growing a lot more members over time (especially with actual multiprocess support), and I think it's better if we separate them in sections like this.


Are all the arrays always present? Or does it only contain nonempty ones?


101235614-Fix-AFN-Error-Output (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


This change shouldn't be included in this patch, please remove it.


Nah, on purpose. It's a size, not a pointer, so I don't usually pad with 0 in those cases.


Awesome!


Why do you need this assumption? It seems like we can treat empty list as zero and this will give us reasonable behaviour.

Empty + Empty = Empty
Empty + Non-empty = Non-empty
Non-empty + Non-empty = Non-empty

This convention is usually used while implementing long arithmetics.


params[:answer_user_name].to_i のto_iは、何故付いているんでしょう？？


Clarity? If you're debugging an application, or implementing new stuff that uses the same code you can just search for it instead of having to go through a lot of annoying busywork re-generating the code by hand. (It's the same argument as for why the huge PICA register structs should have the register addresses in them so you can locate yourself.)


Whoa, I think I'm having a Deja Vu. Is it possible to eliminate this code in favour of the code below?


I think it is better to have a field than accessing a synchronized map at each frame.


It should be Monday next week.


They're always present and accessible but can be an empty arrays... Do we need `?? NSArray()` here?


`operation` and `pemissions` are made of bitfields (though I can reduce the padding size of permissions). `addr0`/`addr1` are pointers. `size` is the only general numeric value.


I think a very specific file path like this should not be in this public repository. Is there any other use for the main function if you remove it or can the whole main function be removed?


answer.total_correct(login_user.user_profile.id)　はletで定義した方が良いと思います。


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1482


On 2015-08-14 00:45:09, Xiaogrill wrote:
> Ingen tid til git-kørekort, slamkoden skriver ikke sig selv!
> Git-politiet fanger mig aldrig!

Jeg ved hvor du bor.  I hvert fald hvilken by.


I don't understand what you mean with the inline comment.


TeamCity FluentMigrator :: FluentMigrator [Build 337](http://teamcity.codebetter.com/viewLog.html?buildId=197209&buildTypeId=bt82) is now running


OK


Typo in the commit message.


Line 77 isn't adding anything to the test. Consider replacing it with an expectation that tests no wikis are in the database.


TeamCity FluentMigrator :: FluentMigrator [Build 337](http://teamcity.codebetter.com/viewLog.html?buildId=197209&buildTypeId=bt82) outcome was **SUCCESS**
Summary: Tests passed: 3119, ignored: 60 Build time: 00:01:42


They don't really have any names assigned by anyone as far as I can see. I'll try to think of something.


OK, I applied the patch but nothing has really changed: I still can't sign anything. Here's a link to the zip containing the three mechanisms files and the latest log from the tests:

https://drive.google.com/file/d/0B_lWKDdIYuQTY0xZV2FhOGVfa0E/view?usp=sharing

I think I might try to develop a test card piece by piece to see if I can figure out exactly what's needed (if I can remember my C programming ;-)  -  it's been a long time since I've written any C code)...


Add `private: false` to `wiki = create(:wiki, user: user)` so its clear to someone reading the test that the wiki is starting as non-private.


wystarzczy galaxy == planetCoordinates_2.galaxy && system == planetCoordinates_2.system && planet == planetCoordinates_2.planet


The only value I know for sure what does is 20 (0 and 2 return some kind of memory usage report, but I kind of just eyeballed them), so unless you can find someone that knows more I don't think there's much point yet.

A lot of this syscalls is easy to RE if you cross-check with the Kernel struct layouts in 3dbrew, so might be a good follow up PR, even though I haven't found anything that uses the other types.


The hiface driver is in mainline linux since v3.11, so I'd rather not add this change.
This repository is on-line just to document the out-of-tree history.

For kernels newer than 3.11 just use the mainline driver.

JFYI this very change has been added in the mainline driver in this commit: https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/sound/usb/hiface/chip.c?id=874b8d422e26e21dd432657e25f679c75440c1bc

Thanks,
   Antonio


gdzie ten plik???
chyba zapomniałeś dać na nim git add :(


QTDIR should be enough to get include path of Qt (we add it to the env few lines below).
And must check platform before adding this path:
```
if not env['PLATFORM'].startswith('win'):
```


Consider adding `role: :premium` to line 110 so its clear to a reader that the user is a premium user.


może lepiej expectedPlanetsNumber ???


In this case the null check is required.


Here as well.


yes, fixed that exactly this way.


The 2 comments above for this test are about making it clear in the test that the setup test data is aligned to the most important parts of the `it` description for this test: "allows a **premium** user to **privatize** a wiki".


Good! My favorite Code School path in is iOS, but also i like HTML&CSS.


Maybe you want to also update documentation above


co gdy dostaniesz inną odpowiedź niż się spodziewasz?
tzn. message innego typu.


Please use `StringUtils::Format("s%02de%02d", m_iSeason, m_iEpisode)` and ditch `MakeSeasonEpisode`.


I think possibly better to keep style changes away from actual code changes. Otherwise it is difficult to see what a pull request actually does.


Check `:create` is the intended controller action here.


this one would be a problem if the map is a cleared one right?
(gonna put some solution)


fast-forwarding develop to borshop-integration-1216-feature/verify-realtime-sync-complete-on-rt = eecdf6f3


I submit my favorite Code School path ...


Какой смысл писать имя переменной без присваения ей значения? Это тоже самое что написать 2; Тоесть бессмысленная строка.


Do /f desc then type your desc in chat after executing the command


The indentation was very inconsistent in both AGS programs from beginning
We decided to use spaces when starting to work on the engine and since then I am trying to force this code style there.
As for Editor, I could not see myself as its maintainer, because I do not work on it as much; but it seems to be usual to use spaces in Visual C# code, and I think contributors used spaces mostly too.


headers() ? ;)


Well spotted!  I wrongly assumed that was how pop() worked.  So I should be able to just use `try...finally` as you tried to tell me :).

I can see the problem in the lrucache pop() (which I removed).

    def pop(self, key, default=None):
    ...
        return super(LRUCache, self).pop(key, default)

it probably should have used `**kwargs`


FixSlabTransition (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1797 of 1797 tests passed)

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


`doi.map(...)` reads strange. Why doesn't doi.getUrl() work?
(shouldn't be `Url` written as `URL`? :))


yes, yes we do - I'll add it!


I see. Sad, because we're dependent on it. It's honestly hard to understand why this functionality isn't built in to MVC and Visual Studio in the first place.


Thanks @wil93. We'll try that :


This one (`Client/Info.plist`) is not needed ATM - no occurence of `org.mozilla.ios` there. But maybe leave it there to be on the safe side with potential future changes that might introduce the identifier back?


My debug build fails to run with a SideBySide error, complaining of missing VC90CRT. I resorted to forking Poco and putting a working openssl build in a submodule for the time being.

The newer build scripts (PS) allows giving it a path to openssl if I'm not mistaken? If so I could just grab the ones I find at openssl.org. Or perhaps even better, use cmake on Windows.


Also, since this is at risk of being computationally expensive, can you put a printout at the beginning and one at the end of the for loop? This will help our operators investigating potential issues.


I think that was a copy-paste of the pattern from another method - I suspect it would be ok though so I'll change it..


Yes, I believe you may fix indentation in the file too, as a separate commit.


Yes. This was fixed ages ago.


@xHFx Ok but I can't do stuff fir you 24/7 because 2 reason:

1. I code either things and play games sometimes
2. I have a life (ik its hard to believe)


このメソッドの中身なんですが、
`create_answer_history(true, params[:answer_user_name].to_i, current_user.user_profile.id, params[:answer_user_id])`
と対して変わらないですね。

呼び出し元で、 `create_answer_history(true)` とかで済ませられるようにしたい感じです（今のところ、 `correct`, `incorrect` からしか呼ばれないみたいなので、内部で params, current_user 取る分には問題なさそう）。


FixSlabTransition (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@frank-dittrich  I will change it to the end. Thanks for your advice.


This is commented out since I don't know if this could affect the script @ concrete5.org


As I said, there is no need for that check here, do it before you create the season/episode string. See my comment below.


`doi` is an `Optional`, hence, it can be absent. You can only invoke methods available by the `Optional` class.


Why are you writing this `Local` class?  It's not used or needed.


Sorry,I don't understand. We have CSearchDialog::modules() in the beginning of pipe that return BtConstModuleList because it is better for CSearchDialog::m_modules that is also constant.

We would just move ugly const_cast to CSwordModuleSearch::unindexedModules()...


Should I include the commit it in the same pull request?

I have time this weekend, if you'd like me too I go could go over every code file in AGS.Editor.NoNative and convert tabs to spaces. But if I do that it should probably be a separate pull request


I don't see enough value to sacrifice backwards compatibility for an subjectively nicer, but almost identical identifier.

Additionally, I guess I did choose the name in order to be close to the naming conventions used in Linux for that concept. So I really disagree with this commit and would like to go back to the old name.


Hi bkhall,

Nice feature, but I am getting two vertical scroll-bars.

I have figured out what is going on:

 and
 both use "overflow: auto" in the style.css, which means you're telling the browser that it should use scroll-bars whenever there is an overflow.

This probably only occurs when you have listed more airplanes in the site-bar then fits on the screen.

I have changed this line:

div#sidebar_container { float: left; width: 400px; margin-left: -400px; height: 100%; overflow: auto; }

into this line:

div#sidebar_container { float: left; width: 400px; margin-left: -400px; height: 100%; }

and now I have just one scroll-bar.

I hope this helps.
Ted


and please add the check for -1 here

```
std::string strSeasonEpisode(m_iSeason > -1 && m_iEpisode > -1 ? StringUtils::Format("s%02de%02d", m_iSeason, m_iEpisode) : "");
```


Sort out all the extra newlines.


The target module is required for dual processor locking.

Sorry, just now noticed that I missed these comment.


Git is a version control system.
Git is free software.
This is my first project.


@dusek Yes, it has been moved to BaseConfig.xcconfig for now. I do leave it there for safe.


This should be "above" in accordance with [BIP68](https://github.com/bitcoin/bips/blob/master/bip-0068.mediawiki#specification) no?


Both


No NEWS update?


@ludovic-henry looks like this breaks one of the System.Math tests on x86 Linux:

```
MonoTests.System.MathTest.TestPow

   MESSAGE:
     #45
  Expected: 1.0d
  But was:  NaN

    +++++++++++++++++++
    STACK TRACE:
    at MonoTests.System.MathTest.TestPow () [0x0081f] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-i386/mcs/class/corlib/Test/System/MathTest.cs:583
at (wrapper managed-to-native) System.Reflection.MonoMethod:InternalInvoke (System.Reflection.MonoMethod,object,object[],System.Exception&)
at System.Reflection.MonoMethod.Invoke (System.Object obj, BindingFlags invokeAttr, System.Reflection.Binder binder, System.Object[] parameters, System.Globalization.CultureInfo culture) [0x00038] in /var/lib/jenkins/workspace/test-mono-mainline/label/debian-i386/mcs/class/corlib/System.Reflection/MonoMethod.cs:295
```


Where are the contributors now?


I dont agree with this change to add_entry. This way only allows adding the points sequentially. Especially it prevents users from adding the first point at ```(0, value)``` which is the natural start for any pulse sequence.


If you are going to do multiple files, then better not include this to current pull request.


It's not sufficient, in my Linux, headers are deployed into:
/usr/include/qt4
QTDIR need to be equal to /usr, but the file /usr/include/QtCore/QtCore don't exists, it was /usr/include/qt4/QtCore/QtCore
Maybe a better solution ca be found, but I see only this one ..


ah sure, haven't seen cleared select


1: M110 missing component name
1: M190 line is too long (61 > 50)


Bare tag din mappedatamat med, hvis du kommer forbi, så kan vi rigtig råhygge.


In that case this pull request is ready to be merged if you agree


This note is solely to make sure I get things right.

Elsewhere in Riak, I have seen records marked by a version prefix, for the good reason of allowing those to be distinguished and dealt with properly when a new version or protocol comes along, during a cluster upgrade. Is this also the case here? And if so, what are the expected scenarios where both _v1 and _v2 could be coexisting?


I think we should make this method public. Any plotting should be possible without instantiating a sequencer and all that. What do you think?


Not wanting to flog any horses, but .... is "focusHighlighting" a state of an item, or a sub-object? It sounds like it's a state, so might be better reworked as `--highlightOnFocus` instead??


sadly no, but maybe https://github.com/springlobby/springlobby/issues/547 needs to be implemented to?!


Out of interest ,why is this called `_restClientClient` as opposed to just `_restClient`?


multiselct returns a vector on success, null on failure/cancel. It's needed to be able to distinguish null from empty list, as they are two different things.

Frankly, anything else is incorrect. The generator should strictly generate code, *not* apply implicit conversion.


TeamCity Subsonic :: Debug [Build 52](http://109.103.226.38:8512/viewLog.html?buildId=35051&buildTypeId=Subsonic_Debug) is now running


good job


I actually fixed the socket io error just now, so localhost is back on app.get('port'), which defaults to 3000. I'll update the README with this information.


I think it can make sens to add it into AvTranscoder because it maybe usefull to have analysers or players which can be based on it.
It requires to pass https://github.com/avTranscoder/avTranscoder/blob/develop/app/avPlay/AvReader.hpp into the library, with some cleans (to read video vs audio streams).


TeamCity Subsonic :: Debug [Build 52](http://109.103.226.38:8512/viewLog.html?buildId=35051&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:02:17


TeamCity Subsonic :: Debug [Build 53](http://109.103.226.38:8512/viewLog.html?buildId=35052&buildTypeId=Subsonic_Debug) is now running


Yes it will be why it doesn't work. The change was intentional: you should always call `(fill ctx)` or `(stroke ctx)` after drawing ops. To always fill a circle meant that you could never have a circle outline.


Поправь пожалуйста тот один момент.


Thanks


TeamCity Subsonic :: Debug [Build 53](http://109.103.226.38:8512/viewLog.html?buildId=35052&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:02:04


TeamCity Subsonic :: Debug [Build 54](http://109.103.226.38:8512/viewLog.html?buildId=35053&buildTypeId=Subsonic_Debug) is now running


dont really need this tag @since here


Covers tag needs namespace


should have a since tag


For loops, especially nested ones, bring stuff into scope outside the loop: minmally that would be auth here, but I'd go further & bring s3_group_members and s3_user_pe_id


TeamCity Subsonic :: Debug [Build 54](http://109.103.226.38:8512/viewLog.html?buildId=35053&buildTypeId=Subsonic_Debug) outcome was **SUCCESS**
Summary: Running Build time: 00:02:04


thanks!


Why so many for each role?


don't forget to:
    cd /var/www/my_geonode
    sudo python manage.py collectstatic


Is this a role that we're planning to make use of in SAMBRO?
1st mention of it although it does make sense :)


const is "too new" yet. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/const


Fixed in 4751e49.


For admin & atuhenticated, I would use the existing default/users/masterUsers.csv for consistency


`notifyCurrentValue()` creates pending change records even though there is no actual change, adding unnecessary memory pressure. I see it’s a bit off from what the change record mechanism of `Object.observe()` intends to, and doing that will confuse widget authors/users when they look into our code.

BTW 1st `computedProperties()`/`refreshRendering()` call in my change does not effective have a notion of “old values”. They merely are “values when observation starts”. If such difference is not desirable, here are several possible solutions (ones but the 1st one will be breaking changes):

* Change parameter names and in-line docs accordingly
* Use a different set of callbacks for “values when observation starts”
* Passing `null` to `computeProperties()`/`refreshRendering()` when they should handle “values when observation starts”


No, strncasecmp is available since 2.6.12 at least. So there were two functions with the same functionality and they removed one of them.


Merged in d1be52cbdec76529f756b4e4c25f0c77ad35da77


Do they really need this?
For the workflows I see, only the MAP_ADMIN would need this.
Public & CUG users only see map via the Home Page
- we certainly could add a fullscreen option to that page :)


Am unaware of any usecase for this in SAMBRO?


The way I understand it (I hope I'm not mistaken), is that if we have
```C++
typedef QList<CSwordModuleInfo *> BtModuleList;
class BtConstModuleList: public QList<CSwordModuleInfo const *> {
public: /* Methods:*/
    inline BtConstModuleList(BtModuleList & items)
    { Q_FOREACH(CSwordModuleInfo * i, items) append(i); }
};
```
Then we can just have `BtConstModuleList` as the type of `ML` and the `const_cast` in the following code becomes unnecessary.

Explicit `const_cast`-s should be avoided. `CSwordModuleSearch::unindexedModules()` should not return a `QList<CSwordModuleInfo const *>`!


I am unaware of SAMBRO using this workflow?


Why?


Heres the corresponding commit message of the removal of strnicmp:

commit cd514e727b18ff4d189b8e268db13729a4175091
Author: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Date:   Mon Oct 13 15:54:25 2014 -0700

    lib/string.c: remove duplicated function

    lib/string.c contains two functions, strnicmp and strncasecmp, which do
    roughly the same thing, namely compare two strings case-insensitively up
    to a given bound.  They have slightly different implementations, but the
    only important difference is that strncasecmp doesn't handle len==0
    appropriately; it effectively becomes strcasecmp in that case.  strnicmp
    correctly says that two strings are always equal in their first 0
    characters.

    strncasecmp is the POSIX name for this functionality.  So rename the
    non-broken function to the standard name.  To minimize the impact on the
    rest of the kernel (and since both are exported to modules), make strnicmp
    a wrapper for strncasecmp.

    Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
    Cc: Grant Likely <grant.likely@linaro.org>
    Cc: Andi Kleen <ak@linux.intel.com>
    Cc: Dan Carpenter <dan.carpenter@oracle.com>
    Cc: "H. Peter Anvin" <hpa@linux.intel.com>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>


So Template creation is just full Admin, right? OK


楼主，请问.NET4.0 可以用吗？


Some comments above. Ultimately the proof is testing:
Can the relevant roles access what they need?
Can the roles not access what they shouldn't?


removed in 364b087


What are the other changes along with the 8gb/4gb fix..


Changed tests for converter lib #3


@dachary i am not sure if we should `if (agent_state)` here. as we discussed over irc, i think we should `assert(agent_state)` instead.


No, not if they're always there.


@eriksargent Merged! :)


Exciting!

Please be aware that the editor <-> server protocol is very much in flux and I plan on really improving it - adding support for multiple concurrent editors as well as internalising many of the refactoring/snippet functionality into the server so these features don't need to be re-implemented for ever editor.

Feature requests/ideas would be very much welcome.

Out of interest, are you using the OSC interface?


Maybe check the callers of that code whether it could be just adapted to "behave as usually pop() does".


このコミットいらなくないですか


```Error: undefined local variable or method `sync' for #<Formulary::FormulaNamespace804888eab13de66387a9102e71c8da8a::Tuttleofx:0x007fdba0b42b88>```

and https://github.com/Homebrew/homebrew/blob/master/Library/Homebrew/download_strategy.rb#L668 already does this..


Leave a comment


Did not fix the issue for me.
I still can't add a coupon for a product that has variations on sale. To be clear I am using a coupon with % discount to product and "no discount to products on sale" selected. And I am trying to apply the coupon to a variation that is not on sale.


Just curious, why this merge was not fast-forward? A commit merged with its parent looks awkward.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 383](http://192.168.30.225:8111/viewLog.html?buildId=2041&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


open and opned are changed in sync why are they separate variables?


Actually.. i just only copied it from Door.php, but you just made me wonder :) :+1:


No, I just want to delete because console.log does not work on JADE. The problem of the slowing the system down was the console that I was using on javascript in order to debug the code, I guess.


@unho yes in Mozilla projects I have units with no words.  I'm not sure how this cleanup impacts that.


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


FixSlabTransition (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


these last two entries seem confusing - or at least non-obvious - so I'd add "_line"?


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


Yes lets not get into changing the default.  Allow the distribution to do this via a daemon flag.

Don't want to go down the rathole of fighting because of some ZFS bug.


@fraenkel did your comment get addressed?


please do not break the parameter list into new line.


should we adjust the message? `Error: WINDOW_DIALOG_SELECT is NULL, this is not possible.`


TeamCity Subsonic :: Debug [Build 55](http://109.103.226.38:8512/viewLog.html?buildId=35055&buildTypeId=Subsonic_Debug) is now running


right, now we dont need it. it was suggested by @tboerger, as we could need it sometime later


I mean that line there is only for backwards compatibility for nodes that
don't send the (now ancient) HAS_PORT flag. Which we always send;
https://github.com/servalproject/serval-dna/blob/0a061e0fe8b63882eb5fda028def23a91aed75ed/rhizome_packetformats.c#L159

Any nodes that doesn't send it, will by definition, not support the new
config value and will implicitly be using HTTPD_PORT as their web server.
So using the config value there will actually break backwards compatibility.

On Fri, Aug 14, 2015 at 5:46 PM, gardners <notifications@github.com> wrote:

> Hello,
>
> I made this adjustable so that I could easily run multiple instances on a
> single machine for working on the multiple radio rhizome synchronisation
> work, and know for sure which instance was being talked to by which Rhizome
> HTTP API client.
>
> I'm open to a better solution.
>
> Paul.
>
> On Fri, Aug 14, 2015 at 3:03 PM, Jeremy Lakeman <notifications@github.com>
> wrote:
>
> > If the port can be configured to anything by any client, this no longer
> > makes any sense. Other old clients will be using HTTPD_PORT, as any
> > configurable client will pass HAS_PORT below. But we should probably
> > document when HAS_PORT was added if we want to break compatibility.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12704310
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12705763>
> .
>


I think leave it rootmount and then allow docker to discuss.  I think anyone who uses this option will have to know what they are doing.


wrong example, should be multiselect ;)


This should be a default, not hardcoded.


Oh, thanks.


TeamCity Subsonic :: Debug [Build 55](http://109.103.226.38:8512/viewLog.html?buildId=35055&buildTypeId=Subsonic_Debug) outcome was **FAILURE**
Summary: Compilation error: Client.Tests\Client.Tests.csproj (new) Build time: 00:00:57


Should be `getExperienceHeld()` and in `Human` it should be `getFoodData()`
Maybe to be more consistent it should actually be `getExperienceData()`