I will improve atomicity in next commit. Rebase is probably a bad idea as this commit is part of a new public branch. Thank you for your methodological improvement proposal.


Since these are only used for LabelViews, I can move these constants there so that we don't have `Flame` anymore.


It's better not to remove the "Temp" folder, the engine uses it for various purposes and does not create it if the folder is absent.


remove app.js and move to gitignore


This is a leftover from an earlier version that was never committed, obviously we can use the reportedTerminalPosition right away. Before the queue held TerminalSize:s, so I had to convert the position to a size, but I missed this line when changing it to TerminalPosition.


Why do you null terminate the file data? It is binary.


Should this be `Type` not `CookieJar`?


Maybe add `fixed = TRUE`?


You'd need to use relative paths for that to work on other systems than yours.


Hi @r0p0s3c , thanks for update, agree with you, but I have 1 request:
1. Could you please change `||` to coffee's `or`


Done


I don't think so, I think it's actually slower, since `strstr` will have to traverse the string regardless. Thus, you get two `O(n)` operations (`strstr` + `replace_all`) for each character we need to replace.

Not that it won't pale in comparison with all the I/O and libspotify calls we're doing anyway ;)


> Any reasons against vendoring?

No, I don't have any at this time.

I asked in order to learn (since the commit message didn't explain the motivation ;) ).


You'd better not hardcode PINs so that you can use tokens which have specific requirements (e.g., minimum PIN size). I had a token which required 6 characters PIN. The best would be to get the PIN from the environment.


The comment for this method in the header file needs to be adjusted.


This ought not be necessary.  It was required?


I suppose it could be applied to *any* abstract class; but atm, it's limited to `CookieJar` (per L51)


FIXED na siya :)


Shouldn't you be filling this in?  You have a partial gvk based on the original types right?  Does the method not handle partials?


Cool, thanks dude.  I'll correct it as soon as I get a chance.
On Mar 14, 2016 5:19 AM, "chineel" <notifications@github.com> wrote:

> Hi Scott,
> Line 1131 There is an error It should be like this
> mSettingsObserver.registerContentObserverLocked(
> mSettings.getCurrentUserId() );
> Instead OF below
> mSettingsObserver.registerContentObserverLocked(userId);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scotthartbti/android_frameworks_base/commit/6fc5b58d1e5d8b1e523aa09945ba1d2d192db80f#commitcomment-16674914>
> .
>


@karlcow small confusion between a docstring and a comment here :)
Wouldn't the highlighted examples be considered as docstrings as they explain what the function does? (I used this [Stackoverflow post](http://stackoverflow.com/a/708674/2220216) as reference)

Thanks a lot :)


Isn't that handled by `grabbedsignature()`:
```
     def grabbedsignature(self):
         signature = self.localsignature()
         if not "_signature" in dir(self):
             return (self.localname, signature)
         if signature != self._signature:
             print "Ingredient signature mismatch:", \
                 os.path.join(self.uri.isubdir, self.localname)
             print "  expected: %s"%self._signature
             print "  obtained: %s"%signature
             return False
         else:
             return True
```


That's alright! No harm intended. It isn't likely, but if someone has the port, they can bypass BungeeCord, and since the nodes are in offline mode, can join as any user they like. There are protections against it, but better safe than sorry.


PS: you don't need the trailing asterisks `/*` or `/**`, `**/dist/` is enough.


This also (re)adds the "Return to view" link. Does it need a separate entry in CHANGES?


Another good value: int rHist = (hValue + cmhValue) / **18725**;


I think so, but I haven't checked anything other than the linked blog post in the pull request.

{{src/default.hbs}} is part of the theme itself, not Ghost, so this change won't interfere with upgrading a server using this theme.  The version I've linked is somewhat out of date though, and of course you'll get better performance if you store a minified version locally, rather than querying the jquery site itself.


@pelwell I already read the page. It mentions, "the source for both will be available in the userland repo soon". But it is not there yet? What license are these binaries available under, Broadcom proprietary, GPL, or something else?????


coolt


Maybe it was due to my CFLAGS; I also had:
```
-DCURL_MT_MALLOC_FILL=0xEE -DCURL_MT_FREE_FILL=0xCC
```
(not sure that's related). But after looking through `hash.c`, I did this:
```
--- a/hash.c 2016-02-05 07:58:13
+++ b/hash.c                   2016-03-14 13:32:58
@@ -279,7 +279,7 @@
       struct curl_hash_element *he = le->ptr;
       lnext = le->next;
       /* ask the callback function if we shall remove this entry or not */
-      if(comp == NULL || comp(user, he->ptr)) {
+      if(comp == NULL || (comp && (*comp)(user, he->ptr))) {
         Curl_llist_remove(list, le, (void *) h);
         --h->size; /* one less entry in the hash now */
       }
```

`||` is not a boolean short-circuit like `&&` is. So now it doesn't crash. But I've no idea if it's correct.


@dobbymoodge And this is exactly why you added the test, but since @liggitt didn't actually finish the work (needs to update the `getPodSpec` method, you can see why I suggested adding the information right here to make it easy.


Yes, I was not really done tinkering with the build before you jumped in :blush:. I think we may need to play with it a bit to see if it can be faster.


Minor change for a big number. Perfect!


Sure, done


Very good!


Watch looks weird.


BTW, I didn't notice the message "**Renamed unconventionally named file** by other contributor" . LOL again


Actually the replace_refs function masks these two classes. Also, the
globally instantiated objects should be used because the base class caches
records by URL. So the idea is to move the get_record logic to a common
place and transform these two methods into two one liners?

On Mon, Mar 14, 2016, 13:27 Samuele Kaplun <notifications@github.com> wrote:

> In principle we would need these 2 classes everywhere we need to load a
> record, not only when the record is referred by another record.
>
> @jmartinm <https://github.com/jmartinm> do you have a place to suggest
> where to put these? Actually can't we override the default get_record
> function?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mihaibivol/inspire-next/commit/99ebd50414af19e3b932cc896b9b2cf8146d4523#commitcomment-16678750>
> .
>


@hadley I believe this was not on purpose that the `tbl` class was not included.


Yes, soon != now.

The binaries are free to download but not to distribute. Wait for the sources.


This should not have been necessary.  Bad permissions for `StatusCheckerRoleName` would be my guess.  Probably missing the `.../*` permission.  I'd rather update that role for everyone than "fix" it here.


This change should be in another commit.


I went ahead and removed it so its not included in the pr


Can you mention the issue in your commit messages? This way I can follow more easily :) This one is applicable to #23


I would name it MPD_Flow_Begin


I would name it MPD_Flow_End


方法


EPL License


Good point, but I don't think we would want "rbd journal reset" to leave the journal in an inconsistent state.  That is something I missed when I integrated the tag allocation.  I'll open a tracker ticket to fix it. Thx.


Comments


Yes.


Remove unused code


Explain in the commit message what you did exactly.


Yeah, meant to put that in the WIP commit


@greut C'était sur le suivi du tutoriel officiel (http://guides.rubyonrails.org/getting_started.html)

Ce sera corrigé dans le prochain commit


lets just use `%s` here 1 has no sence.


Ah, I see.  I don't personally care about that either as I always want `linear = True; autosquash = True`.


Will this show a retry button too?
It would be nice to not have to restart the installer if Tribler is running.


1、苹果近两年WWDC都发布了什么新技术?
2、苹果公司四十年来里程碑式的产品有哪些？


Noted.

FYI since you appear to be working on bumping Smack to 4.1 in jenkins-jabber-plugin: I've an compiling Smack 4.1 branch at https://github.com/Flowdalic/jabber-plugin/tree/smack41. But there is a strange bug in there which I couldn't debug yet.


Why would we choose to pass a config instead of the client and pool?  Client seems more mockable and a cleaner split.


不好意思關於.all.delete_all的這個問題我已經知道了
因為這邊的delete_all也是呼叫elation.rb這裡面的method
所以沒有dependent也會執行delete
但我去看了querying.rb這支class他裡面並沒有說他是用relation.rb裡面的delete_all
請問你是怎麼知道呢？

KURO huang <kurotyan514@gmail.com> 於 2016年3月14日 下午7:42 寫道：

> 最大的原因應該是我反骨吧ＸＤ
> 因為rails101的教材是後來才教到dependent: :destroy的這個用法，一開始是用destory_all的樣子
> 然後在找destroy_all的用法的時候，就剛好看到很多文章再說delete_all跟destroy_all的差別
> 所以就好奇了的把dependent拿掉自己去試了一下，所以才發現的
> 然後其實我用有through
> 這是我的兩個model內容你可以看看
> group.rb
> <https://github.com/kurotyan514/rails101/blob/master/app/models/group.rb#L6>
> user.rb
> <https://github.com/kurotyan514/rails101/blob/master/app/models/user.rb#L9>
> 其實我自己還有去試了delete_all跟destroy_all跟dependent: :destroy
> 看看這三個方法那個花的時間比較少
>
> 但其實看完你的文章之後我還有點小疑問，
> 那就是我能夠了解了為什麼會update成nil的原因了
> 但我其實還不太能了解為什麼他會這樣子做呢？
> 把重要的foreign keys update成nil，然後造成孤兒資料的結果
> 還有就是＠group.group_user.all.delete_all
> 是因為把.all的東西當作參數送進delete_all裡，所以才會不變成update嗎？
> 如果不會太麻煩妳的話希望能夠幫我解惑一下啊～感恩
>
>
>
>
> Bruce Li <notifications@github.com> 於 2016年3月14日 下午7:13 寫道：
>
>> 不知道是否能夠引用你的文章內容呢
>>
>> 當然沒問題 [image: :laughing:]
>>
>> 另外，我好奇為什麼你想要拿掉 dependent 以及不使用 has_many :through
>> 呢？這應該是很普遍的做法，有什麼特別的原因不採用嗎？
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kurotyan514/rails101/commit/21f5c92fff1e3f3288d571438c9b48b3c4bd96e9#commitcomment-16677300>
>> .
>>
>
>


Hi @randaz81,
Thanks for these changes! Should we also update `kp`, `kd` and `ki` as in `iCubLondon01`? (see https://github.com/robotology/icub-main/blob/master/app/robots/iCubLondon01/hardware/motorControl/icub_head.xml).


We definitely need to handle duplicates when we convert from another format. Most converters call removeduplicates() on their generated PO files. But I can't think of a case where we want that behaviour during parsing. So raising an error is consistent with gettext tools, but also consistent with XML parsing that would not accept a mall-formed XML file (such as XLIFF or TMX). The rest of the parser is not that strict, but I guess we should at least consider being strict in this way, since we know duplicates cause problems.

The alternative is to use the first (or last) occurrence and only log a warning. That way Pootle should not get its data corrupted, and we make the change a bit more gradually. I'm not sure if it is much better, as I guess few Pootle admins will see this warning in the logs, and Virtaal users will only see this if opening a file from the command line.

I already mention that other gettext tools don't accept files with duplicates. I see poedit opens it, but complains when "validating". Gtranslator complains when opening and only shows one of the two units of a duplicate pair. I didn't test lokalize.


`||` is a short-cut too. The second operand is evaluated only if the first operand evaluates to false. (I tried to find a good authoritative link to point to, but I failed. The [MSDN page on logical OR](https://msdn.microsoft.com/en-us/library/f355wky8.aspx) says it though.)

It doesn't really make sense to check that comp is non-NULL twice in the same condition!


`javascript.enabled = false`のprofileでWeb Driverを起動してもJavaScript OFFにならない。
原因がわからなかったので、`hide-if-js`の要素をJavaScriptを使って表示するコードを追加。

```ruby
  profile = Selenium::WebDriver::Firefox::Profile.new
  profile["javascript.enabled"] = false
  wd = Selenium::WebDriver.for :firefox, :profile => profile
```

環境
selenium-webdriver 2.52.0
Firefox44.0.2
ruby 2.2.1p85


Checkmarx scan results 14 Mar 2016 13:00 UTC +00:00

High
Medium
Low
Info

New
1
14
9
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

74ec8dd86b1786c87e383b73c9af571f69bd3375	@AndrewAtCheckmarx


I wanted discovery bits split out from health checking, etc. Can make a new role for that, I suppose


Doesn't this leave `res` undefined?  In other words don't we need to do something like:

```
if buildbot:
   # the above stuff
else:
    state.add_comment(':bomb: Buildbot not configured for this repo')
```


ggg


nodiratime is included in noatime so there is no need to add it.


Was a copy of the upstream init. I can hoist the pool out


Yeah, I made that mistake. But I also realized that `force` only works for buildbot in the first place (working on a patch to fix that)


This line is present twice:  `if self.args.show_visibilities:`
May be add placeholder to template and replace in with empty string:   `'%s%s' % (other, ' %s' % visibility if self.args.show_visibilities else '')`
or something like ' '.join([version, scope_string, visibility, release_state])


You can supply a "MB_RETRYCANCEL" flag instead of "MB_OKCANCEL" that will show a retry/cancel option.


Maria,
I’ve upped the version from 2 to 3.
Cheers,
-Bill M.

From: Maria Mikhno [mailto:notifications@github.com]
Sent: Monday, March 14, 2016 3:11 AM
To: CISecurity/OVALRepo
Cc: William Munyan
Subject: Re: [OVALRepo] Update oval_org.mitre.oval_var_1711.xml (ae59ce3)

You have not raised the version.

—
Reply to this email directly or view it on GitHub<https://github.com/CISecurity/OVALRepo/commit/ae59ce3db64db24d3752545e27b60f667ce105f5#commitcomment-16673320>.

...
This message and attachments may contain confidential information. If it appears that this message was sent to you by mistake, any retention, dissemination, distribution or copying of this message and attachments is strictly prohibited. Please notify the sender immediately and permanently delete the message and any attachments.

. . .


Enough ;)

And you'd have to do it somewhere else. If I make it an option you won't have to leave your IDE. Nicely keeps the responsibilities close together.


让世界更有爱


The problem is that the ARGB32 is 32 byte format while wlc expects 8 bit format (1 byte per component)


Add to main branch


Nice, thank you. I miss this one.

I did not notice that sample was using the word8.

Also, I though that we might always use the LE and BE for word.


Seems sane to me.  If someone is later concerned about chattiness we can make this configurable.


Then let's do that for unfriendliness' sake.

The rest of the PR looks nice to me, so if you can verify it works fine, I'll merge it.


d


`result.copy` only replaces the fields given as parameters. The value of `incrementalData` is not affected by this operation.


Смысла не имеет, т.к. потеряете совместимость и мало что выиграете. Сделано по большей части для удобства разработчиков, чтобы не писать в методах **контроллеров** конструкции типа
$user = cmsUser::getInstance();
и так далее.

> cmsUser::addSessionMessage()

это статический метод, смысла вызывать его из объекта нет.

> cmsUser::get()

если обращений больше одного/двух, имеет смысл использовать обращение через объект

> cmsCore::error404()

это тоже статический метод, также вызывать его из объекта не имеет смысла, он для этого и сделан статическим.


this is sweet! knockout is really awesome I'm glad we're using it.

can you set the order = null after we click "submit order."


And should we also apply this change to `iCubLondon01`?


OK


May be join this arguments together:   `-i dest='show info'`?


Please replace `&&` by `||` to retain the original functionality.


Cairo doesn't seem to have 8bit format, so it might make sense to add ARGB32 format to wlc (opengl should support it.. not sure about GLES2).


It would be good to have initial selection + clear selection working, too.

Care to work on that? :)


Ненужный файл


I thought there were graceful delete cases the controller was specially handling. @derekwaynecarr do you know if the controller needs watch permission on resources?


Commented out the original lines, copied html from the WPTouch plugin preview.


We Do so much but we care too less.. Everyone wants to make everyone proud but when a person dies inside the family judge him/her. Take care and care for someone who go's From being hot and sexy to natural... Nature is the beauty but how do we get there?? We go reversed to be born again! I want this for the world (FAMILY AND FRIENDS) to know I was a clown and funny and and proud... But you saw me devistated and scared THAT FOR I SAW WE ALL ARE CONNECTED . Love each other and take care for each other and look out for yourself! Thats where you will find God... WHEN THE WORLD SHUTS DOWN YOU START TO SEE THIS IS OUR HEAVEN!!!


The reason `@import 'options';` appears first is because of defaults being loaded in first.

We're defining our `em` and `rem` root sizing defaults, in addition all of the media query variables later on.


copy-item is a PS cmdlet, it cannot be tested with lastexitcode for success. I suggest here to throw the error on the else branch of if ((Test-Path $src)). The error message should be: "ERROR: The windows-curtin-hooks module is not present, please execute ""git submodule update --init"


![img5517b2269ea893 00868213n](https://cloud.githubusercontent.com/assets/14913140/13745398/9e1db648-ea0f-11e5-97db-b78e50d77853.jpg)


where is the Readme part?


There's already a `repo_url` variable we could use it looks like.


> Yes.

Joy.


Sooo, whats up?


@jeroenj yes, but that was not what I was thinking yesterday... :grimacing:

Well, it's still less strict than it was before :expressionless:.


@plexus is it possible that this merge get published in a public version on rubygems.org?


Let's get one commit with all the controller permissions and re-wiring.


Predicates.and is the default behavior if you add multiple Predicates.
So better remove Predicates.and and use two lines with filter.add([Predicate]); for better readability.


Makes sense to me.


True! Fixed in branch measurements_addon.


new


I checked and it actually says that this method is only supported by IE. What happens when this is run on Chrome or Firefox then? http://quirksmode.org/dom/core/


@JonathanRosado yes, what you are suggesting is a proxy route and `shipyard` does with `swarm`. That might useful, but it is not what I am suggesting. An ILM client should be able **also** to query a registry for its repositories (search), whether it is a public one or a private one, **using the same approach**. If we rely on proxies only, then we will have to add one for Quay.io, Google Platform, etc. and the  client will have to know the endpoint and the request structure of that particular public registry. In fact, part of ILM's objective, is to provide this generic API to the registries, otherwise, a client might as well hit the actual endpoint for the registry :smile:


Check the docs - CAIRO_FORMAT_ARGB32 is 32 bits per pixel, or 8 bits per component. I believe it's equivalent to WLC's WLC_RGBA8888.


Also better use the exact text from the card: nonblack creature that entered the battlefield this turn, because the filter text is used to generate the rule tooltip text of the card.


```
each pixel is a 32-bit quantity, with alpha in the upper 8 bits, then red, then green, then blue. The 32-bit quantities are stored native-endian. Pre-multiplied alpha is used. (That is, 50% transparent red is 0x80800000, not 0x80ff0000.)
```

*each pixel is a 32-bit quantity*


ok but the server is not running for example ./named_parameters


There should really be a cron job too, to check if you've forgotten to re-enable the hook.


Right. I think what I said is still correct. No? WLC_RGBA8888 is 32 bits per pixel too, no? If nothing else, the length of the data should be the same, so we should be seeing garbage instead of segfaults.


Default value is off. As I guess it should be(?). I think these fields
however are a bit legacy, would be nice to clean all of it up and rather
have some states that make sense.

But how I understood this is that it is like a totem for wether we can use
the clip or not, but I'm not sure :)

Framerate because it was missing too. Though not sure why we really have
that since it should rather be sniffed from the file I think.
On 14 Mar 2016 02:23, "petterreinholdtsen" <notifications@github.com> wrote:

> But what is the default value for this field? And why is 'framerate'
> introduced too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Frikanalen/frikanalen/commit/3bd240895b317052d36a84dcfc7b2ba0510c3859#commitcomment-16672954>
> .
>


Basically you should allocate your buffer differently as well, width * height * sizeof(unsigned integer) would be the right way (or some cairo_uint_pixel if they have dollar API typedef)


The case instructions:

```c++
      case 10:
        // HTU21D-F
        sensorValue = htu.readTemperature();
        break;
      case 11:
        // HTU21D-F
        sensorValue = htu.readHumidity();
        break;
```

Possibly cause the program to crash with some version of the "Adafruit_HTU21DF_Library" (https://github.com/adafruit/Adafruit_HTU21DF_Library/blob/master/Adafruit_HTU21DF.cpp).


@osm0sis I must have been asleep during the last commit. I hope to have resolved the problem now, so if you could try again with a future build on 4.4 that would be much appreciated :-)


...and using the 'cycle' example I pasted lights up the pads in the order 1, 2, 3, 4, 5, 6, 7, 8
whereas the current code in `test.py` lights them up in the reverse order (I dunno if that was intentional?)


updated_time now has auto_now set, which indeed sets it to 'now' whenever
it is saved.
On 14 Mar 2016 02:33, "petterreinholdtsen" <notifications@github.com> wrote:

> Is the updated_time field updated automatically now, or not? I do not know
> Django enough to tell, but could not find any obvious code to keep it
> updated.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Frikanalen/frikanalen/commit/af7e83ba9598d85429f1dd813a2dc1330cb86b3b#commitcomment-16673038>
> .
>


Yeah, I don't think anybody will have an issue with it. I just saw it and wasn't sure if it was intended. :)


Sure, I can allocate it like that, but the numbers will be the same.


@knagato 修正しました。
https://github.com/mozilla-japan/gecko-l10n/commit/2c9e03aca4a91e7c33ec5601ffb66d12eee2ce43


Sir in the salebonus.h file why did you kept an empty array at first and then you added 5 five values to it ? And why did you added # before initializing the value of i ?


It would be better/shorter to handle it this way:
```java
return versions(topic).stream().findFirst();
```


Ah! True, it's a left over, it moved to the camera itself.


reverted to make the k8s bump commit actually match the referenced commit, then repick after


@william-ferguson-au I missed some minor changes while splitting, so I need to do one more PR anyway. Should I move this to ACRAConstants?


I'll squash the revert into the bump commit before merging


I believe this is a great step in the right direction. I just can't help and think that too much abstraction can become frustrating on smaller pieces of code.

Our boilerplate acts as a foundation, it gives you the most basic styles to work with and build off of.

Having variables for some of these defaults is fine, however, unless that variable is utilized multiple times, there just doesn't seem to be a reason as to why we should have a variable for `$body-background-color:` or `$body-foreground-color:` for example. This just introduces unnecessary complexity, even frustration. If I know my body styles are as such, why do I need to go under another sass file just to edit that variable?

Further more, I believe all form variables should be contained within the `_forms.scss` file. Reason being that those variables are only utilized there. Even then, they're more than likely not necessary. By adding `$input-border-width: rem(1px);` we're automatically assuming that all sites will have a border on their inputs. Not only that, again we're abstracting variables for something that only has one place it's called for.

The reason these are not variables is because they are basic form styles to get a developer started. The moment we start adding all of these variables is the moment we start looking something like Bootstrap or Foundation.

If these changes were live and I started a project, I would be ignoring most of these as that does not work with my project's flow. We have small and large projects, and there needs to be a medium as to what is and isn't necessary. If I have a large site with a complex set of input fields, yes, I will define variables at the top of the `_forms.scss` file, otherwise if it's a small project, I may simply call for the defined colors from `_colors.scss`.

The link colors are fine, but even then, those ought to be moved into the `_defaults.scss` for easy access.

Also, what happened to the media query map list?


Just checked - we are using stage 2 (default) for babel, which includes object rest and object spread properties.
So I guess they should be transpiled fine according to http://kangax.github.io/compat-table/esnext/


Nah, this looks like it should be internal to ReportExecutor


As I said I am ok with warning on duplicates when parsing PO. And deal with duplicates when converting from other formats (a situation that most likely should be handled by those formats parse as well).

Should these two thing be moved to issues, or are you able to directly contribute PRs?


translate the first paragraph of the manual


Yeah, that will not obviously fix it. The GLES2 will still expect it as 8bit per component. That needs change. You could try adding WLC_RGBA32 (or something) and then https://github.com/Cloudef/wlc/blob/277b1a19d462c9e9b19947efafb65694a3f502ee/src/platform/render/gles2.c#L64 GL_RGBA, GL_UNSIGNED_INT here


y u do this?


OK, I'll remove this in #7930


Fixed in `dev`.


Ok we will talk more on this change and select with is better.


I'll try that, but I'm still fairly certain that we're already looking at 8 bits per component.


Bug: missing evaluation parens in jxt.


cache self._machine_manager.getActiveMachineInstance().getMachineDefinition() for performance (nitpick)


Huhuh.

You guys are mad, you know that?

Cannon meet mosquito ;)


@bendem Thanks for the feedback. I wasn't sure what collection would be best here so thanks for the info. As for the implementation details I fully intended to clean it up and make it nicer if this goes into master. This is just kind of a rough sketch if you will.


Thanks for the fix!


I've also tried copying your crappy-borders plugin and just filling the buffer with 0xFF with similar results.


@HeroCC, I see. :)


Found a small typo with "incuslive".


Ok I will change the predicates.

About the card text, I wasn't sure which to use.  Doing a gatherer search
for the card, the picture they have displayed of the card has the text I
used ("came into play") whereas the card text section says ("entered this
turn").

On Mon, Mar 14, 2016 at 8:12 AM, LevelX2 <notifications@github.com> wrote:

> Also better use the exact text from the card: nonblack creature that
> entered the battlefield this turn, because the filter text is used to
> generate the rule tooltip text of the card.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bieno002/mage/commit/812c348ecea6e2f389b05d2e858015170a6bcc39#commitcomment-16679638>
> .
>


Float and string literal regex changed


@malirod , is it needed here?


I'm torn if using more semantic HTML (ul, li) would be better here. Hmm...


Agree. That would prevent a faulty reference to a null object.

Em 13-03-2016 16:25, David Pinho escreveu:
>
> The antiPatterns list is used on the following lines, in this order:
> |sourceAnalyzer.analyze(repository, metrics, antiPatterns, repoSys,
> languages);| Line 54 at RepositoryAnalyzer
> |analyze(commits, metrics, antiPatterns);| Line 50 at SourceAnalyzer
> |processAST(file, commit.getName(), metrics, antiPatterns);| Line 62
> at SourceAnalyzer
> |astProcessor.process(commitName, file, ast, metrics, antiPatterns);|
> Line 92 at SourceAnalyzer
> |for (IAntiPattern antiPattern : antiPatterns) {| Line 58 at ASTProcessor
>
> As you can see, the list is being passed as a parameter and it is only
> actually used in the line 58 at ASTProcessor. The main point here is
> that if the user set an empty list of anti-patterns, the condition
> |(antiPatterns.size() > 0)| will not let the visminer process the
> metrics. Analyzing the code I can't see any problem triggered by an
> empty list, therefore I think this condition is unnecessary.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/visminer/Visminer/commit/24e86bd801a87156f8792756f116e1e88fbcdf6a#commitcomment-16669152>.
>


Does the crappy border in orbment currently work as is? (At work atm) Also if you can post stacktrace. But _I'm_ pretty certain we are talking about 32 bit per pixel as the documentation clearly says so :)


You cannot suppose that the service is going to run over HTTP, extract the protocol also from the asset URL


Why not a %else here ?


OI


Four 8 bit components (ARGB) fit into one 32 bit pixel.

And yes, oddly enough orbment works fine. Will get you a stacktrace when possible, I'm at work too.


iki commit terakhir po ?


You have a typo 'PlugginError'


Yes, but the fact that it isn't Four 8 bit chars, is the problem. Since when you pack into 32 bit int, you need to _unpack_ it, there is also endianess that will basically make RGBA <-> BGRA depending on system.


Yeah, I need to to be an actual LimitType to do the comparisons correctly


Nemělo by to vést na produkci? :)


Right, got it :-) New version uses `deps`.


Unfortunately this does not work because _ScanIter cannot use the modified cluster version of scan above. When I am done writing the tests, I might write a _ClusterScanIter.


Not that it will be in any way visible, but I would place the <br/> outside the <i>


Hello!


just checked  and the master actually works:

https://github.com/benoitc/hackney/blob/master/rebar.config.script#L1-L6


Good point, can do


Try to use on_post_product_offering_validation instead, this way you can ensure that the product_offering pricing has the correct structure. Imagine that the pricing does not contain an 'unitOfMeasure' field


Why use TestCase if then each test is an assert in the test. How do you know which failed and caused the overall test to fail


I also change the background of the login page to suit the app color more, it is very easy to change again~~


What's special about this number, or any of the other big numbers?


requests has a method that returns the JSON content without parsing it:

units = resp.json()


Yeah, but my code is aware of that and I'm reversing the channels when I call cairo functions to draw things. And it should not cause a segfault, it should just make the colors wrong.


Maybe we could put some message to our logs about the response from schema repo when it is different from OK?


Where's the test for single number?


Shouldn't we log "Not Implemented" for not implemented APIs on riot? Or this is the real implementation?


Static so there is no possibility of memory management bugs-- since a reference to the mempool and a pointer to the block template are passed into CreateNewBlock, and the block template is re-used, the block template could keep a pointer to the mempool object and bad things would happen if emptyMemPool was a local variable.

The block template doesn't keep a pointer to the mempool, so a non-static emptyMemPool would be fine... but it makes me more comfortable if the empty mempool has similar scope / memory management to the real mempool (which is global).


Was this the simplest thing? Using an array for 2 numbers?


Dang, sorry about that, I was not on the latest commit.


lgtm


(-1910078, 'Thank you, friend. I\'m in your debt.', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, 'Kernobee - Start Escort'),

(-1910079, 'We made it! Thank you again, friend! Scooty will reward you handsomely.', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, 'Kernobee - End Escort'),


There is a bug with github. My commit changes owner.


same issue as the previous methods


This construct is still not bullet proof. I would get rid of the get/set and wrap the entire get + nil check in a read lock then the entire compile and set ... in a write lock.


regex...!


The always-true if is intentional-- see the next commit, where build_tip is more interesting.


@gaearon I have no idea. But the files were being scanned for sure. Maybe there's a bug where they parse them but don't mark the errors because the core node_modules ignore overrides?


cool


seems this was fixed by Robert the next day in commit d9065dc


No usages. (I did not use this class and wanted to check how you use it. :)


First issue by hands on system: Seems BL trigger is affected. I mean transitions between BL levels is broken. Seems the sensor change it value every second even using static brightness control.


@ErwanAliasr1 Because `%else` is potentially evil/confusing


Hmm doesn't this destroy cairo after you create border buffer? Would this make the border essentially undefined? (Not that familiar with cairo)


This is not correct, the pricing_model field is a dict. You have to check that it contains a 'pay per use' field, and then iterate over contract.pricing_model['pay per use']


@martz21 Please pull these changes


The only resource the controller needs the ability to WATCH is `Namespace`.


that is not necessary, user is never building the package. Building is made by prepublish script in package.json and because build folder is missing from .npmignore that it will be published as well. It is not necessary to keep the building files on git .

here is the documentation for publishing modules
https://docs.npmjs.com/cli/publish


Please review the changes made.


war das der bug?


应该是 `this.drp.dimension.refresh(this.drp, this.drp._calendar_.type);`，下同


Not afaik. The border is owned by us, this just destroys cairo's references to it. It's not cairo's business to free the buffer we handed it.


应该是 `this.drp.dimension.refresh(this.drp, this.drp.calendar.type);`，下同


In general, a pipeline may produce more than one output. Would that work?


make static const?


I used basic in a few of my sentences. What I really meant is that those are basic generic placeholders.


Since this plugin only supports a single pricing component you can directly get it, and raise an error is there isn't any component or if there are more that one


@dikkedeur

(-1910022, 'What? You served me well but..', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, 'MYZRAEL_1'),

(-1910023, 'Why have you summoned me so soon? I haven\'t yet reached my full power!', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, 'MYZRAEL_2'),

(-1910024, 'No matter. You were a fool to help me, and now you will pay!', NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 0, 0, 0, 0, 'MYZRAEL_3'),


Right. I have been thinking about this but always postponed.
I'll try to re-break the text.


nu met haalBestellijst() die een ArrayList retourneerd


Right.


@pepelsbey Any insights on why this was removed? I had scrolling as a feature request on my slidedeck (https://github.com/RubenVerborgh/WebFundamentals/issues/18#issuecomment-195476452 by @Dromin).

Is it also scrolling inertia that was causing trouble for you, or are there other reasons?


Sorry, but this way someone will be always unhappy :) In this case it is us. We need SDV results as part of the build.
So I suggest to add environment variable to disable it and the default should be create SDV log.

This comment is for all the other SDV removal as well.


They are my commits, but become others'.


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


Testing Multiple


Note that an abstract pipeline to produce a list of samples may be run as something that produces a text file with samples, or a database table, or a Spark RDD.


Will be removed soon. Patches under test.


@nadako Please try to run the integration test with this updated instruction. Let me know if there is any problem.


After this commit the following example doesn't function properly (See comments for details):
```
<html>
<head>
  <script src="processing.js"></script>
</head>
<body>
<p><canvas id="canvas1" width="200" height="200"></canvas></p>

<script id="script1" type="text/javascript">
function sketchProc(processing) {
  processing.draw = function() {
     // Draw something
  };
}
var canvas = document.getElementById("canvas1");
var p = new Processing(canvas, sketchProc);
console.log(Processing.instances); // Works fine. Prints out an array with one element
</script>

<script id="script2" type="text/javascript">
console.log(Processing.instances); // Something is wrong. This prints out an empty array.
</script>

</body>
</html>
```

Basically the processing instance de-registers or exits once the code within a script tag completed execution.


@chleh @wenqing With the new tests still one test failed (see [Linux build](https://svn.ufz.de:8443/job/OGS-6/job/Linux/6018/consoleFull#11764404489fe0c9b9-0424-48f2-9469-679211ebf45b)):
```
The following tests FAILED:
	  7 - ogs-ParallelFEM_GroundWaterFlow3D_NeumannBC-mpirun-diff (Failed)
```


    #0  0x00007ffff5e2e106 in __memcpy_avx_unaligned () from /usr/lib/libc.so.6
    #1  0x00007fffecb0109e in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #2  0x00007fffecb01492 in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #3  0x00007fffecb01f02 in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #4  0x00007fffecaf31db in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #5  0x00007fffecaf3422 in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #6  0x00007fffecaf3848 in ?? () from /usr/lib/xorg/modules/dri/i965_dri.so
    #7  0x00007ffff7baeef7 in write_pixels (context=0x7e1250, format=WLC_RGBA8888, geometry=0xcb939c, data=0xdd0cc0)
        at /home/sircmpwn/sources/wlc/src/platform/render/gles2.c:819
    #8  0x00007ffff7baf6c1 in wlc_render_write_pixels (render=0x682488, bound=0x682440, format=WLC_RGBA8888, geometry=0xcb939c, data=0xdd0cc0)
        at /home/sircmpwn/sources/wlc/src/platform/render/render.c:93
    #9  0x00007ffff7bbb284 in wlc_pixels_write (format=WLC_RGBA8888, geometry=0xcb939c, data=0xdd0cc0)
        at /home/sircmpwn/sources/wlc/src/extended/wlc-render.c:30
    #10 0x0000000000423665 in render_view_borders (view=2) at /home/sircmpwn/sources/sway/sway/render.c:111
    #11 0x00000000004186ad in handle_view_pre_render (view=2) at /home/sircmpwn/sources/sway/sway/handlers.c:159
    #12 0x00007ffff7b9c2f7 in render_view (output=0x6822b0, view=0x804170, callbacks=0x682580)
        at /home/sircmpwn/sources/wlc/src/compositor/output.c:286
    #13 0x00007ffff7b9c741 in repaint (output=0x6822b0) at /home/sircmpwn/sources/wlc/src/compositor/output.c:350
    #14 0x00007ffff7b9ca87 in cb_idle_timer (data=0x1) at /home/sircmpwn/sources/wlc/src/compositor/output.c:396
    #15 0x00007ffff72c8157 in ?? () from /usr/lib/libwayland-server.so.0
    #16 0x00007ffff72c8832 in wl_event_loop_dispatch () from /usr/lib/libwayland-server.so.0
    #17 0x00007ffff72c6ee5 in wl_display_run () from /usr/lib/libwayland-server.so.0
    #18 0x00007ffff7bb706a in wlc_run () at /home/sircmpwn/sources/wlc/src/wlc.c:260
    #19 0x0000000000420a13 in main (argc=4, argv=0x7fffffffe808) at /home/sircmpwn/sources/sway/sway/main.c:222


Here too, remove floor_div.


The type hint implies `cookiejar` is _an instance_ of a `CookieJar` though. In this case it should be a `type`, which is a subclass of `CookieJar`. However, I don't think Python's type hinting system is expressive enough to indicate the bound, therefore you have to settle for `type`.


Can you submit the change for the WiKi? https://github.com/YanVugenfirer/kvm-guest-drivers-windows/wiki/Building%20the%20drivers


@jeroenj yeah, it wasn't :smile:

Thanks for noticing :bow:


Przydałoby się obsłużyć od razu wszystkie numery telefonów klienta.


You have a double hash here


good


should be `asc`


... as if rendering wasn't broken before.


Its amazing how much USB drivers control.


test comment


comment
=======


Oh, I see... No matter, I've generalised it to any class with abstract methods


Hey, did you think about opening a PR for this?


Commit wurde von Murat Firat getätigt


@endJunction  With the source code in the master branch?


`public static void main(String[] args) {
  		SpringApplication.run(PlateApplication.class, args);
//test
  	}
  }`


That's works quite nicely :)


@mfojtik @smarterclayton this breaks one of the fundamental assumptions about these tests - that they operate entirely within a namespace that is given to them. If we need this "other-namespace" test we should be creating and cleaning up a new namespace for the explicit purpose of this test in this file. This aids in overall cleanup for the test suite itself. If we agree on this, I'll make the PR to update how this test works. Thoughts?


All right. :-)


Restoring previous versions still fails in 1.3.5. See steps to reproduce. {scc 1.3.8}


LGTM


is there something functionally different, or only a refactoring to ditch "return early"?


that breaks Laravel 5.1 LTS compatibility


Don't let this get ignored forever. We will need a seamless way to go from Graylog Collector with graylog-server2 v#1.x to v#2.x


agree... we want to test cross-namespace, but this test should set up and clean up the other namespace


张文潇


Yes, swfobject-bower is the one we need because loads the latest.


@davies maybe you can help me confirm/deny a possible bug highlighted by Coverity. It notes that `advanceToNextPage` above is synchronized and modifies `pageBaseObject` and `offsetInPage` but this method, while it only reads them, is not synchronized. It could read them in an inconsistent state. Is that a legitimate problem?


@dikkedeur I uploaded my entire scripts_text table, it shouldn't break anything since there was either edited/added stuff so it should be easier that way, to get what you need. It can also be made as sql update for my fixes.

link: http://www.speedyshare.com/tuKW3/texts.sql


I thought you werent sure if there were two microphones? This is where I got confused as well. I think there is only one microphone as well but we are calling top and left microphones.


[COMPUTING MACHINERY AND INTELLIGENCE.docx](https://github.com/sellyrukmana/ai/files/172025/COMPUTING.MACHINERY.AND.INTELLIGENCE.docx)


This. I don't miss


Great.


Edit comments


:-D
Then what's the point of open source? Yes, the default content type of a "style" tag is "text/css". Yes, the default script content type is "text/javascript". It's true that the "type" attribute *used* to be required in HTML4 and XHTML, and *now* it's optional, not the other way around. Yet it's *now* that we're starting to use the script and style tags for other things than CSS and JS, like using a script tag to hold JSON data and setting its type to `application/json` - that's a perfectly valid technique. So I'd argue that it may not be immediately obvious that a script tag contains *java*script; furthermore, a server from which a subresource is loaded can send it's own content-type header and it's a little more obvious that the browser shouldn't run it when the content-type doesn't match.. so yeah, it may not have any actual *effect* on the behaviour currently, but it's good practice, therefore it's making the original code better, and that is a perfectly good reason to change it.


Why?!


masalan


lol


See test results at https://travis-ci.org/UDST/urbansim/builds/115855537


I needed something to test the reduction chain, so I hacked it in.

I have non-polarized reduction working now, though without footprint.  I
also tested 'semi-polarized' reduction, with ++ and -- channels reduced
independently.  Everything goes through, but I don't know if there are
guarantees on the order of the channels in the bundle, so the wrong ones
may be associated.  Also, I have no way of handling a missing channels.

The polarization correction component should be easy to get working, but it
will need to be updated to the current system.  I may get some time this
week, but sasmodels takes priority.

On Sun, Mar 13, 2016 at 10:52 PM, bmaranville <notifications@github.com>
wrote:

> I am planning to pass the input datasets to each fieldUI generator.
> "multiple" fields will become a table or list of inputs of the same length
> as the number of datasets in.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16671534>
> .
>


I will, thanks a lot!


Oh, my mistake! I've just plugged in some headphones, and found that both `direct.py` and `drums.py` play the correct samples when each of the pads is tapped!

I had another look at `drums.py` and see that this is because you print the **pad** but then still use the **channel** to select which sound sample to play https://github.com/pimoroni/drum-hat/blob/master/examples/drums.py#L36
So while the drums.py example still technically works fine, it's perhaps not the most straightforward *example* code to follow? ;-)


Add program


**Code Quality - Nitpick**

You test indicates that you are searching for non-existent episodes or description, thus the name of this test method should appropriately desribe that. It could be something like: `testSearchForNonExistentEpisodeOrDescription`


@pelim this was broken in https://github.com/laravel/framework/commit/f7062509d7e43e3d15d74db102008cf63ed90f89 Maybe @mirzap can run parallel 5.1 LTS branch?


After few retries, add an instance variable called `dead` and remove all the buffered messages with an error.
Then, if we get another `send` call, we can simply reject it.


Thanks for including my post! Hope it's useful :)


** Code Quality - Awesome**

I love this :heart:


The code was build with master branch at 4e3c7d2fd5811c947edec4e41bc25b13a2a51d96.


I guess http://apidock.com/ruby/Module/alias_method can help here?


previous one was better


Nope, removed!


Thanks!


I guess this change is not required. Extracting method only for customer_id is better.


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


Resolved in b4f5da1


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 364](http://ci.infinispan.org/viewLog.html?buildId=37426&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 6396809633bf909a9a6f7540994af9804d44f89f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:33


how about `original_signature_pdf_url`


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


This code throw exception if result is undefined (in case of error)


Wow! Look at that - this commit makes 0% change to the size of angular.min.js.gzip (54884 bytes before and after). I didn't expect that. And angular.min.js is slightly smaller (155103 bytes after; 155130 bytes before)!


`RackList` was probably meant here?


Just curious what flexGrow property does for the flex container. The doc says it's not inherited, which means you have to set it on each flex item anyway: https://developer.mozilla.org/en-US/docs/Web/CSS/flex-grow


I am afraid that `${keywords//:/|}` is a bashism, not supported by posix shells.


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


yep, i just did not add because the other stubs in this file are not doing it


合并分支


Did you consider re-entrance?

I personally think that using static because its safer is in this case not defensive programming because using static is not free and it weakens the global design by adding additional (but admittingly weak) assumptions.

Thats just my €0.02, feel free to ignore.


Ah, yes, I was saying the same thing.


> It doesn't really make sense to check that comp is non-NULL twice in the same condition!

Duh me!

Then it's something else. With `--test-event`, it crashes on Win-10 every time. It does not seems related to `http` vs. `https` (I've tried BoringSSL + SChannel).

Maybe enabling the old `Curl_hash_print()` code could give some hints. But how?


That would make sense.


rename, or add comment about what is this name


I think you need to answer these questions from your blog:

How long is the secret good for? Does it expire?  If I reboot the machine, can I still use the secret?
What benefit do I get from storing my credentials elsewhere say in the gnome-keyring?


Excelente :)


Typesetting


LGTM


ack


In FridgeNS.js Change
`var $elem `
To
`var elem `


If there is no stitching, no need to generate reconciliation meta data


I usually tend to use StringUtils.isNotBlank() so that even blank char are properly handled (for example trailing whitespace in properties files).


fantastic!  Thanks for all the late-night work getting this in shape.
I'll try to find some testers for unpolarized data.

Brian Maranville
NIST Center For Neutron Research
100 Bureau Drive MS 6102
Gaithersburg, MD  20899
(301) 975-6034
http://www.ncnr.nist.gov/instruments/magik/

On 03/14/2016 09:46 AM, Paul Kienzle wrote:
> I needed something to test the reduction chain, so I hacked it in.
>
> I have non-polarized reduction working now, though without footprint. I
> also tested 'semi-polarized' reduction, with ++ and -- channels reduced
> independently. Everything goes through, but I don't know if there are
> guarantees on the order of the channels in the bundle, so the wrong ones
> may be associated. Also, I have no way of handling a missing channels.
>
> The polarization correction component should be easy to get working,
> but it
> will need to be updated to the current system. I may get some time this
> week, but sasmodels takes priority.
>
> On Sun, Mar 13, 2016 at 10:52 PM, bmaranville <notifications@github.com>
> wrote:
>
> > I am planning to pass the input datasets to each fieldUI generator.
> > "multiple" fields will become a table or list of inputs of the same
> length
> > as the number of datasets in.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16671534>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16680487>.
>


kkkkk


`service` evokuje uplne neco jineho nez vyber jazyka/jazykove mutace Heureky.. Proč nepoužiješ `language`? :)


Maybe use a MessageException (I think the stacktrace is useless here).


Not really. I could go with both. I'm not leaning in favor of any. You prefer `none`?


**More than a typo it was a whole error!**


public class UsingExceptions
{
   public static void main(String[] args)
   {
      try
      {
         throwException();
      }
      catch (Exception exception )
      {
          System.err.println( " Exception handled in main" )
      }

      doesNotThrowException();
   }

  public static void throwException() throws Exception
  {

  try
  {
     System.out.println( "Method throwException" );
     throw new Exception();
  }
  catch (Exception exception) // catch exception thrown in try
  {
     System.err.println("Exception handled in method throwException" );
     throw exception;
  }
  finally
  {
     System.err.println( "Finally executed in throwException" );
  }
 }

 public static void doesNotThrowException()
 {
   try
   {
      System.out.println( "Method doesNotThrowException" );
   }
   catch (Exception exception)
   {
       System.err.println( exception );
   }
   finally
   {
      System.err.println( Finally executed in doesNotThrowException );
   }
  }
 }


Please test against master.


test commit


Don't we have a constant for this "magic" ".sonar_lock" file name?


@deads2k nice, it's like you've got some kind of :crystal_ball:

Seriously though, good call.


@raffij lol I used that to help me visualise (figure out) the rotation scaling problem I was having the other day. Answer is here https://github.com/madebymany/croppy/blob/abe14526399e0030cd75a78e9bb276a31c5357d6/src/javascripts/plugins/rotate.js#L63-L72


**Debasish Panda**
> I have added a simple method for generating UPCA barcode.
> Will keep updating for various formats.


I thought about it and I explicilty went for `Abort`:
The client is already way slower consuming data or died already. So even if we give it time to consume the last chunk completely - it doesn't buy its anything as we never send the rest of the entity. Thus he ends up with an incomplete response anyway just occupying our resources longer.
Therefore killing the client right away.


BTW it is conventional to use `…/jenkinsci/${project.artifactId}-plugin…`, but you have defied the convention of using `${project.artifactId}-plugin` as the repository name. Intentional? CC @daniel-beck.


call BCMDHD_SDIO without BCM4358
зы: а ларчик-то просто открывался


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.2.21.a5b5a274d1e62db95544b5a7aa72ed8500977f96](http://build.palaso.org/viewLog.html?buildId=70315&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1049, ignored: 13 Build time: 00:07:17


I'm gonna try integration tests now, but I just tried accessing the website, and it says 403 Forbidden for some reason :-/


Ah, I didn’t notice you were already handling that case in another place :) anyway, it’s shorter that way…


Mentioning pindexBestHeader while its not used in this method (only by the caller) is confusing. I guess you added an argument later and forgot the docs?


Running this commit through our test cluster.


You should use a utility to build this string from an object.


:tada:


@OHerasym, I think we can leave this, since it just cleans the internals.


Thank you.


Well it seems to crash in driver and memcpy. So something must not agree with the given buffer size + buffer. Maybe print the values of g in here https://github.com/Cloudef/wlc/blob/master/src/platform/render/gles2.c#L819 to confirm that I'm not doing something stupid in clamp_to_bounds function.


I thought there was a reason, but after testing it seems it was accidentally left in. It can be changed to a single get or to keep the value either true or bool I will change it to this if you don't object:

$this->remember = $request->request->get('remember') ? true: null;

I didn't feel like writing out the entire debugging process but here goes:

* Visit select_project.php, choose a project and click "remember".
  * $this->remember is set to true since "remember" is in the post.
  * Project cookie is set with "remember" is true.
* Logout of eventum (project cookie remains)
* Log back into eventum and visit select_project.php
  * $this->remember is set to false since "remember" is not in the request.
  * [getProjectID()](https://github.com/eventum/eventum/blob/master/src/Controller/SelectProjectController.php#L128) loads project ID from cookie but not "remember" value.
  * New project cookie is automatically set with $remember=false
* logout and project cookie is cleared
* login and be forced to choose a project again


Oh, I was blind. Will fix.


super práce.. jinak možná bychom tam mohli dodělat ještě blbuvzdornou kontrolu, že když už to jednou bylo odeslano, tak se celý object vyresetuje nebo se bude zablokuje (nějakou private proměnnou) odeslání znova..


**POLLOKA
**


Should I manually prepare `www/dbconfig.json`?


Could this be made more generic (and future proof) with:

```
if [[ $TEST_TARGET == jquery* ]]; then
```

This uses the double square bracket test which allows pattern matching


@alloy: Any ideas how to avoid such pesky runtime hackery?


Might be if interest to @juba: let's have nice tidiers for FactoMineR models!


You will have to explain whats going on in pe2.py. You need problem statements in this code files like I said.


CC was complaining about code duplication so had to fix. Also, could you please review against the latest commit.


No hemos tenido la conversación de que los mensajes de nuestro código deen ser en inglés. Creo que vale la pena reflexionarlo más.


We could maybe add a progress report (like for FS indexation / language analyzers) to report progress to the user.


Je to právě schválně, aby se ti lidé zamysleli a pochopili, že je to jsou to dvě různé služby. Není totiž jedno kam to pošlou. Na jednom je známe, na druhém ne. Proto `service` a ne `language` - je to totiž ve výsledku úplně jiná služba.


I feel sad that we have `jquery-old`. What happens if we started testing on more than two versions?
How about we have `jquery21` and `jquery22`, then it will be cleaner to add `jquery30` later.


Looks great. I am glad you figured out how to use flex boxes: I learned from you.

Couple things I noticed:
1. the expand icon should probably not render in expanded view
2. the expand view does not work in Firefox

Couple questions I have – since XY view should match the table display.
1. Which actions should I listen to if I need to detect table switch or table removal?
2. When a table is removed, should we clear all state (in redux store) associated with this table?


The documented precondition in different from the assertion.


create students in methods


old Safari has no idea who is `Window` !!!!!


You can simplify this, and remove the need for the additional function by appending `or 'json'` to the line above. If there's no extension then `os.path.splitext(path)[1][1:].lower()` will be an empty string. This evaluates as falsy, so adding the `or ...` allows the value to be set when it would otherwise be falsy.


@renesansz :) Yes these are the final changes. Based on discussion here -> https://github.com/BranchMetrics/documentation/pull/208#event-587638699 I've updated to simplify the process.


I'll check it properly when I'm off work, but fwiw I have looked at those values in gdb before and iirc they were correct.


Jo, asi dobrý nápad ;)


We'll need to ensure that @gkalpak includes this change in his work on phonecat


¿Por qué Pair Programming no requiere de Code Review? Todo debería requerir Code Review aunque sea desarrollado entre dos personas. No siempre entre esas dos personas estará la persona responsable de aceptar el desarrollo.


Note that I moved data normalization to the server; the current join
algorithm assumed the data was already normalized.

I'm not sure how well the algorithm will respond to a large deadtime
correction; may have to switch it from poisson to gaussian statistics.  I
guess this should be a ticket...

On Mon, Mar 14, 2016 at 9:58 AM, bmaranville <notifications@github.com>
wrote:

> fantastic! Thanks for all the late-night work getting this in shape.
> I'll try to find some testers for unpolarized data.
>
> Brian Maranville
> NIST Center For Neutron Research
> 100 Bureau Drive MS 6102
> Gaithersburg, MD 20899
> (301) 975-6034
> http://www.ncnr.nist.gov/instruments/magik/
>
> On 03/14/2016 09:46 AM, Paul Kienzle wrote:
> > I needed something to test the reduction chain, so I hacked it in.
> >
> > I have non-polarized reduction working now, though without footprint. I
> > also tested 'semi-polarized' reduction, with ++ and -- channels reduced
> > independently. Everything goes through, but I don't know if there are
> > guarantees on the order of the channels in the bundle, so the wrong ones
> > may be associated. Also, I have no way of handling a missing channels.
> >
> > The polarization correction component should be easy to get working,
> > but it
> > will need to be updated to the current system. I may get some time this
> > week, but sasmodels takes priority.
> >
> > On Sun, Mar 13, 2016 at 10:52 PM, bmaranville <notifications@github.com>
> > wrote:
> >
> > > I am planning to pass the input datasets to each fieldUI generator.
> > > "multiple" fields will become a table or list of inputs of the same
> > length
> > > as the number of datasets in.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> > <
> https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16671534
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16680487
> >.
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/reflectometry/reduction/commit/407daa143e76f637f44b6f1a4190804d1fc183b9#commitcomment-16680769>
> .
>


since this is a bool, rename it to isBuyer is better


i'm ok with any change, just don't get what the value should be then that boolean cast breaks it. it should be '1' or '' then? and that is afaik exactly what boolean cast does


ditto


Nope. It's a self-assigned declaration. Resolver has nothing to do with this, this test didn't belong there.


For the integration test, yes. Just copy
src/haxelib/server/dbconfig.json.example to www/dbconfig.json.
The website should work without this though, since it is copied
automatically when building the image.

On Mon, Mar 14, 2016 at 10:07 PM, Dan Korostelev <notifications@github.com>
wrote:

> Should I manually prepare www/dbconfig.json?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HaxeFoundation/haxelib/commit/1f5b723b5956fb04ec984775d981fdcb12a32877#commitcomment-16680984>
> .
>


hej @anasofiapinho good thinking! :sparkles: could you add "CET" to the time?


@umiyosh Are you okay with me incorporating some of your commits into the upstream repository? I realized Doxygen updated its installation process with 1.8.10, so I may update to support that process when I have time.


It's not a big deal, but it's a good skill to learn. Before we meet, check
out how we fixed Ari's repo.

J
On Sat, Mar 12, 2016 at 1:59 PM Adam <notifications@github.com> wrote:

> Hey Jason,
>
> I just saw this message. What can I do to rectify this problem? Can we talk
> about it on Monday?
>
> Thanks,
>
> Adam
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/adamskriger/birthdays/commit/02d5d89881319afb15f12141623854c327570450#commitcomment-16663772>
> .
>


as for type, float64 is better


The Teams App's clock runs on UTC – so we'd actually need UTC / GMT  :)


Right, that was a mistake


A slightly better test is `expect_is(bind_rows(d1, d1), "data.frame")`


ha! good to know! so we'll use UTC then.


yup


Although I guess in this case you want to make an exact assertion about the class string


put all vmturbo api related method to a separate file, together with the VMTApiRequestHandler definition


Why there is a parseInt alone and the information is not being saved in some variable?


On Sat, 12 Mar 2016 07:00:26 -0800, Gustavo Sverzut Barbieri wrote:
> +1, I'd rather keep those in our generated .h files

Not now, I will clean up and simplify things first, only then add new
files, but yes, we probably want to keep those somewhere during the
build in the end.


D3D12_RANGE is defined to have end - start = size, so the last -1 should be removerd


On Debian-based systems, `apt-get update` updates the packaging metadata, but doesn't actually install any upgrades.

You'll want:
```sh
apt-get update -qq
apt-get dist-upgrade -qqy
```


@RubenVerborgh, touchpads are just too sensitive these days and you wouldn’t want to spoil you slides’ content by accident touch. No one really uses mouses on stage, mostly laptops with touchpads. This feature of course could be limited to the list mode, but in this case it will interfere with expected scrolling behaviour just like it does currently with spacebar. So it might be carefully tested again if there’s someone interested enough, but it should work fine both with mouses and touchpads which is rather impossible, AFAIK.


赞！就是想有这样的效果^_^


Appart from that LGTM (I guess adding test is difficult)


cf. issue #27


repeated code... obtain a paratenDisplay variable before the if


This will create a filename ending in a dot such as: `0.` Could you enhance the tests so we're also covering filenames without a dot such as: `0`? It would also be good to check that the 'Could not find a parser for the file extension' message is displayed for invalid formats, but not for valid formats. This test would fail without the fix included in this patch.


I also vote for having the warning message.


Hey thanks for your patch. I think it would be better if you do not repeat instructions `make thirdparty-update` is the same for both.


Why have a `cancellable` parameter if it's explicitly marked unused?


`Class1,DoSomething` has no object instance of type `Class2`. This test should only pass if `foo` is declared in a standard module; `foo` in `Class1` shouldn't resolve to anything here.


remove empty line


this is duplicated code


initialisation is ver un-javaish


correct , nice!


I think moving the variables into their relative files would be a good change as well.  It does attempt to resemble frameworks in its rigidity and it would make sense to not rely on those variables outside of the modules.
We were already making these assumptions, but also in a non-obvious spot.  A majority of them were set in `base/_defaults` which could now remain untouched for most cases. Many of these variables make sense to be reusable.

The media query list was moved out of options in the prior commit as it was not optional.


I'm afraid we need a Soletta enum for these things, it's too Linux-specific, no?


Same as above: `Class2.foo` is public, but it's defined in a class module; `foo` in `Class1.DoSomething` should resolve to a `null`, leaving `foo` without references.


Undefined variable: besparelse


Disregard, it's static.


The separate condition for Chrome is causing problems for me on the current google-chrome-beta (50.0.2661.26). If I remove the condition and use the setTimeout code in Chrome 50, everything seems to be OK.

Also, the setTimeout code seems to be compatible with current google-chrome stable (49.0.2623.87)


oki, tak aspon nazev te konstanty dej HEUREKA_CZ aby teda obsahoval nazev te sluzby..


displayAnnotationIsPresent -> isDisplayAnnotationIsPresent


static import


SATA SSDs only support 32 (31 effective) tags, but lots of other devices support higher depth. That said, the 256 in the example is a bit extreme, it should be scaled down. I don't know what the JESD endurance test mandates, I'd have to look that up.


@amuniz Please choose: Rename of the repo and issue tracker component, or rename of the artifactId. I suggest the latter, to better align display name and artifact ID, and make the plugin actually discoverable in JIRA.

Thanks @jglick


The commands didn't change here.


May I ask why delete Fail2ban function? Do we need Firewall for MailCow? Thank you.


你怎么把markdown语法去掉了，没有语法，阅读起来不好啊。


@chleh The problem is that we had discussed before:
The last entry of the ctest PETSc ksp settings were not taken by the ctest. Therefore the reference results of  ``GroundWaterFlow3D_NeumannBC`` was obtained by the default ksp settings in the previous ogs6 as
``
cg with bjacobi
``
On the contrary, the ksp setting in the Christoph's commit is given in the project file as
``
gmres with bjacobi
``
@endJunction
I am going to update the reference results of master branch of ogs6-data repository now.


Could simplify this to just `return glnx_shutil_rm_rf_at(...)`.


sdfsdf


I wonder why would one choose this download path instead of the cloning one. You don't set the git origin repo to point to our upstream in your instructions and why not ``unzip``?


space needed


I do not believe the above code is correct.  In my testing, __lzcnt64() is not available on 32 bit modes.  I think the correct code would be:

```C
#ifdef WIN32
#include <intrin.h>
#if __x86_64__
#define ntz(x) (__lzcnt64(x)-1)
#else
#define ntz(x) (__lzcnt(x)+__lzcnt(*(((unsigned*)x)+1))-1)
#endif /* __x86_64__ */
#else /* ! WIN32 */
```

Or something similar.


git rm --cached arquivo


In that case I have to wait until https://github.com/official-stockfish/Stockfish/pull/600 is merged into master, because I want to test against this patch.


This is the Javascript snippet for "classic" Google Analytics, but Pombola uses the newer Universal Analytics. See https://github.com/mysociety/pombola/blob/44a5507ec1f675df0baad903d1dd6eb26c8fca7f/pombola/templates/default_base.html#L68-L74 for the right snippet...


There is no reason to create a new listener each time the holder is bound to a new crime. You could create the listener just once when the holder is created. When the holder is bound, the value of `mCrime` is changed, which would change the behavior of the listener without the lister itself changing at all.


Would be better:
```
Lint.prerequisite_fail_msg = "'{}' shpuld be a list.".format(repr(Lint.prerequisite_command))
```


oh I see.
I will approve.


Forgot about that patch waiting for the merge...sorry.


yep, using this method you do no require have git installed, but i have missed the `git init`. I guess who is following this path is probably not interested in using git.


Addresses issue #19


TeamCity Emplea.do :: Build - Pull Requests [Build 29](http://137.116.74.213:8111/viewLog.html?buildId=89&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 30](http://137.116.74.213:8111/viewLog.html?buildId=90&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 29](http://137.116.74.213:8111/viewLog.html?buildId=89&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 31](http://137.116.74.213:8111/viewLog.html?buildId=91&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 30](http://137.116.74.213:8111/viewLog.html?buildId=90&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 31](http://137.116.74.213:8111/viewLog.html?buildId=91&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


The group name inside Doxygen has to be a single token, but that's not true for the name to be displayed at the end (change the 2nd CoAP_Transport_IP to "CoAP Transport IP"). Also, why not IP transport instead of transport IP?


I don't see how.. Only if you send a ChunkedMessageEnd before a ChunkedMessageStart.. But I see the point. Let's be conservative here.


you win...


It's because containerd isn't remote on Windows.  It's implicitly implemented inside libcontainerd. I've factored out cleanup, and ultimately will be removing remote entirely from the Windows path as it is not required.


Iäm terribly sorry about the name of this commit. I have no idea what I was thinking. May have been drunk.


Edit comments


the idea was that you either want to provide static list of caches/configs (in which case dynamic = false) or none (in which case dynamic = true) in other constructors. This one is preconfigured caches only. With this signature we enforce at least 1 config, and optional additional ones (as vararg):

```java
Bucket client1;
Bucket client2;
new CouchbaseCacheManager(new Config("first", client1, 200),
    new Config("second", client2, 300),
    new Config("thirdWithDefaultTtl", client1, null));
```
If two configs with same name are in the list, only the last one is taken into account (as would be the case probably with a Builder as well).


Addresses #20


Add test case about S_FALSE(Expected as no error)


It fixed the hang for shamu. How can I keep this commit if u remove it?


Aye, I'll pipe the whole string to sed, that should do it: `sed 's,:,|,g' <<< "addhl -group /dlang/code regex …"`


As for the execroot bit, that's to write the spec out on Linux. That's not required on Windows as there's no process transition boundary to pass the spec off to.


Would this potentially fail (due to undefined behaviour when overlapping regions are passed to `memcpy`) if `data` is given as a pointer in to the original aux string (for example, doing an update to set the value to the second part of the current tag value)? https://github.com/jenniferliddle/htslib/commit/dd86fd6c4352f369caf745994b0b4dcd433c1af2#diff-ca6081b9142d46d6855f0b9b9a8ee469R1305


This is incorrect, Renewal_t1 can never be passed as an option, the value is unimported, same for Ip_lease_time and Rebinding_t2, would be nice to have 1 test that tries 3 different broken configs, one with Renewal_t1, another with Rebinding_t2 and another with Ip_lease_time.


TeamCity Emplea.do :: Build - Pull Requests [Build 32](http://137.116.74.213:8111/viewLog.html?buildId=92&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 33](http://137.116.74.213:8111/viewLog.html?buildId=93&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 32](http://137.116.74.213:8111/viewLog.html?buildId=92&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:04


TeamCity Emplea.do :: Build - Pull Requests [Build 34](http://137.116.74.213:8111/viewLog.html?buildId=94&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 33](http://137.116.74.213:8111/viewLog.html?buildId=93&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 34](http://137.116.74.213:8111/viewLog.html?buildId=94&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


Couldn't you simplify things by eliminating the need for the 'IF' conditions - the 2 different scenarios [  _csvAutoStart: function () **vs.** _csvExportForm: function () ] - and having to keep writing/tweaking the code to accommodate one or the other - by just always using the code that is used for the _csvExportForm: function () ??? (And just never display the form for selecting the options if not needed )

That code in the  _csvExportForm function is the code that correctly includes the joined tables elements in the correct order specified in the 'Included elements'. Otherwise, if the Front end option is not used ( _csvAutoStart is used ), then the elements from the joined table are always put in the last columns.

Please see this thread [http://fabrikar.com/forums/index.php?threads/list-csv-import-export-problems-and-discussion.43508/](url) for a list of the quirks I've found with the List CSV Export/Import.


Moved file `openshift` to `open_shift` to enable dynamic loading of files.


Same as above.


this commit looks clean


whitespace diff.


1) why is this css not in a base.sass file as well?
2) is the filter property really necessary? The web backend does not support IE<9 anyway


great !


TeamCity Emplea.do :: Build - Pull Requests [Build 35](http://137.116.74.213:8111/viewLog.html?buildId=96&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 36](http://137.116.74.213:8111/viewLog.html?buildId=97&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


TeamCity Emplea.do :: Build - Pull Requests [Build 35](http://137.116.74.213:8111/viewLog.html?buildId=96&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:04


@endJunction I have already updated the reference of GroundWaterFlow3D_NeumannBC locally? Can I update master of ogs6-data repository directly?


TeamCity Emplea.do :: Build - Pull Requests [Build 37](http://137.116.74.213:8111/viewLog.html?buildId=98&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) is now running


it was missing a check on the second array element, after `.split`.


TeamCity Emplea.do :: Build - Pull Requests [Build 36](http://137.116.74.213:8111/viewLog.html?buildId=97&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


TeamCity Emplea.do :: Build - Pull Requests [Build 37](http://137.116.74.213:8111/viewLog.html?buildId=98&buildTypeId=EmpleaDo_DeploymentToQa&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


Now the results are obtained with
``
gmres with bjacobi
``


I don't see a difference in shadows...


are we guaranteed that ace is loaded when we reach this?


You do not need to do that anymore if you use the dev-master version of the admin bundle (which is bad). Only if you use a stable version, until 2.3.8


@LGA1150  @andytimes  不，虽然我不知道有IPV6的话能做什么，但我确定没有IPv6，我们学校并不是校园网，是联通的6M宽带，一个月50软妹币TAT感觉自己好多要学的- -！PS：我是汉子


need rental check
`if (rental.getCar().getID().equal(CarID) && rental.getStatus().equal(String.valueOf(StatusEnum.Rented))
     ....`


Should the download button go next to the Basic Data Summary Table, instead of by the title?


@jarednova You're exposing your license key here buddy. Just sayin. ;-)


Нужен комментарий к коду. А то так просто не разберешься...


Actually we're using it wrong. We should get rid of the `_get_tagged_name` and develop `_initialize_result_dir`, which should try/catch create the directory until it succeeds. Then it should return the `sage_tagged_name`.


"Logowanie"


same for me, since the last update it's broken and with a broken roster it's no longer usable for us.


This change causes an immediate because you can't cast pMsgData to a json_object* and expect it to work.

While I'm testing locally, I've patched old ppString = (uchar**)pMsgData straight into the function which fixes the issue.

I'm happy to open a PR to fix this and add a test somewhere, but could you explain why ppString is no longer a thing?


PS: This fix is good, the proposal above was for the future...


why get the error `unknown func startify#fortune#get_random_quote()`


Tutaj na polski również :)


I see it, not too noticeable. Can the shadow be red?


Why not put ``valid_token_count`` into the source Endpoint layout?


You shouldn't need to catch errors and report.  QUnit will report uncaught errors.
Instead, just do assertions on AddMonths result.


I think it is a good idea to avoid mixing a **relative** width with a **absolute** padding. It can lead to different behaviour depending of the width of the container (i.e. scrollbars).


closes #1


Look out for my test


why are they chained sequentially and not in parallel?


I agree.
Sorry, but I didn't fully understood your suggestion as I'm still a newbie and I still haven't figured out all the Journal code. Do you suggest that I create an instance of the `journal::Journaler` and invoke its `get_tags` method (from my understanding this will not start the replay, but will initialize the Journaler internal data structures); or for instance, invoke directly the `cls::journal::client::tag_list` method? (Both these options would run from a new static method in `librbd::Journal`)


this commit makes sense; a function rename and some tidying up


If we changed this to
``` python
raise BuildError(
              "Building %s: 'activities' matrix is all zero for %s. "
              "This is because no evaluation points fall in the firing "
              "ranges of any neurons." % (conn, conn.pre_obj))
```
then I think we could remove the `try/except` above.


dsadas


what's so different? just updating the formatting, just not much actual codefix...


The main disadvantage of the Hexen format is that the map names are forced to be in the MAPxx format.

... I have 9 episodes with 111 maps to juggle :D

May I suggest a ZDoom-style MAPINFO with only the barebones properties?


We do this because we don't have access to `conn` here.


Ah although we don't have the conn obj here, so the error message wouldn't be quite as good.


EBEN VAN DYK


I think the `searchInput` occurs in many files and could be worth to move into its on **.jspf**-file. (In the medium run, although it might not require much effort. Please create an issue if you decide to postpone it.)


notebook work well bc it doesn't take input as a reference variable


It indeed checks returns like how was being done previously. I'd say it's OK, go for it.


I don't see how to do it in an other way.
Note that I'm using it to overload mailboxManager.


the entry is removed (if it already exists) only because it's being set in the code that follows... this way a double entry is prevented


I would be more explicit here and check for rugby as well and throw an IllegalArgumentException if it's neither.


Includes fix for #22


The `MAX_MEM_ALLOC_SIZE` is a prototype yet.
@matze it would be great if you could add this flag to the existing information of GPU nodes. To be on the safe side, we shouldn't try to allocate buffers larger than [CL_DEVICE_MAX_MEM_ALLOC_SIZE](https://www.khronos.org/registry/cl/sdk/1.0/docs/man/xhtml/clGetDeviceInfo.html).


不要になった変数定義の削除。


@danfang @wheresrhys Just fyi that npm hasn't pulled in this change and is still showing the outdated README. I don't know if there is a way to force a change, but just a heads up.


add a new line before pmSupplyChainNodeBuilder


不要な変数の削除。


the FF equivalence testing looks thorough, that should give us the confidence we need

changes to FF just involve the management of deprecation


「l」を使用しない（癖でしたすみません）


timeZone取得関数へ移動。


「ｌ」を使わない。


@saarangsoltani please check my work and let me know what you think
thanks,
Samira


cool


Evan - I think we can skip that line going down the enter of the tables. I did something as a mock up yesterday and it looked kinda silly.

Also the max-width for station class I fixed with using the column row class built in with Materialize. So working on a way to get the outgoing/incoming header to be fixed when the user scrolls past it will be perfect!

Nate


Addresses #24


userNameをサニタイズ。


minimal


getCommentedDate関数に切り出し。


Steps `3b`, `3c`, etc. follow step `4` and `4a` (don't know if this was on purpose).


You need to use %n rather than %1 here and below for correct plural handling.


Do you still need component and autowired?


Missing space in `suites_failed =0`.


コメントをサニタイズ。


brand new test module


ご卒業おめでとうございます！！


Addresses #26


contribution 500, yay me!!!


コメント時間整形関数。


>i can save and reload my game now !

Nice.

> Question: this new code has an impact on the gameplay/mod ?

No gameplay changes, just slightly faster lookup times for global variables. Should not be noticable at all though.

>Reinitialised : Special Keys Statistics Report

I have no idea. Never heard of that.


rename ff > _ff


タイムゾーン予測出力関数。


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 19039](https://teamcity.mesosphere.io/viewLog.html?buildId=163958&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:17


コメント投稿フォーム出力関数。


I don't understand what you mean with "causes an immediate". In any case, we changed the interface because many compilers complained about the different alignment of uchar** and e.g. json_object*. As void is guaranteed to be properly aligned for all needs, this solves that issue. On some exotic platforms, the cast when using uchar ** could actually fail, so this was a real issue that we solved. Note that this was done some weeks ago, it was just omjournal that got forgotten, which turned up when we made it build under CI.


We're using unsigned int, not just unsigned


I would like to add out that Rosetta’s licence prohibits derivative work, so no you would not be able to do that. I think it is best if you stop shooting these random thoughts in line comments and send a proper email about these important matters. There I am more than happy to discuss, until then…


@todderasesareddot and @bhaux :

You should cherry pick this fix.


either `return` or `await` here. as this method returns a promise


Did I see somewhere that this needs to pinned to `~2.2.0`


+1 warning msg


The point releases of Ubuntu 14.04 (after .1) include an upgraded kernel by default. If you want to have an older kernel, you'll need to install from the original 14.04 ISO (as per [the 14.04.3 release notes](https://wiki.ubuntu.com/TrustyTahr/ReleaseNotes/14.04.3)).

This is available [here](http://old-releases.ubuntu.com/releases/14.04.0/ubuntu-14.04-server-amd64.iso). You'll notice old-releases in the URL; generally we want users installing on to hardware to use the latest HWE kernel to avoid spurious bug reports, so the older ISOs are archived. Rest assured that (a) the kernel in this ISO is supported for trusty's full lifetime, and (b) this is a long-term archive; these images won't disappear.

(This does, however, mean that upgrading all packages after installing from the ISO is _imperative_, as you will be vulnerable to all sorts of nasty security things that have appeared (and been fixed) since 14.04 dropped.)