The following day we took leave of our hosts. We decided toreturn, inasmuch as our mission was accomplished; and TushegounLama explained to us that he would "move through space." Hewandered over all Mongolia, lived both in the single, simple yurtaof the shepherd and hunter and in the splendid tents of the princesand tribal chiefs, surrounded by deep veneration and panic-fear,enticing and cementing to him rich and poor alike with his miraclesand prophecies. When bidding us adieu, the Kalmuck sorcerer slylysmiled and said:"Do not give any information about me to the Chinese Cheap Jordans authorities."Afterwards he added: "What happened to you yesterday evening was afutile demonstration. You Europeans will not recognize that wedark-minded nomads possess the powers of mysterious science. Ifyou could only see the miracles Diesel Jeans and power of the Most Holy TashiLama, href="http://www.noltecanada.com/">Canada Goose when at his command the lamps and candles before the ancientstatue of Buddha light themselves and when the ikons of the godsbegin to speak and prophesy! But there exists a more powerful andmore holy man. . .""Is it the King Livres of the World in Agharti?" I interrupted.He stared and glanced at me in amazement."Have you heard Pittstown New York Website about him?" Coach Outelt he asked, as his brows knit in thought.After a few seconds he raised his narrow eyes and said: "Only oneman knows his Cheap Jordans

 holy name; only one man now living was ever inAgharti. That is I. This is the reason why the Most Holy DalaiLama has honored me and why the Living Buddha in Urga fears me. Stiftungsneuigkeiten Solingen undefined


> To me, your `Z_RAISE_FOR_OBSTACLES` seems more like the latter

This is just a redo of a PR from @AnHardt. You can blame the terrible naming on that crazy fool. The general concept is sound, and that is what we need to evaluate first. Specifically, we should focus on whether the changes to `Marlin_main.cpp` are complete and correct, and I will be happy to change the naming of these settings to whatever best fits their usage.


formrows.py 的功能和效果：点击按钮browser，弹出一个对话框，选择一个文件（也可以直接在输入框中输入  路径／文件名），文件名显示在输入框中，并且函数返回文件名。


Objective function:
obj = sum(sum(abs(U))) - 0.8*sum(rho);

Don't forget! We could change the W vector to have 4 variables, corresponding a x_min, x_max, y_min and y_max. It will make it possible to have more than one size unknown obstacles.


I think with this we should update the ORM min requirements to 0.5.0 for this one, then tag this release as 0.5.0


@trq


See https://github.com/hadley/ggplot2/issues/1264


RoomAirflowNetwork (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1798 of 1798 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)


we don't have a common core module like worldedit, so even if we still have both they will be entirely separate codebases/maven projects/whatever.


my b, my b


        formrows.py 的功能和效果：点击按钮browser，弹出一个对话框，选择一个文件（也可以直接在输入框中输入 路径／文件名），文件名显示在输入框中，并且函数返回文件名。
        可直接测试：python formrows.py


絆持ち本人から、フォームがちゃんと見られない。という不具合がありました。


@paf31 You are definitely right. If `cas` is empty it should be handled by `foldl'`, apart for the non-correlation between them.

What I'm thinking is this could be another problem: if `cas` is empty it's handled by `foldl'`, otherwise `cas` has an element, and I can use `head` to see it and calculate the number of arguments that the definition has. So it will never fail, right?

Sorry if I'm bothering a lot, but according to your comment on the issue:
```
You can get the length of the list of arguments from the Case statement and pass it as a function argument.
```
I'm not actually doing that in `numArgs` using `length` on `caseAlternativeBinders`?


Super. Will delete this repo.


Corrigé...: https://github.com/infra-geo-ouverte/igo/commit/f1a6525d7f36d7ecb62b8c92b2c3982f237f4d23


naming: pre_interval_pre_interval


Good. As long as this is in the test suite, then the relevant functionality will be tested.


Awesome. Thanks for the review. I'll merge and tag.


Git changes line endings for text files; sometimes it mistakes binaries for text files and therefore replaces "line endings" within the binary. I updated the .gitattributes to not treat dll files as text, hope it helps.


%u=>%d: if -1, means the key does not exist in related hashmap. Good for statistics.


get_fields_list() and get_users_list() in notifications_setup_view.py should have the @login_required decorator so they can't be without authentication.


This seems needlessly complicated.  If you're going to hard-code the limit to 65535, then why not hard-code it within `read_pbm_integer()` and save all of the argument passing?


add md


instant msg?


not sure whether these changes are to blame but i'm getting lots of insert errors for movieinfo around not having default values - plot, actors, tagline etc as an example. also tmdbid "incorrect default value 'NULL' for an integer"


実行方法が変わっているのであれば、READMEも直して下さい。


note? comment?


This commit message sounds like the name of a prog rock album.


a true genius


Was the console logging slowing the system down?


so this still needs "-M"? Just checking


TurnOffWarmupReportForRevDD (Myoldmopar) - 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)


did it earn 200 points or 100?


Also, this patch does *not* fix #141 in my testing.


:heart:


condition_map.collision_times should be moved to data_warehouse_t, as the condition_rotator thread always rotate between the 2 condition_maps.


Hrm, getting an error in my dev enviroment due to the guid.h

```
Severity	Code	Description	Project	Line
Error	C1083	Cannot open include file: 'guid.h': No such file or directory	Kodi	  31
```


It is really outdated. This year I'm dropping gulp in favor of webpack.
But thanks!


Moving this to the bottom means it won't be used at all. MsHid is available on all platforms, so if the factories are checked for availability from top to bottom, it'll always pick MsHid on Windows instead of MsBluetooth. The down-side of this is MsHid doesn't support automatically connecting to the Wiimote.

Is it for you the case that WiiDeviceLibrary does not work when MsBluetooth is placed above MsHid?


Isn’t this cheating? :)


If, and only if, Ruby goes OOM due to too many users, then maybe you'll need to retrieve only ids and then instantiate users on the go. something like:
```ruby
Carto::User.all.pluck(:id).each { |user_id|
user = ::User.[user_id]
 # current code
}
```


- user_profile_id は self から取れそうです
- ただ、ここはテーブル全体の検索になるので、クラスメソッドにするのがよさそうな感じです
- さらに @user_profile.total_correct で正解数を取得できるようになるといいかもです


Agreed. I send you a PR with the corresponding update.


What is that space in the commit message?


```
- PUT Object - Copy - Amazon Simple Storage Service :
  http://docs.aws.amazon.com/AmazonS3/latest/API/RESTO   bjectCOPY.html
```


I'm not to use an automatic detection.
I fact if you have Qt4 and Qt5 installed with debian packages, you'll have /usr/include/qt4/QtCore/QtCore, but also /usr/include/qt5/QtCore/QtCore.
Which one the user want to select ? difficult to select automatically.
If we modify to match to QTDIR variable, we can change to QT_INCLUDE_SUFFIX, which can be set to qt4 or qt5 (by default empty). it will join the include path with: QTDIR/include/QT_INCLUDE_SUFFIX.
Are you agreed with that ?


Apart from my concerns about OOMing and leaving too many open connections, the rake itself is fine. If you want more help just shout me


RpcResponseEnvelope


Can you assert that the above is not voided before the web service call?


I updated the title of #269 to include `GetSections()` in our considerations.


cool


Well, other than `Player::getAvailableSubtitleStreams` and `Player::getAvailableAudioStreams` are there any?


Fixed ceylon/ceylon-ide-eclipse#1439.


Added PR #16 16


This class should be removed.


Looks like you misspelled "canola"!


On 08/14/2015 10:58 AM, Sam Batschelet (hexfusion) wrote:
> I guess here I should use Data::Transpose::Validator
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/interchange/Perl-Dancer-Conference/commit/5ca4273261dbf892190b47f67ed75ea02d59523a#commitcomment-12707295>.
>

Yes.

--
Perl and Dancer Development

Visit our Perl::Dancer conference 2015.
More information on https://www.perl.dance.


You are right. I only see the diff i misinterpreted it.


Just curious: I thought part deletion is not based on partId ( https://github.com/basho/riak_cs/issues/955 ), but was I wrong?


You are absolutely right, I should have tested this more rigorously before blindly committing it!

I'm developing a fix now...


why?


ok


its good


フォームの項目名をセットしています。


updateFormDataはデータの編集時にDBに保存する処理になります。


good!


@davidfowl as I explained in a previous comment `GetSection(key)` can be used to retrieve a section at any arbitrary level with a relative key path while `GetChildren()` can only be used to retrieve the direct descendants of the current node. You can think of `GetChildren()` as `GetChildSections()` although as I also explained in a previous comment these are the names we have agreed upon with a larger group (including the BCL folks) so I want us to stick to these names in this PR and not change them until we have a chance to further discuss them again.


WOOHOO, FINALLY!


Add apache license header. Please ignore if already added


Ugh, this is a difference in behavior between EMR and the local devstack (which explains why the acceptance tests didn't fail):

on EMR:
```
mulby@emrmaster:~$ sudo -u hadoop /bin/bash -c 'echo $HOME'
/home/hadoop
```

Apparently there is a global setting which specifies whether the `-H` flag is necessary or not. It can't hurt, though, so I'll put it into every call I think.


Besides the inline comments, please add license header in the new files being added. Please ignore if already added.


OK, I'm changing it to Carto::User but how lighter is for this case? I did a quick test with, I have 4 users in my local dev environment, and at least for the initial `Carto::User.all` it does more queries than `User.all`. After that they do the same amount.


Why are we sending a fake UUID? Can it be removed and handled in the caller if it is expecting it?


Great! By the way, is there an option to know which stream I process, because I need set messages from STDERR as errors and from STDOUT as warnings.


colum is default now, needs the overlay to get 2 :+1:


Nah, check my other comments, inside use `::User` (old model) as for dropping connections is the only one with that method right now


Indeed the loudnessValidator app can be built with both qt4 or qt5.
Maybe the simpler solution is to add a QT_VERSION build parameter to select qt4 or qt5 (by default qt4, which is the most tested version with the app).


```js
helpers.exec(..., {stream:'both'}).then(function(contents){
  console.log(contents.stdout, contents.stderr)
})
```


too late :)


In an earlier commit this routine was added. Since we are not using it anymore, can it be removed from Utils if it isn't being used anymore.


kkkkkkkkkkk


ok


The label `done` does not exist; the code does not compile anymore. Did you mean `end`?


you need to build your depends again.


FIX "wave: README: GearCM 11 20150814 Stable"


Please keep the alignment


This is longer talk especially that I spent few hours on it just yesterday. I said on meeting that the code is not optimal. And I review this part once more.
Referencing to `Long` versus `Int` the second is enough. It is created every time when `RetainedStackManager` is instantiated and I don't assume that there will someone collecting >4bln frames during one debug session


43523 - [Conjured Mana Strudel], Only mana buff applys


Did you mean `|=`?


Wow, this is terrible man. Really bad.


Hey Andrew,
Just found out in 5.4 they introduced this shortened syntax for declaring arrays, I didn't know.
I was getting errors. We must be on 5.3. Need to upgrade!
All good, I just modified my local file.


やってみます！


typo: AUDIO_SAMPE_SIZE -> AUDIO_SAMPLE_SIZE


FixSlabTransition (Myoldmopar) - 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)


It's not the non-existing filename that's the problem, it's the non-existing directory requested.


Instead of passing isHaEnabled and isSystemVm as parameter, why not define a enum for priority; high, medium and low. You can pass this priority as a parameter than.


> and I don't assume that there will someone collecting >4bln frames during one debug session
Famous last words :)

It's only 2b, since it's a signed int. It might be enough, but Akka [claims 50 million messages/second](http://letitcrash.com/post/20397701710/50-million-messages-per-second-on-a-single), so if we assume only half of that, an Int gives you about 85 seconds of debug time before an overflow.


> I'm sorry but I don't understand what are you talking about.
TR_* are just used as a value for return. Why are they related to CHECK_PART?

Indeed…  I didn't check myself, sorry for the additional noise.


Neinein, nicht im Base, bloss im Wettingen-Plugin!


...but I don't care so much about Int vs Long, more about why there's this logic necessary here, and not a simple lookup. I am only trying to understand, your code might be perfectly correct.


Yeah, but I meant to delete it, it is wrong in any case (though I had it out, since it is there, it might explain some of the bugs I saw when copying the object array) :)


Looks like you have already added license headers in all new files. Please ignore my earlier comment.


This is an unnecessary change.


PHPCS: Usage of ELSE IF is discouraged; use ELSEIF instead


No need for extra var:
```javascript
PTL.user.init({
  {% if user == object %}
  userData: {{ user.as_json|safe }},
  appRoot: '{{ user.get_absolute_url }}',
  {% endif %}
  lastEvent: {{ object.last_event.as_json|safe }},
});
```


Hello,

Okay, so the solution then is for me to run servald status after startup of
an instance to get the HTTP port out.  I'm happy enough for that.  We can
kill the commit for that change then, and just keep the OSX compilation
fixes.

Paul.

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

> 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
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/servalproject/serval-dna/commit/4dbd6f4695dfc606427249155dfdab3ffec41305#commitcomment-12707032>
> .
>


SignedOff


aja oops, die muessen natuerlich beide gleich lauten. Ich wuerds dort
aendern wo es am einfachsten ist.

2015-08-14 11:47 GMT+02:00 Max Mulatz <notifications@github.com>:

> @tawan <https://github.com/tawan> Könnte es sein, dass in
> mandrill_api_key.rb
> <https://github.com/klappradla/im_graetzl/blob/master/config/initializers/mandrill_api_key.rb>
> ein Typo ist? Auf ElasticBeanstalk haben wir als ENV-Variable nur
> *MANDRILL_KEY*, nicht *MANDRILL_API_KEY*.
>
> Würde zu dem Error passen den ich in den logs bekomme (missing mandrill
> api key, etc.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/klappradla/im_graetzl/commit/b12b61d9ce7f454e1610d5cd51b7b8cd68c01dac#commitcomment-12707150>
> .
>

--
*DI Tawan SIEREK*

Phone:    +43 650 700 20 12
Email:     tawan.sierek@gmail.com
Address: Margaretenstraße 61/3 | 1050 Vienna | Austria
Skype:   Tawan Sierek


in .bashrc
```sh
cat_pn() {
	awk -F/ '{print $(NF-1) "/" $(NF)}' <<<"${PWD}"
}
```

and then `repoman commit -m "$(cat_pn): bla"` maybe?


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.


Haha, ok, ich kümmer mich drum.

> On 14 Aug 2015, at 12:44, Tawan Sierek <notifications@github.com> wrote:
>
> aja oops, die muessen natuerlich beide gleich lauten. Ich wuerds dort
> aendern wo es am einfachsten ist.
>
> 2015-08-14 11:47 GMT+02:00 Max Mulatz <notifications@github.com>:
>
> > @tawan <https://github.com/tawan> Könnte es sein, dass in
> > mandrill_api_key.rb
> > <https://github.com/klappradla/im_graetzl/blob/master/config/initializers/mandrill_api_key.rb>
> > ein Typo ist? Auf ElasticBeanstalk haben wir als ENV-Variable nur
> > *MANDRILL_KEY*, nicht *MANDRILL_API_KEY*.
> >
> > Würde zu dem Error passen den ich in den logs bekomme (missing mandrill
> > api key, etc.)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/klappradla/im_graetzl/commit/b12b61d9ce7f454e1610d5cd51b7b8cd68c01dac#commitcomment-12707150>
> > .
> >
>
>
>
> --
> *DI Tawan SIEREK*
>
> Phone: +43 650 700 20 12
> Email: tawan.sierek@gmail.com
> Address: Margaretenstraße 61/3 | 1050 Vienna | Austria
> Skype: Tawan Sierek
> —
> Reply to this email directly or view it on GitHub <https://github.com/klappradla/im_graetzl/commit/b12b61d9ce7f454e1610d5cd51b7b8cd68c01dac#commitcomment-12707918>.
>


Why in WTMS required Jersey ? Spring already have full support for rest based application @Controller to @RestController make a mvc controller work like a REST endpoint @satyamashish  @sathishpgowda


So better to use old model than this?


啊喂！ 你们算了吧 = = 天天收到观光团的邮件很可怕的好不好=。 =


啊完全忘了...orz


Лучше via_get_reg


Shouldn’t you check here whether the return has an expression?


Awesome! Thank you!


I think this will not work, #define is compiled when you compile the code and CVAR_GET_FLOAT it's set when in-game level is loaded, this shouldn't even compile.


Mudar para: curl_setopt($ch, CURLOPT_USERAGENT,  self::USER_AGENT);


Mudar para: curl_setopt($ch, CURLOPT_USERAGENT, self::USER_AGENT);


you messed tabulation ;p
now it uses two `tab`s to tabulate


yeah, but I'm using negatives too and limited memory. I realize the code is tricky.
`ordinal = 3

key (msg_to_sent, 3) -> value {
AsyncStackFrame { bang$,  AsyncLocalVariable(msg_to_sent, 3) },
AsyncStackFrame { receive AsyncLocalVariable(msg_which_came, 3)}
}`
so now I click on `msg_which_came` and I have to find stack trace for `ordinal < 3`...

let me look at it again (especially why I look for local variable in a set of traces...


fix_intel_compile_linux (mbadams5) - x86_64-MacOS-10.9-clang: OK (1791 of 1791 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)


ok, de hol van a többi? :)


commits in `profiles` directory should prepend `profiles: ` to the summary


Something got mixed up here, action_support is required here, but not used, and it's used in other views (where it's not required).


RemoveMoreBadCharacters (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)


This should be
```
gtk-sharp-module.eclass: gmcs is derpecated and dropped, use mcs
```


This hasn't changed yet?


This hasn't been fixed either?


You're still iterating over the child views here.


w tym miejscu B[A[i]] jest ustawiane na true,
mozemy zamienic ta linike na taka
```
if(! B[A[i]]) { // jesli B od A[i] nie wystapilo wczesniej, to
   B[A[i]]=true; // zaznacz ze wystapilo juz
   distinct ++;// policz kojena 'nowa' liczbe
}

if(X==distinct) // mamy wszystkie liczby mozemy zakonczyc
```


WasmEncoderBuilder sounds a bit funny. WasmModuleBuilder?


yes it doesn't work because you broke the script ;p
see that `:` missing...


Caution: HtmlParser::ParseInPlace has significant side-effects (on which the following FindElementByNameNS relies)!


I want to be sure, what do yu mean by partId? UploadId or PartNumber or
part UUID.


RemoveMoreBadCharacters (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)


@Sean-Der This gave a warning about different levels of indirection.
https://github.com/Sean-Der/igbinary/blob/php7/igbinary.c#L2119
should be like
https://github.com/Sean-Der/igbinary/blob/php7/igbinary.c#L2090


Can you please note in the commit message that you modified the common driver header to accommodate winioctl necessary changes (am I right?)!


What is this for?  Why is this hard coded in?


Can we also have another version of this method that takes byte* start and byte* end? That would make it fit better with the existing tests that use the wasm-macro-gen.h functionality.


Anything wrong here?


We should only keep mimtypes of allowed file types.


This will be more readable either as a byte array or WASM_MACRO_EXPR().


Again, I am not following why we have keys embedded given test apps should be created automatically.  Also, these API keys are now out in the wild, so should be deleted from the Ably dashboard


updating isn't so hard now because people don't usually ask to add 100+ sites ;p


Why is this commented out?


Remove the indentation


updated in commit: c5ca8c7
thanks :+1:


Done here 7c7af30


Looking forward testing this :)


Done here ba8e0b4


Yea, we decided not to merge this.


Why exactly this version? Why not newer?


But why should it be changed? isChecked has nothing to do with the highlight, which is decided by the item.


More pythonic to write:
```
if qt4_include_suffix:
```


this should probably be removed?


@stevenday I think it's good to have this function in commonlib, but this code needs updating with the changes made in https://github.com/mysociety/alaveteli/commit/30cfd94c83702dee8a776c0dd4858dedd0adc755 (which has now been merged into the alaveteli develop branch):
- [ ] SQL statement in the `commonlib` PR needs a comma
- [ ] Needs to be called from `site-specific-install`, not `install-as-user`


I didnt manage to rewrite it using the collection view even though i tried. As long as the MenuView does it in the same way, cannot I leave it like that?


Hi starksk,

Good work. Loading a KML file works great!

One small note: You cannot specify a fully qualified local path or a local URL since Google needs to read the file. The UserMap must be an URL which is published on the internet.

You should change line:

UserMap     = "";               //Fully qualified local path or URL

into:

UserMap     = "";               //URL (publicly available domain)

I hope this helps you.

Ted


because they are not changed in sync ;-)


I basically copied auth tests from Rest client. This has to be changed.


I think it's safe to just remove the `\` and leave the message key blank. Should I just remove it and update the translation in crowdin?


Sorry Kostis, this is an aborted attempt to get proper to compile with
Erlang 18.
It was initially forked so I could use zotonic (which depends on proper),
but the latest zotonic works with Erlang 18. So I will delete this fork.
Thanks for all the great Erlang open source tools you and your team have
developed, by the way. I've always wanted to say this!

On Friday, August 14, 2015, Kostis Sagonas <notifications@github.com> wrote:

> You are of course free to do any change you feel like in your fork of
> PropEr, but I am just wondering: what exactly is the purpose of this change?
>
> The code snippets which appear within the NO_MODULES_IN_OPAQUES if-def
> macro are there only to be able to build PropEr with Erlang/OTP releases
> where these types are built-in (and not module-qualified).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/voila/proper/commit/2f9e06638cf6f555ad42c218cde9de611492524c#commitcomment-12707130>
> .
>


@Nadahar I don't think there was a real reason for updating antrun so feel free to revert it


Thanks for highlighting, we shall be converting it to RestController.


It was uncommented in a later commit.


fixed - thought it is nice to use it in the mixin, because botn MenuView and MultiselectMenuView use it


You are losing changes here (and I guess also in other places, the `send_notice` seems suspicious):
If I do `w:geometry{ width = 100 } w:geometry{ x = 10 }`, the drawin will only be moved, but the resize is lost.

I gave this a try and the following seems good enough for me. Note that I only checked in `drawin.c` for places where a pending resize has to be applied, but I can't really imagine places outside of drawin.c which might need this. (And I avoid creating a list of all drawins with pending changes on purpose. E.g. your code doesn't make sure that a drawin isn't on that list twice. Come on, iterating over all drawins in C isn't really bad. This is mostly bad in the "lots of cache misses" sense and optimizing on such a level doesn't make sense for everything except for tight loops that need lots of time.)

(Now that I just read through the diff again, I think that `drawin_moveresize()` is wrong: It should change `w->geometry` first and only afterwards emit signals, so that the other properties are already updated when the signals occur, similar to how this is done in `client_resize_do()`.)
```diff
diff --git a/event.h b/event.h
index 82f7fdd..e1ce8fe 100644
--- a/event.h
+++ b/event.h
@@ -31,6 +31,9 @@
 /* luaa.c */
 void luaA_emit_refresh(void);

+/* objects/drawin.c */
+void drawin_refresh(void);
+
 /* objects/client.c */
 void client_focus_refresh(void);

@@ -40,6 +43,7 @@ awesome_refresh(void)
     luaA_emit_refresh();
     banning_refresh();
     stack_refresh();
+    drawin_refresh();
     client_focus_refresh();
     return xcb_flush(globalconf.connection);
 }
diff --git a/objects/drawin.c b/objects/drawin.c
index b24240e..d5f03a2 100644
--- a/objects/drawin.c
+++ b/objects/drawin.c
@@ -34,6 +34,7 @@
 #include "drawin.h"
 #include "common/atoms.h"
 #include "common/xcursor.h"
+#include "event.h"
 #include "ewmh.h"
 #include "objects/client.h"
 #include "objects/screen.h"
@@ -139,6 +140,34 @@ drawin_refresh_pixmap(drawin_t *w)
     drawin_refresh_pixmap_partial(w, 0, 0, w->geometry.width, w->geometry.height);
 }

+static void
+drawin_apply_moveresize(drawin_t *w)
+{
+    if (!w->geometry_dirty)
+        return;
+
+    w->geometry_dirty = false;
+    client_ignore_enterleave_events();
+    xcb_configure_window(globalconf.connection, w->window,
+                         XCB_CONFIG_WINDOW_X | XCB_CONFIG_WINDOW_Y
+                         | XCB_CONFIG_WINDOW_WIDTH | XCB_CONFIG_WINDOW_HEIGHT,
+                         (const uint32_t [])
+                         {
+                             w->geometry.x,
+                             w->geometry.y,
+                             w->geometry.width,
+                             w->geometry.height
+                         });
+    client_restore_enterleave_events();
+}
+
+void
+drawin_refresh(void)
+{
+    foreach(item, globalconf.drawins)
+        drawin_apply_moveresize(*item);
+}
+
 /** Move and/or resize a drawin
  * \param L The Lua VM state.
  * \param udx The index of the drawin.
@@ -148,52 +177,33 @@ static void
 drawin_moveresize(lua_State *L, int udx, area_t geometry)
 {
     drawin_t *w = luaA_checkudata(L, udx, &drawin_class);
-    int number_of_vals = 0;
-    uint32_t moveresize_win_vals[4], mask_vals = 0;

     if(w->geometry.x != geometry.x)
     {
-        w->geometry.x = moveresize_win_vals[number_of_vals++] = geometry.x;
-        mask_vals |= XCB_CONFIG_WINDOW_X;
+        w->geometry.x = geometry.x;
+        luaA_object_emit_signal(L, udx, "property::x", 0);
     }

     if(w->geometry.y != geometry.y)
     {
-        w->geometry.y = moveresize_win_vals[number_of_vals++] = geometry.y;
-        mask_vals |= XCB_CONFIG_WINDOW_Y;
+        w->geometry.y = geometry.y;
+        luaA_object_emit_signal(L, udx, "property::y", 0);
     }

     if(geometry.width > 0 && w->geometry.width != geometry.width)
     {
-        w->geometry.width = moveresize_win_vals[number_of_vals++] = geometry.width;
-        mask_vals |= XCB_CONFIG_WINDOW_WIDTH;
+        w->geometry.width = geometry.width;
+        luaA_object_emit_signal(L, udx, "property::width", 0);
     }

     if(geometry.height > 0 && w->geometry.height != geometry.height)
     {
-        w->geometry.height = moveresize_win_vals[number_of_vals++] = geometry.height;
-        mask_vals |= XCB_CONFIG_WINDOW_HEIGHT;
+        w->geometry.height = geometry.height;
+        luaA_object_emit_signal(L, udx, "property::height", 0);
     }

+    w->geometry_dirty = true;
     drawin_update_drawing(L, udx);
-
-    /* Activate BMA */
-    client_ignore_enterleave_events();
-
-    if(mask_vals)
-        xcb_configure_window(globalconf.connection, w->window, mask_vals, moveresize_win_vals);
-
-    /* Deactivate BMA */
-    client_restore_enterleave_events();
-
-    if(mask_vals & XCB_CONFIG_WINDOW_X)
-        luaA_object_emit_signal(L, udx, "property::x", 0);
-    if(mask_vals & XCB_CONFIG_WINDOW_Y)
-        luaA_object_emit_signal(L, udx, "property::y", 0);
-    if(mask_vals & XCB_CONFIG_WINDOW_WIDTH)
-        luaA_object_emit_signal(L, udx, "property::width", 0);
-    if(mask_vals & XCB_CONFIG_WINDOW_HEIGHT)
-        luaA_object_emit_signal(L, udx, "property::height", 0);
 }

 /** Refresh the window content by copying its pixmap data to its window.
@@ -211,6 +221,9 @@ drawin_refresh_pixmap_partial(drawin_t *drawin,
     if (!drawin->drawable || !drawin->drawable->pixmap || !drawin->drawable->refreshed)
         return;

+    /* Make sure it really has the size it should have */
+    drawin_apply_moveresize(drawin);
+
     /* Make cairo do all pending drawing */
     cairo_surface_flush(drawin->drawable->surface);
     xcb_copy_area(globalconf.connection, drawin->drawable->pixmap,
@@ -224,6 +237,8 @@ drawin_map(lua_State *L, int widx)
     drawin_t *drawin = luaA_checkudata(L, widx, &drawin_class);
     /* Activate BMA */
     client_ignore_enterleave_events();
+    /* Apply any pending changes */
+    drawin_apply_moveresize(drawin);
     /* Map the drawin */
     xcb_map_window(globalconf.connection, drawin->window);
     /* Deactivate BMA */
@@ -317,6 +332,7 @@ drawin_allocator(lua_State *L)
     w->cursor = a_strdup("left_ptr");
     w->geometry.width = 1;
     w->geometry.height = 1;
+    w->geometry_dirty = false;
     w->type = _NET_WM_WINDOW_TYPE_NORMAL;

     drawable_allocator(L, (drawable_refresh_callback *) drawin_refresh_pixmap, w);
@@ -564,6 +580,10 @@ luaA_drawin_set_shape_bounding(lua_State *L, drawin_t *drawin)
     cairo_surface_t *surf = NULL;
     if(!lua_isnil(L, -1))
         surf = (cairo_surface_t *)lua_touserdata(L, -1);
+
+    /* The drawin might have been resized to a larger size. Apply that. */
+    drawin_apply_moveresize(drawin);
+
     xwindow_set_shape(drawin->window,
             drawin->geometry.width + 2*drawin->border_width,
             drawin->geometry.height + 2*drawin->border_width,
@@ -599,6 +619,10 @@ luaA_drawin_set_shape_clip(lua_State *L, drawin_t *drawin)
     cairo_surface_t *surf = NULL;
     if(!lua_isnil(L, -1))
         surf = (cairo_surface_t *)lua_touserdata(L, -1);
+
+    /* The drawin might have been resized to a larger size. Apply that. */
+    drawin_apply_moveresize(drawin);
+
     xwindow_set_shape(drawin->window, drawin->geometry.width, drawin->geometry.height,
             XCB_SHAPE_SK_CLIP, surf, 0);
     luaA_object_emit_signal(L, -3, "property::shape_clip", 0);
diff --git a/objects/drawin.h b/objects/drawin.h
index 29e0c83..1bd19bb 100644
--- a/objects/drawin.h
+++ b/objects/drawin.h
@@ -40,6 +40,8 @@ struct drawin_t
     drawable_t *drawable;
     /** The window geometry. */
     area_t geometry;
+    /** Do we have a pending geometry change that still needs to be applied? */
+    bool geometry_dirty;
 };

 ARRAY_FUNCS(drawin_t *, drawin, DO_NOTHING)
```


Redundant Information (Deleting approved)


Sorry for the post above. I forgot that the background is more than just a nicer name (described in #72).

So, I'm fine with that change.


I would not like to see this change going in, the script really only works for the repos in the crowbar/ org. if we at some point can make it reusable for another org (or at some point switch to two orgs), then we can revisit this.


Perfect.

There are many pseudo-classes in CSS, but I'd say the most useful are those related to link state (`:hover`, `:visited`, `:active`, etc.) and those related to collections (`:first-child`, `:last-child`, `:nth-child`, etc.) Beware though, some of those are not well supported: only the newest browsers implement them.

You can find useful information on the [Can I Use?](http://caniuse.com/) website. For instance, you've been using the `rem` unit and [here's its support table](http://caniuse.com/#search=rem): Internet Explorer 8 doesn't know about it (and IE8 is still widely used, especially in the corporate world where people can't always chose their browser, so it's something that must be taken into account—people usually provide fallback values in pixels).


Right. I meant why you have added the if() check. its always true..


Implemented mongos options in branch saintedlama/mongos https://github.com/mafintosh/mongojs/issues/241 would be awesome if you could give that version a try (= alpha test)


Warum nicht generell? Was hätte das für unerwünschte Konsequenzen?


too much? :)


Because you're manually having to sync duplicate state. Every single frontend framework out there was made specifically to avoid just that, because there's no way you can maintain doing that in a large application.
There are three pieces of state here:
 * the list of items, e.g. `['foo', 'bar', 'baz']`
 * the list of selected items, e.g. `['foo', 'baz']`
 * the highlighted item, e.g. `['bar']`

You only want to make changes to that state and let the views react to those changes, not make the changes to the state, and then repeat the same changes in views manually.

When you check or uncheck an item, you should only need to worry about updating the state, i.e. either call `removeObject` or `pushObject` on the array that hold the list of selected items (called `value` in this case, I think?).


Oh right, yeah I suppose so :)


fixed - iterating over items now. While changed the highlightItem type to item, forgot to rewrite the commented part :)


It seems that native pow does not behave as we expect it. It should return 1.0d, instead of NaN. I will certainly have to special case it then.


#http://nescent.github.io/popgenInfo/
#basic pop stats http://nescent.github.io/popgenInfo/StartSNP.html

library(poppr)
library(rgl)
library(pegas)
library(ape)
library(adegenet)
library(ade4)
library(gstudio)
D <- read.structure('C:/2015TIM/Analysis/R_directory(working)/Netwing_Anal/RADstats/nomisspops.str',onerowperind=F, n.ind=24, n.loc=198, col.lab=1, col.pop=2, ask=FALSE)

library(hierfstat)
library(genetics)
Mydata2 <- genind2hierfstat(D) #this doesnt work
??Genind2hierfstat
??hierfstat
Mydata3 <- genind2loci(D)
#http://nescent.github.io/popgenInfo/StartSNP.html
div <- summary(D)
plot(div$Hobs, xlab="Loci number", ylab="Observed Heterozygosity",
     main="Observed heterozygosity per locus")
plot(div$Hobs,div$Hexp, xlab="Hobs", ylab="Hexp",
     main="Expected heterozygosity in function of observed heterozygosity per locus")
bartlett.test(list(div$Hexp, div$Hobs))
#I can't convert to Mydata2
basicstat <- basic.stats(Mydata2, diploid = TRUE, digits = 2)
names(basicstat)
boot.ppfis(Mydata2)
x <- indpca(Mydata2)
plot(x, cex=0.7)

hw.test(Mydata3, B = 1000)#testing for hardy weinberg equilibrium
#2222222222222222222222222222222222222222222222222222222222222222222222222222222222
#http://nescent.github.io/popgenInfo/DifferentiationSNP.html

x2 <- as.genlight(D)

glPca for genlight objects
??genlight

#3333333333333333333333333333333333333333333333333333333333333333333333333333333333
#individual-based geentic distances
#http://nescent.github.io/popgenInfo/VignetteIndvidualGeneticDistancesSNP.html#section-1-convert-the-data

distgenEUCL <- dist(D, method = "euclidean", diag = FALSE, upper = FALSE, p = 2) #Individual genetic distance: euclidean distance (dist {adegenet})
hist(distgenEUCL)#The analysis is applied on allele frequency within individuals as represented in the genind object. We can use the function dist() from adegenet which provides different options. We will use the euclidean distance among vector of allele frequencies.
D2 <-genind2loci(D)#Individual genetic distance: number of loci for which individuals differ (dist.gene {ape})
distgenDIFF <- dist.gene(D2, method="pairwise", pairwise.deletion=FALSE, variance=FALSE)#The option pairwise.deletion = FALSE in the command distgen() removes all loci with one missing values : you an see on the histogram that we get a maximum distance of 3 loci out of 100. We can see that we get 98 loci with at least one sample missing. Then using the option pairwise.deletion = TRUE in the command distgen() allows you to keep loci with one missing value.
hist(distgenDIFF)

missing_data <- info_table(D, type = "missing")#ow much missing data
sum(missing_data["Total", 1:198] > 0)
barplot(missing_data["Total", 1:198], xlab = "Locus", ylab = "Percent Missing")

distgenDISS <- diss.dist(D, percent=FALSE, mat=FALSE)#number of allelic differences between two individuals
hist(distgenDISS)

#in order to

boxplot(distgenEUCL, distgenDIFF, distgenDISS)

#for sequence data
library(phylotools)
library(mmod)
myData1 <- read.phylip("Scarel_min5.85_21indivsnec_7i.phy") # this creates a DNAbin object
class(myData1)

### R code for;
# load diveRsity
library(diveRsity)
# calculate stats
diff_stats <- diffCalc(infile = "myfile.gen", outfile = "myresults", fst = TRUE,
                       pairwise = TRUE)
### END


this was imho mostly fallout of the org change imho... unless someone creates release/../../../../bla branch names in the future :-)


Hi Huzz,

As PL was cancelled today I can't give this my full attention.

When we tried this before I think this is where it fell down when we tried something in the Works.

It all worked fine until we ran two requests at the same time and then the scope would mix up the state of this shared object.

Hence the use of attaching functions to the prototypes and then "new"ing up a new object at runtime.


Thanks a lot for this change. Just a thing: seems there is a missing comma at ``` 'MALTA' 'MARTINIQUE' ```.


could you remove this commit?


the patch will exist, but it will likely be empty..


@jsquyres, have tou tested this? I think Perl's `system` will return 0 even if the `mkdir` command fails. You need to use `$?` to access the exit status of the `mkdir` command. It would be much better to call the Perl function `mkdir` directly.


Ah never mind I didn't see the use in close(), sorry about the noise


thats not a very useful log (its just the python exception). the real value is in the build log itself..


So If you write don't block the flow of error, then the built-in error handler of express will take the error and do exactly what you wrote; also, if the user writes its own handler, it gets the error from the last next and interrupts propagation to the eventually built in error handler. In other words, if you write next(err) the error will keep propagating to the (1) built in handler or (2) user handler - with this code, you are blocking propagation to either of those


fix_intel_compile_linux (mbadams5) - 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)


I would assume this would be `invisible()` due to convention.


@ccx-testerbot runtests


Inside a `<thead>` tag, the `<td>` tags should be replaced with [`<th>`](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/th) tags. Browsers won't complain if `<td>` tags are used because that's *valid*, but semantically speaking, using the proper tag is a good habit to make. It has some default styling that makes it more obvious it's a *heading* in the table, and it provides a better experience for many disabled users (see [Creating Accessible Tables](http://webaim.org/techniques/tables/data) for guidelines).


Okay, nvm. That's for data manipulators.


When using a `<thead>` tag, altough most browsers won't complain about it, it also cleaner to use a `<tbody>` tag to wrap all the `<tr>` tags containing the actual "content" of the table.


Use activity as postfix


You forgot to specify a branch, so I can't help you yet... Please try again by using "runtests {branch name}".


Wieso fragst du mich das jetzt und hier und nicht während des FaceTime-Anrufs vorhin als ich laut und deutlich gesagt hatte: „du musst dann halt das info.yml ins Wettingen-Plugin kopieren und da die allowed_roles hinzufügen“? Aber du hast da wahrscheinlich nur halb zugehört.

Aus folgenden Gründen:

* Weil ich nicht weiss, wie viele Sites es gibt, die bereits Nicht-Admin-User haben und ich nicht deren Rechte wegnehmen möchte beim nächsten Update. Dazu bräuchten wir eine Migration.
* Auch dann finde ich es eine schlechte Idee: so kann man den Usern Rechte auf Seiten zuweisen, ohne dass diese danach tatsächlich den betreffenden Seitenbaum editieren können.
* Wir haben bereits ein Release 1.5rc1 gemacht, da dürfen wir im 1.5-Branch keine Rückwärts-Inkompatiblen Änderungen mehr einführen.
* Wir hatten dieses Modul nicht ohne Grund und nicht ohne Diskussionen für alle Admin-Login-Enabled-User geöffnet und ich bin nicht bereit, dies ohne erneutes Abwägen aller Vor- und Nachteile für ein „es wäre schon, wenn“ für eine Site, zu opfern.


RemoveMoreBadCharacters (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)


@ccx-testerbot runtests master


Das sieht auch wieder nach einem Cherry-Pick statt einem Merge aus. Welchen Befehl hast du verwendet zum mergen? `git merge`?


s/grunt.js/Gruntfile.js/ ?


How about play-slick bumps to 2.3.9 and as well Slick itself in the 3.1.0 release? // @dotta


GABYhechs


Thank you for the request. It has been added to the queue (id: 44). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/view.php?id=44) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Danke. Ich habe das überhört, weil ich dann grad intensiv mit was anderem beschäftigt war.
Werde es wieder aus dem base entfernen.


We need to support, maintain and extend this code for many years to come. We can't afford to settle for "as long as it works, it's good enough". In the long run it *always* pays off to spend some extra time to do it the right way.


FYI, not suggesting you change anything as this looks really good, however if you do want to test Auth with a real URL request as opposed to a stubbed one, you can use our echo server we set up.  See https://github.com/ably/ably-js/commit/e77b2c6c197bc71f3d27084fe54524724a00bc92


Ja

> On 14.08.2015, at 13:37, Raphael Schweikert <notifications@github.com> wrote:
>
> Das sieht auch wieder nach einem Cherry-Pick statt einem Merge aus. Welchen Befehl hast du verwendet zum mergen? git merge?
>
> —
> Reply to this email directly or view it on GitHub.
>


Ich habe den Commit im release-1.5 gemacht. Dann git sync. Dann git checkout master und pull. Dann merge release-1.5


Sure. Already fixed.


Saw it, thanks!


Hm, seltsam


Wow have I given you merge rights?
Cool less work for me :)


This fails to compile in the 1.0.2 branch, because the "done" label does not exist.


Das müssen wir mal zusammen anschauen, da stimmt etwas mit deiner Konfig nicht.


I'm not the one to judge what version slick should use in 3.1. I'd suggest you to discuss it in the Slick mailing list. About play-slick, we'll use the same version slick uses.


I would change this to something like `Extension in "system/modules/xxx" does not exist`. That would be more obvious as a hint what's exactly missing.


@ExSport merged, maybe @SubJunk can handle the publishing?


fix_intel_compile_linux (mbadams5) - 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)


Sounds like we need to add version numbers to Mrc.py. I suggest we use the cockpit version and start at 1.0.


Thanks @ArthurHoaro


if fd is a negative error there's no need to call ceph_close on it


Ich glaube, du machst keinen Pull (des masters) vor dem Merge. Dann macht das `git sync` nachher einen Rebase und Rebases können mit Merge-Commits nicht richtig umgehen. Also immer: pull (oder sync) vor dem merge. Und nach dem Merge am besten nur push, nicht sync, dann siehst du sofort, falls jemand wieder dazwischengewerkelt hat.


fd is an error, no need to try to write to it and then close


Does it really make sense to support actions here if selecting an item anyway doesn't close the menu?


maybe add some other ops? mkdir?


I'll release it tonight


clean up the other files you created at the top?


`Ember.isArray`


Can you suggest what to do here? I'm hesitant to add new skin variables as I'm not sure how freely we want to add new skin variables.


clean up the other files too?


overall looks good!  does it run successfully?  :)


pentru ce dracu mai ai eachrile astea aici?


Ok, nach dem merge habe ich einen sync gemacht. Ich versuche, das in Zukunft nicht mehr zu machen!


why is this a backtick? :P


@robpaveza, it looks like a typo:
this works with `successCallback` instead.

Tested on Windows 10 Mobile, build 10512 - works well.


@SharkHunter your past mistakes have come back to haunt you :).  I thought you were too busy.


That makes sense, thanks. I was working with some older code that relied on the previous behavior. Now I understand what's going on.


Happy to merge this but I think we need some kind of comment to say the original implementation comes from Lin Shao.


You also changed the buttons in the header of settings views. For those buttons:
1) you removed the 'or' when there was one (that's fine, even if not in the scope of this task),
2) you renamed oe_highlight into btn-primary (this won't be necessary with the patch i'm currently writting), and
3) you changed oe_link into btn-default for the 'Cancel' button (that's not ok, we should keep oe_link i guess).

So please, revert at least 3), and 2) if you don't mind, to keep the diff minimal. Once you'll have done that, I believe that the PR will be ok. Thanks!


Yeah but I can contribue some at least


Der ist nicht so schlimm, wichtig ist es vor allem, *vor* dem merge einen Sync zu machen. Danach ist (bei unserem Repo) eher unwahrscheinlich, dass dir in der Zwischenzeit noch jemand hineinp(f)usht.


I'm a bit out of ideas as well. It works good enough for me now so...


faci show si apoi hide mai sus care e rostul?? ...


Moved to the wiki


> I still don't think there is value in that (you can see the output in the jenkins job trigger already as it goes on via stdout)

I disagree for [reasons I already explained](https://github.com/aspiers/automation/commit/28fde6c945ea128281b9120ac0ff40bdd176a0e8#commitcomment-12562351).

> Also, this logging on the fly makes it harder to rename the build log to build.err later.

I don't understand at all: why does it make it harder, and anyway why do we need to rename it?


Thanks, but I don't use bash, and it still would need me to type extra stuff.
I should see if I can upgrade portage so I get a repoman which does this automagically for me.


Cause its near to impossible to find symbols on Mac :D


trebuie sa ai un singur if aici care sa verifice daca exista websites daca exista atunci face append altfel da hide la elementu ala de websites si face append.

Tu ce ai acolo cu append in for-ul ala o sa iti adauge cate un link de add new site dupa fiecare website.. singura chestie care te salveaza e variabla aia dar e stupid puteai pur si simplu sa faci append dupa for in afara lui... Ce rost are sa faci append in for??


bad cherry-pick, about to fix, thanks.


@twinssbc No worries.

If you have a set of events in a scope, then change the event dates (e.g. through a `for` loop), the previous dates for that event isn't updated - and only is when the refreshView is called. It's not so much for adding new events (which are kept even after calling refreshView if they're in the scope), but more a fix for altering the events object (e.g if a user wants to edit an event period).

Let me know if I need to explain more :)

Jake


No that's not right, and [I already explained why](https://github.com/aspiers/automation/commit/e8288b2de4b2be0431d4086b15112ed2eb5f7298#commitcomment-12562513).


Now fixed in the devel branch, to get that version type:

```
library(devtools)
install_github("emilopezcano/SixSigma", ref = "develop")
```


you don't really need bash for that awk line. awk works in any shell and `$(foo)` is POSIX syntax


about to fix, sorry about that.


Were you able to fix this issue with golint?


@dotta gotcha. @szeiger Any reason not going with the latest version of HikariCP that works with Java 6, i.e. 2.3.9?


The contributors screen is refactored and uses the same layout as GDE. I refactored the list into PeopleListFragment. Gde lists, Translators and Contributors are all using the same. It is merged before the last release :smile: I forgot to delete this then.


+ifeq (metallium_d802,$(TARGET_PRODUCT))
+    PRODUCT_MAKEFILES += $(LOCAL_DIR)/metallium_d802.mk
+endif
+


Is "NONE" intended, what happens if you set "C" instead?


It is equivalent to adding `style="text-align: center;"`, which is even more prominent in our code base to be a bad practice, so I'd prefer not to change it.


+ifeq (metallium_d802,$(TARGET_PRODUCT))

    PRODUCT_MAKEFILES += $(LOCAL_DIR)/metallium_d802.mk +endif +


Hi,
I am able to login to the appfolio but iam unable to customize the UI. If we can do customization where we can write the code. Can you please help me over here.

Regards,
Satya.


Use `UINT8*` instead of `BYTE*`


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/44) page, and verify those files. If you have a question about this report, please contact @wforums.


Well it works, but because its on the localhost, the ip address returned wont be found, but when you push it to heroku, it will or at least should work.


Space is 32bit bitmask, so it would be 32 - 1, however it might be good idea to add this to wlc.h so it's not magic.


Maybe a note that due to api compatability issues we don't use the native crypto api?


1: M111 unrecognized component name: bibedit


I have feeling, all these above could be done better.


whitespace around =


## This is a markdown file


This bypasses the advantages you get from array observers.
```javascript
var value = this.get('value');
if (Ember.isArray(chosenValue)) {
    value.replace(0, value.get('length'), chosenValue);
} else {
    if (value.contains(chosenValue)) {
        if (value.get('length') !== 1) value.removeObject(chosenValue);
    } else {
        value.pushObject(chosenValue);
    }
}
```
(I haven't tested this, but you get the gist.)


We already know that it will also need to work for the SUSE-Cloud organisation: repositories such as `cct`, `supportutils-plugin-suse-openstack-cloud`, `cloud-tools`, `horizon-suse-theme`, `suse-cloud-upgrade` will all need to be part of the gate.  This commit is an important step towards allowing that, so what's the reason to exclude it?  In the worst case, someone manually invokes the script on an unsupported repository, and finds it doesn't work.  But then they'll probably realise there is very little work required to fix it so if we're lucky they'll submit a PR.


space after comma ','


This is on an outdated commit


Glad to help :smile:


This would be better implemented as a try/except to avoid the race condition of the file being deleted after the `os.path.exists()` call.
For example:
```python
try:
    cache_file = open(cache_file_path)
except IOError, e:
    if e.errno == errno.ENOENT:
        # file didn't exist
```


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

VSO State: Done


what is this _blame_ feature thing? I guess who introduce the changes... www.google.ca - like this preview functionality for the markdown....


Good catch, thanks.


My idea to support different configurations for different targets was to reset the rules here, but that requires manually clearing and restoring the default ruleset. It also requires storing the ruleset for each file, since `require` only loads and executes any given file – and thus only applies the file's rules – the first time the file is required.


Disabled test: case not yet supported


The change is okay (will merge it as it is), but the naming convention of dualprocshm-pcp.c and dualprocshm-c5soc.c is confusing me. Maybe prefix -pcie is the right instead of -pcp? Or why is there any difference at all between Nios in c5soc and pcie?


I think yes - at least in TR1. TR1 engine takes timer field to define a time interval on which Lara's look direction couldn't be changed from "look at" target. Best example is first "looking point" in Caves (where Lara stares at the passage where three bats are attacking her).

Seems that in TR2 the behaviour was changed, and it's now possible to break out looking direction any time, not regarding how much time has passed since Lara arrived in "look at" trigger sector.


teste


Hey @giovannelli. Thanks again for submitting this PR. I'm a little confused about why you're removing this SSL stuff?
TLSv1.2 is typically considered more secure than SSLv2.3, which is what we fall back to if we remove this code (while using rest-client 1.8.0).


:name would be enought here


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
149:4: E131 continuation line unaligned for hanging indent
176:54: E261 at least two spaces before inline comment
176:55: E262 inline comment should start with '# '


Just `describe`


HomeWork


Out of interest, I assume generally operations are async in nature for Realtime, and even for REST I understood, however this method definition does not appear to offer this asynchronously.  Have I misunderstood this?


```ruby
return if !name || skip_access
 ```


Move it to rails_helper


24:1: E402 module level import not at top of file
25:1: E402 module level import not at top of file
26:1: E402 module level import not at top of file
27:1: I101 Imported names are in the wrong order. Should be CFG_BIBSCHED_LOGDIR, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_PYLIBDIR
27:1: I201 Missing newline before sections or imports.
27:1: E402 module level import not at top of file


This can be done more clean by modifying `TR_FD_TRIGFUNC_CAMERATARGET` switch case in `TR_Sector_TranslateFloorData` function. Just replace `single_events += buf;` line for this case with `cont_events += buf;` line. Then script function will be called every time trigger is called.

The problem here, is camera won't be updated *every* frame, instead it'll be updated on fixed 1/60 time interval, which probably will result in flickering between Lara camera and fixed camera. You need to introduce a flag which will specify that camera should not be switching back to Lara, and this flag must be set inside `lua_setFixedCam` function. Then this flag must be reset somewhere in the finalizing code of game loop on fixed-step basis.


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
144:16: E126 continuation line over-indented for hanging indent
189:15: E126 continuation line over-indented for hanging indent
192:1: E402 module level import not at top of file
223:17: E203 whitespace before ':'
224:20: E203 whitespace before ':'
225:18: E203 whitespace before ':'
226:22: E203 whitespace before ':'
227:20: E203 whitespace before ':'
228:19: E203 whitespace before ':'
229:20: E203 whitespace before ':'
230:21: E203 whitespace before ':'
231:37: E203 whitespace before ':'
245:4: E131 continuation line unaligned for hanging indent
274:1: W391 blank line at end of file


OK, but that's the recommended usage.  Otherwise a lot of the benefits are lost: https://hub.github.com/


Thankfully some kind folks over the years have added to the pom.xml so that hopefully it works well. I will continue using the ant build, but that certainly doesn't mean anyone else has to.


Looks like *scrubetc.iso* is a typo.


How we could do CSwordModuleSearch::unindexedModules(modules())? We cant cast from QList<const CSwordModuleInfo*> to QList<CSwordModuleInfo*>


Yes, it should be, there is no need to ever hard code API keys


looks weird :) windows/...noos.c

Does it make any difference to the Windows side if the FPGA has an OS or not?! I can say from the FPGA side it doesn't matter, what is your view?


Much better! lgtm


Why `0` ?


Why you overwrite this with the same value?


Please, when I'm working on it don't also modify the code ...
I have a working patch for it, but I need to discard your changes.


Should this be async? The rest client is doing this synchronously, that's why i did it this way.


Move to spec/factories and create separate file for each factory.


Great, THX guys!


Why is this on separated commit? Isn't better to be squashed on previous one?


1. I don't think you'd have problem with import and shading since you're going to depend on the internal version inside the bundle (assuming you've changed the package name of the original lib). This is a case also where you'd want to explicitly tell what you want to export since you want to export your package and not all the packages inside the shaded jar. In those cases you'd use this snippet (eg jackson-databind)
                    <Export-Package>
                        com.fasterxml.jackson.databind.*
                    </Export-Package>

2. You can use the excludeDependencies (http://felix.apache.org/documentation/subprojects/apache-felix-maven-bundle-plugin-bnd.html)  keyword on the plugin to exclude those dependencies that you don't want. But for simplicity's sake, if it was me I'd use the plugin's automatic detection where no additional configuration would be needed and leave it like it is where more configuration is necessary, since you have already a working solution otherwise.

I'm not knowledgeable enough about all projects from fasterxml to say which ones would benefit from this simplification but the rule of thumb for me is, if there's no compile-time only dependencies or shading, you'd probably be able to use the automatic detection with no need to further configuration. The biggest advantage is that you don't need to maintain the package list on the imports and exports. Only other thing that I recall which matters is this:  if you need to add a dependency which happens only at runtime, you have to add it explicitly on the import-package section, e.g:
						<Import-Package>
							*,
                                                       com.acme.coyotte.*
						</Import-Package>

In this case, you had to add com.acme.coyotte.* because you don't depend on it on compilation but only on runtime (typical for loaded classes).


1: M111 unrecognized component name: dojson


Look at `= render foods` (it will require extra partial)


`s/paths/['paths']/` and we're GTG!


Ok, but I would have though that for the Realtime library we would expect callbacks / events as a default, and for REST the opposite.  See https://github.com/ably/ably-ruby/blob/master/lib/ably/rest/client.rb#L185-L192 for REST time and https://github.com/ably/ably-ruby/blob/master/lib/ably/realtime/client.rb#L98-L107 for Realtime time in the Ruby library.  Would you not expect all Realtime operations to return immediately?


Hi,
Can we customize the appfolio from our local. If we can where i have to write the custom code. Can you help me over here.


What about *lock-dualproc.c* without any *os/noos* prefix?


Hi @dugjason,
you are right Jason, I removed it for testing purpose cause I got this error:

ConnectionError: : Failed to complete SSL verification (Network error: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed)

I've upgraded my OpenSSL library, which was the cause of the error, but I forgot to enable the default ssl on the gem.


If we have
`BtModuleList CSwordModuleSearch::unindexedModules(BtModuleList &modules);
...
BtConstModuleList unindexedModules(CSwordModuleSearch::unindexedModules(modules()));`

we are still unable to `indexAllModules(unindexedModules)` (`BtModuleIndexDialog::indexAllModules(const BtModuleList &modules)`)


Thanks!


A panel.conf -ra ven egy értekezés, még nyitott:
https://github.com/lxde/lxqt/issues/385


Hi, only a personal preference here. I prefer to depend on projects (libInternationalization) instead of jas (gvsig-i18n.jar). It's a bit easier to forgive to push the jar and if someone makes changes in libInternationalization and not push the jar (other developer maybe doesn't have libInternationalization in her workspace) the jar will be not up to date. Depending in the project makes that the workspace have to has all the real dependencies.

And, when you depend on projects it's easier to make changes on gvSIG to make the "git status" not throw a lot of "not really modified jars". We can talk about this offline.


Hrm, your talking about doing the setup with the bat files again? http://kodi.wiki/view/HOW-TO:Compile_for_Windows#Setting_up_the_build_Environment
Not doing a Rebuild on the Solution level? (as that doesn't work for me)


@brooksreese I understand the idea but :-1: on this because the original meaning is gone.


Why do you think `CSwordModuleSearch::unindexedModules(modules())` requires casting from `QList<const CSwordModuleInfo*>` to `QList<CSwordModuleInfo*>`?


Changed in 8d27336bbbf5180efddbf6b0579f5927c3740843.


As3 uses writeObject/AMF3 here... Will this be a place where the platforms differ?


On commit message:
s/where/were
s/discarting/discarding


yup, you need to execute those bat files again.


Took it from pypi because you used that as URL (and similar packages take it from there), but personally I'd prefer GitHub as well.


Note: File name discussion in zelenkaj/openPOWERLINK_V2#8


1: M111 unrecognized component name: dojson


1. No description section for the trace subroutine. You need add somethng to describe the arugments and example of how to use trace subroutine.
2. The indent of code has problem.
3. What if the user does not input 'I','D','W','E', for $level?


You copy pasted the bacon make right fix the typo


1: D100 Missing docstring in public module
21:1: F401 'bd69x' imported but unused
21:1: F401 'bd01x09x' imported but unused
21:1: F401 'bd2xx' imported but unused
21:1: F401 'bd9xx' imported but unused
21:1: F401 'bd8xx' imported but unused
21:1: F401 'bd7xx' imported but unused
21:1: F401 'bd5xx' imported but unused


Sorry. I should've asked you first.


23:1: I201 Missing newline before sections or imports.
70:1: E302 expected 2 blank lines, found 1


67: D401 First line should be in imperative mood ('Thesauru', not 'Thesaurus')


30: D400 First line should end with a period (not 'e')


23:1: I201 Missing newline before sections or imports.
126: D401 First line should be in imperative mood ('Statu', not 'Status')


> This is probably critical for some business, but
as I don't know which ones, let's remove it.

This doesn't sound right :confused:


Yeah that sounds great!  with a couple inline comments


```
let(:admin) { create(:admin) }
let(:api_meal) { attributes_for(:api_meal, user_id: admin.id) }
```


It's unnecessary as they end up being owned by root:root anyway, and it's usually frowned upon using it nowadays, too. Mode/user/group should be set in %install if necessary.


So is that an extensions and MPDN release tonight?


Please follow the commit message format consistently: https://wiki.gentoo.org/wiki/Gentoo_git_workflow#Commit_message_format


This is related to potential changes in protocolv3. For current protocol
it's an arbitrary selected number.


Messed up, don't use space instead of "+", use a dot "."


Found the issue. I'll push the changes to the server soon.


Well if everything goes well... It's getting late here that's why I was trying to quickly patch it up so I could do a release. Otherwise it'll have to wait till tomorrow...


:clap: :clap: :clap: :clap: :clap: :clap: pr


Can you please check which of these CONFIG_DLL_* macros are really needed at app side.
(I assume at least the CONFIG_DLL_PRES_* only live in DLLK)


The original signature for that method was:
```C++
static QList<const CSwordModuleInfo*> unindexedModules(const QList<const CSwordModuleInfo*> &modules);
```
in cswordmodulesearch.h:103-104. Why change the argument type?


This comment seems to be invalid. If "equivalent" means "having the same value", then `ax == az` would be true, but the condition above (line 118) also establishes that `ax == ay`, which means `ay == az`. And that (`ax == ay && ay == az`) is already checked at line 96. This branch here would never be reached!


Seems like a great step in direction of #157, right? ;-)


I want to know how you sovled rest-auth cutom user issue. Email me amitt.bhardwj@gmail.com


Wait. This patch add a breaking change.
Click on the cross of the SimpleUpdateForm -> MPDN crash.
The form is entering in loop because when closing the form push the cancel button already. Which trigger the closing, with trigger the cancel button etc ...


I never said my shell was POSIX :)


That's the kind of reasoning that leads to lobotomies...


Could do with adding docstrings to all of these functions.


Maybe you want to use `max_column_scale` instead of `max_box_scale`.


Yes there was another patch immediately following this one... but that was before I saw your comment. :P


@hasufell thanks, sorry for that.


I think that `user_id 0` is redundant here. How code that you just copied, is related to `:meal` factory?


fixed and repushed !


Sorry, @masatake -- brain fart!


It seems that left clipping bit is on the right and right clipping bit is on the left. right ?
Shouldn't they be inversed ? (CLIPPING_RIGHT = 1 and CLIPPING_LEFT = 2)


@hasufell okay. thanks for info


If `CONFIG_USE_SYNCTHREAD` is not defined, this typedef is useless, right? Maybe guard it with `#if defined(CONFIG_USE_SYNCTHREAD)` ?


looks good :)


Why did you choose this way?
IMO this is only confusing because the placeholder in JSON work differently than Texts.format.
Texts.format replaces the entire placeholder and all children with the replacement with the new value. Or shows itself and all its children as fallback.
This one here only uses the first child as fallback.
IMO this makes this less easy to configure. Because you cannot see what will be replaced on the first look on the raw json.


I think the `sudo apt-get install -y gifsicle jpegoptim optipng jhead` line is not needed if you use the new syntax


I like it :) On the right track...


Same here with `CONFIG_USE_SYNCTHREAD` ?


Well, I believe we might advice the user some sane default, like:

```Consider adding "unattended_install.import.import.default_install.aio_native" (or similar) test as first one and/or "remove_guest.without_disk" test as last one to remove it after testing.```


Ok, I've seen now commit b9b1f8c23f6836ee584207a8e6d7c21e302cc1d7 :smile:


I am not sure. Just checked the java implementation and it's also synchronous :/


What happens if we change URLs to maybe a different service with different information (but conveniently the same website layout..)? Then we'd have to tie the cached information to a specific URL so that we don't return cached results across different datasets.


Just recognized in system-linux.c that the sync thread stuff is always guarded with `CONFIG_USE_SYNCTHREAD` - does it make sense for system-windows.c too?


why do you use 'xcatcport' instead of 'xcatlport'? 'l' means 'log'.


Could you remove the useless comments?


perlfunc(1) says:

```
           The return value is the exit status of the program as returned by
           the "wait" call.  To get the actual exit value, shift right by
           eight (see below).  See also "exec".  This is not what you want to...
```

In this case, I didn't care what the actual exit value was -- just if it was 0 or not (hence, wait(2) returns 0 and is and'ed with the return status of 0).

```shell
$ cat foo.pl
#!/usr/bin/env perl
use strict;
use warnings;
my $rc = system("mkdir /this/will/not/work");
print "RC is: $rc\n";
$ ./foo.pl
mkdir: /this/will/not: No such file or directory
RC is: 256
```


It should actually be othis.bimServerApi.schemas[othis.schema][typeName], thanks will check that in


indent problem


This is almost certainly wrong, `cur.z` would be incremented twice in one iteration (here and on line 113). Consequently `cur.y` is not changed at all.


Okay! I'm just curious about why it's locked LK_EXCLUSIVE. Probably we should ask mjg@ about this part of code.


I guess you don't need to write *CFG_KERNEL_STACK_DIRECTLINK* in *ENDIF* part again?


Did not see your cmdlog process handle the 'INT' or 'TERM' to stop gracefully.


This behaviour is not consistent with the other functions. We need to think what is a good way to deal with null values here... It is similar to the problem with empty strings...


Add a comment to explain why a Django TestCase is needed? Is it only for Python 2.6?


It's not really possible to asses the changes in this pull request as the formatting is all changed


Aha ok. Thanks :)


I plan to finish it when I have time, but I don't have time at the moment.


Is this point in commit message still part of this commit?
*- Update Windows driver project to add install path.*


`serialVersionUID`


use backtick syntax then


Merge done


For all the options relying on a boolean argument you could do something like this:

```
'--auto-start=[Auto-start destination service]:boolean:(1 0)'
```

This is how it is handled in the other completion files.

And for the --timeout option you could also add ':timeout (seconds)' at the end, otherwise we don't what is the unit.


Well actually I thought it was pretty clear in the message and commit that this wizard does nothing the chatter can't do. Aka posting a message. Moreover it uses a subtype that does not exist, leading to a note that is not sent to followers, ... So it is not critical, not necessary to keep, nothing is lost, except the possibility to do the same thing using 2 different ways.

For lobotomies, I don't know. It's a tough question actually.


Working, thx.


using the built-in one caused ``TypeError: unbound method test_pyflakes() must be called with AboveStaticAnalysisCodeTest instance as first argument (got nothing instead)``, see https://travis-ci.org/divio/django-cms/jobs/75417249


1: M111 unrecognized component name: legacy


I think the thumbnail background gradient looks a bit closer to the design, by changing that background, and not the dimmed grey of the inactive tab, I think it makes it a bit more clear which one is highlighted:

![image](https://cloud.githubusercontent.com/assets/102265/9274135/f97dd998-4260-11e5-8aa6-f5d9a9ace80f.png)


Have a look at [this SO question/answer](https://stackoverflow.com/questions/273192/in-python-check-if-a-directory-exists-and-create-it-if-necessary) about creating directories like this and avoiding race conditions.


I didn't see it was missing from the .bat. I thought you were readding
MinGW's mp3 again.

On 14 August 2015 at 07:37, jb-alvarado <notifications@github.com> wrote:

> Without this I was not able to compress with -c:a mp3, so I guess ffmpeg
> has no mp3 encoder by it self.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jb-alvarado/media-autobuild_suite/commit/703a616b38035a43411f30b38dfb21a19bd144a9#commitcomment-12704703>
> .
>


This change seems to result in
```
local:
    - Rendering SLS 'base:rabbitmq' failed: Jinja variable 'pkgs' is undefined
```
After reverting this change, it seemed to work fine again


Could you make it conditional on the Python version so we can drop it with Python 2.6 support is dropped?


Another nice use of `any()`!


So when i build the project
```
npm install -g grunt-cli
npm install
grunt build

```

how can i add source map? :beer:


Curious: why did these lines need to be commented out? Perhaps the input file was malformed, causing a runtime exception?


@KrishnaRamamoorthy just wondering: is there a difference between `varchar` and `varchar`2? Because from what I can make out online, they're currently synonyms, except that `varchar` is going to be used for something else in the future so Oracle recommends that you use 'varchar2'.  (We're using MySQL/SQLite, so not sure how that'll affect us.) However, the term `varchar` seems to be more widespread in forums and documentation than `varchar2`.

Also, in Oracle there's some difference in how it stores null values ("`varchar2` considers a blank string as null"). I don't know if other databases like MySQL follow the same pattern. Is this difference significant or does it not make a difference as to which one we use?


new function structure for write_flash_float


I think you need sleep for a while before trying immediately


yeah, thanks. This should help the newbies until we find a better way.... (TODO card added)


Sorry, I remember that you said we should use `varchar2` but I'm not sure whether you meant as opposed to `char` or `varchar`.


nit: extra linefeed at EOF is missing. See the red marker. It is required by git to keep the EOFs consistent.


Duplicate `pre.footprintpredict.com`.


thanks! :)


is there a problem with using the django test case over unittest even on py2.7+?


changed the write_flash_float function to write all variables at the same time as it is not possible to write to one location more then once.
Closed the Flash again after writing by: Set LOCK bit and clear WRT bit.


sounds good!


Changed the prototype of the function


Yeah I ended up just going with that approach, thought this would be more "elegant"... but it wasn't meant to be.


changed the 3 write_flash_float functions to the new one


Shouldn't this be ```quant_tuple[0]```? Also, please add context


wrong name


It has been changed to 1.4.2.
https://github.com/JetBrains/kotlin/commit/50c93019c41672994057320852df5b2ecc478065


:+1: More secure indeed. Alternatively, you could have replaced 0.0.0.0 by 127.0.0.1 to only allow local connections, that's what I did on my repo :). :gift:


typo ?


Why ```not in```? Shouldn't ```lots``` contain the available lots?


test http name


It creates an unnecessary database transaction.


Need a description for the subroutine


thx


The way I got around this in recode was with a messy bunch of cfg files.

I had a chat.cfg that would unbind enter/esc from a/b, run chat, and bind enter to "execute rebind.cfg"
So it would send the message, and trigger that cfg to rebind enter/esc to a/b.

Even doing that programmatically would be ugly... but it works.


(Okay, it's not a fix, I just didn't know what I changed) ;)


May be move this on top of function. First clear any pending interrupts, then enable global interrupt reg.
In this version if there any pending intterupt, it could be services before this point


OK, the implementation seems simple enough.


Why didn't you modify displayDateTime to be this? If we want consistent time representation, then we should have a function that displays the date and time.


Sections needs to be uncommented


In groovy, `java.io.*` and `java.util.*` are always implicitly imported, so these imports are unecessary


can this be simplified?


Definitely options, I just think we want to be explicit about the versions so we know what we have.


Nice! Thx.


Thanks for confirming. I'm making some changes, so will put this back in with one of my commits


These three lines would be more concise as `def representation = [success: 'true', preference: null]`


those falses are now zeroes


should be `true` not `'true'`


good point, thanks.


That's one sweet looking beast.


Sure


another option besides constructing and writing the map like this would be to simply hardcode the JSON string and write it out directly.


Clever.


need to make it more open source


Codestyle mandates 'egypt' bracket style https://code.google.com/p/embox/wiki/CodeStyle


OK


Thanks for the credit,

Gabriel Vermeulen
TEHTRI-Security


Please update the same in bower.


Why do you use "PRIVATE" instead "PUBLIC"?


Stupid checkbox hack! Nobody will understand that! Seriously!


You can remove this – I'll update the changelog when I cut the relase


Fair enough. You are specifying `--innter-runner-chdir=test`, but never explains the base dir where the tests are.


Yep, good check


Done.


So, when the user selects a variable in the Variables view, can't you also know it's ordinal, in case there is one? I think  you save the ordinal together with the value, in `AsyncLocalVariable`. So, maybe it's not too hard to find the exact ordinal of the variable there.


nice cat


nit: prefer a more descriptive name for this variable


This could be improved, yes, so VLC-Qt would depend on Qt4 (or 5), but not on libVLC (as you usually don't need to link with it again).


The NEW behavior of a policy CMP0023 is to not to allow mixing of the keyword and plain signatures of target_link_libraries for single target


I think you should return true here.


thanks!

actually, I think that that dependency is wrong, it should only be on bitcore-wallet-client, all this is still WIP :)


что с отступами?


What is the purpose of ```j``` here?  It's initialized to 0 and then never changed.


@jbrooksuk why do you need `name` for the token :) ?

Maybe better track `created_at` and `expires_by`


What's wrong the example buttons? I don't mind if we get rid of them, but those are part of the rubric item type previews we talked about with Pamela.


You might want to check the return value of this function.


We're passing a timeout of -1, so I believe poll should never return 0.  Might be good to assert that.


测试


@n0mer essentially you're able to name the "application" which is using the token.


looking for consistency here with the query-refactoring branch. Should we rename to innerHashcode and innerEquals there or use doHashcode and doEquals here?


I see; you're not incrementing ```j```, but you're decrementing ```numFds```?  But it looks like you're only decrementing it in the case of ```i == 0```.  I thought at first that you were using this as a way of exiting the loop early once you found all of the descriptors that had been signaled, but in that case I'd think you'd need to update numFds for ```i > 0```, too.


@nrc yeah that comment is basically completely wrong! I must have copy-and-pasted it from somewhere.


I have added comments here but in future I will add them under line where you need change something.

1. Class and file should be from the capital letter.
2. "C:\\d1\\(copy)" - should argument. Not hard coded.
3. In catch shouldn't be logic. It is bad approach do programming logic with try-catch. Read about it.
4. Read about inner and nested classes.
5. put every class in separate class.


Awesome.


editing Jodegithub


appreciated :+1:


@mfsny you missed to change the test name. I did it now :)


Missed this one, can you please open an issue, so I don't forget or open a PR to fix it?


Hello. Excuse me.It will support z77x-D3H? (rev. 1.2) Thank you
Sorry for my bad English.


@jsquyres, yes, you are right, sorry about that.


`Task.ConfigureAwait` does not modify the task, but returns a `ConfiguredTaskAwaitable` which wraps the task in an awaiter that does not capture the current context. Currently, the `ContinueOnAnyContext` extension method is a no-op, and all tasks are capturing the current context.


```
In [1]: class A(object):
   ...:     @classmethod
   ...:     def new_a(cls):
   ...:         return cls()
   ...:

In [2]: a = A()

In [3]: a.new_a()
Out[3]: <__main__.A at 0x7f4d0f3e1390>
```


Why not : `try conn.close() catch { case NonFatal(_) => }` ?


Nope it was nothing to do with the org change.  It's to prevent directories like this:

http://tu-sle12.j.cloud.suse.de/mkcloud/barclamp-crowbar:1348:ac029ed52cda571f13dadf88f75c93774344013e:release/

which causes the PTF directory hierarchy to have multiple inconsistent levels of depth.


this doesn't test the `width` keyword at all


...though I guess there is no need since that's just passed to another function


Good catch! I'll fix this.


@DanEEStar gibts die funktion commit_managed()? Wie es aussieht gibts das nicht..


@mindjuice FYI—these are intentional. ;-)


You should not alter codding style here.


If I'm understanding the loop correctly, I think it'd be easier to understand if it were instead written something like this:
```C#
// Check the special descriptor.  If it's set, read from it to remove the data.
if (fds[0].revents == 0)
{
    if (ReadSpecialFd(fds[0].revents < 0)
        throw new InvalidOperationException(...);
    numFds--;
}

// Find all of the remaining set descriptors
for (int i = 0; i < count && numFds > 0; i++)
{
    if (fds[i].revents == 0)
    {
        if (retVal == null) retVal = new List<int>();
        retVal.Add(fds[i]);
        numFds--;
    }
}
```
Note that I'm also suggesting here that we lazily-allocate the ```List<T>``` only when we know there's data to be returned, as it seems like it could be relatively common that there won't be, either due to cancellation or because only the special descriptor was signaled, or there was some error.


Why do you check `sniffs` for being `object`? It should be an array not an object.


@Rashed97 simple isn't it? :smile:


It would be better to enclose sniffs list into quotes marks to get something like this `--sniffs="first,second,third"`


Again, you should not alter codding style here.


Could `failForResponse` also handle the `response == nil` case and then you'd only have to call it once at the very beginning and only handle the error case and success case afterwards?


@xhaggi Is not breaking long parameter list into new line an agreed upon style? If yes, what is the agreed upon style? When that exist and it's written down, I'm more than happy to follow it, to the point. Otherwise it's a subjective opinion and personal preference, and 2 minutes later someone else comes by says "too long line, please break line".


1: M111 unrecognized component name: legacy


Coding style should be untouched.


1: D205 1 blank line required between summary line and description (found 0)
27:1: I201 Missing newline before sections or imports.
30:1: I101 Imported names are in the wrong order. Should be escape_shell_arg, run_shell_command
31:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.system should be before from invenio.utils.shell
34:1: I101 Imported names are in the wrong order. Should be CFG_BIBCATALOG_SYSTEM, CFG_BIBCATALOG_SYSTEM_RT_CLI, CFG_BIBCATALOG_SYSTEM_RT_DEFAULT_PWD, CFG_BIBCATALOG_SYSTEM_RT_DEFAULT_USER, CFG_BIBCATALOG_SYSTEM_RT_URL
34:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.dbquery
35:28: E126 continuation line over-indented for hanging indent
41: D400 First line should end with a period (not 's')
50:9: B901 blind except: statement
55: D101 Missing docstring in public class
57:30: E261 at least two spaces before inline comment
59: D400 First line should end with a period (not 's')
112: D401 First line should be in imperative mood ('return', not 'returns')
112: D209 Multi-line docstring closing quotes should be on a separate line
112: D202 No blank lines allowed after function docstring (found 1)
112: D400 First line should end with a period (not 'y')
112: D205 1 blank line required between summary line and description (found 0)
112: D208 Docstring is over-indented
118:26: E261 at least two spaces before inline comment
172:46: E261 at least two spaces before inline comment
190:46: E261 at least two spaces before inline comment
194: D102 Missing docstring in public method
230:5: E303 too many blank lines (2)
230: D401 First line should be in imperative mood ('create', not 'creates')
230: D400 First line should end with a period (not 'e')
263:19: E127 continuation line over-indented for visual indent
277: D400 First line should end with a period (not 'e')
287: D400 First line should end with a period (not 'e')
291: D400 First line should end with a period (not 'e')
299: D400 First line should end with a period (not 'e')
336:13: F841 local variable 'dummy' is assigned to but never used
350: D400 First line should end with a period (not 't')
357:56: E251 unexpected spaces around keyword / parameter equals
357:58: E251 unexpected spaces around keyword / parameter equals
357: D209 Multi-line docstring closing quotes should be on a separate line
357: D205 1 blank line required between summary line and description (found 0)
357: D208 Docstring is over-indented
426:37: E261 at least two spaces before inline comment
428:18: E261 at least two spaces before inline comment
437: D400 First line should end with a period (not 's')
441: D401 First line should be in imperative mood ('Thi', not 'This')
441: D400 First line should end with a period (not 'd')
441: D205 1 blank line required between summary line and description (found 0)


1: D400 First line should end with a period (not 'k')
4: L102 copyright year is outdated, expected 2015 but got 2013
26:1: I100 Imports statements are in the wrong order. import getopt should be before import sys
30:1: I101 Imported names are in the wrong order. Should be task_get_option, task_init, task_set_option, task_sleep_now_if_required, task_update_progress, write_message
36:1: I101 Imported names are in the wrong order. Should be CFG_PYLIBDIR, CFG_VERSION
36:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask
39:1: I101 Imported names are in the wrong order. Should be fetch_last_updated, split_ids, store_last_updated
46:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.api should be before from invenio.legacy.search_engine
48:1: I101 Imported names are in the wrong order. Should be get_all_modified_records, get_all_new_records
48:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.dblayer should be before from invenio.legacy.bibcatalog.utils
55: D203 1 blank line required before class docstring (found 0)
55: D400 First line should end with a period (not 's')
59: D203 1 blank line required before class docstring (found 0)
59: D204 1 blank line required after class docstring (found 0)
59: D200 One-line docstring should fit on one line with quotes (found 3)
63: D102 Missing docstring in public method
70: D102 Missing docstring in public method
77: D401 First line should be in imperative mood ('Submit', not 'Submits')
86:51: E126 continuation line over-indented for hanging indent
93: D401 First line should be in imperative mood ('Doe', not 'Does')
93: D400 First line should end with a period (not '?')
106: D210 No whitespaces allowed surrounding docstring text
106: D400 First line should end with a period (not 's')
106: D205 1 blank line required between summary line and description (found 0)
155: D210 No whitespaces allowed surrounding docstring text
155: D400 First line should end with a period (not 'k')
239:42: E128 continuation line under-indented for visual indent
256:18: E128 continuation line under-indented for visual indent
292: D401 First line should be in imperative mood ('Generate', not 'Generates')
332: D400 First line should end with a period (not ',')
332: D205 1 blank line required between summary line and description (found 0)
368: D401 First line should be in imperative mood ('Construct', not 'Constructs')


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2014
29:1: I100 Imports statements are in the wrong order. import os should be before import sys
33:1: I101 Imported names are in the wrong order. Should be task_get_option, task_init, task_set_option, write_message
34:29: E126 continuation line over-indented for hanging indent
35:1: I101 Imported names are in the wrong order. Should be CFG_INSPIRE_SITE, CFG_REFEXTRACT_TICKET_QUEUE, CFG_SITE_SECURE_URL, CFG_TMPSHAREDDIR, CFG_VERSION
35:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask
36:28: E126 continuation line over-indented for hanging indent
43:1: I101 Imported names are in the wrong order. Should be DESCRIPTION, HELP_MESSAGE
44:1: I101 Imported names are in the wrong order. Should be FullTextNotAvailable, check_record_for_refextract, extract_references_from_record
44:1: I100 Imports statements are in the wrong order. from .api should be before from .cli
47:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from .config
47:1: I201 Missing newline before sections or imports.
48:1: I101 Imported names are in the wrong order. Should be split_ids, task_run_core_wrapper
50:1: I100 Imports statements are in the wrong order. from invenio.legacy.docextract.record should be before from invenio.legacy.docextract.utils
52:1: I101 Imported names are in the wrong order. Should be field_get_subfield_values, record_get_field_instances
52:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine
54:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcatalog.api should be before from invenio.legacy.bibrecord
57: D210 No whitespaces allowed surrounding docstring text
57: D400 First line should end with a period (not 's')
57: D205 1 blank line required between summary line and description (found 0)
74: D210 No whitespaces allowed surrounding docstring text
74: D400 First line should end with a period (not 'k')
135: D103 Missing docstring in public function
197: D103 Missing docstring in public function
226: D103 Missing docstring in public function
241: D401 First line should be in imperative mood ('Construct', not 'Constructs')
246:9: E128 continuation line under-indented for visual indent
247:9: E128 continuation line under-indented for visual indent
248:9: E128 continuation line under-indented for visual indent
249:9: E128 continuation line under-indented for visual indent
272:9: E128 continuation line under-indented for visual indent
273:9: E128 continuation line under-indented for visual indent
274:29: E127 continuation line over-indented for visual indent
291:9: E128 continuation line under-indented for visual indent
292:9: E128 continuation line under-indented for visual indent
293:9: E128 continuation line under-indented for visual indent


23:1: F401 'os' imported but unused
26:1: F401 'cfg' imported but unused


25:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine


32:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine
41:1: I100 Imports statements are in the wrong order. from tempfile should be before from invenio.legacy.search_engine


I suggest asserting ```Debug.Assert(Monitor.IsEntered(this));```.


if we already ... what?


Thanks for the catch, fixed now.


Do you need to check/validate the return value?


fixed, now should be clear :)

2015-08-14 15:14 GMT+02:00 Iulian Dragos <notifications@github.com>:

> So, when the user selects a variable in the Variables view, can't you also
> know it's ordinal, in case there is one? I think you save the ordinal
> together with the value, in AsyncLocalVariable. So, maybe it's not too
> hard to find the exact ordinal of the variable there.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wpopielarski/scala-ide/commit/658c739d8bd24cf7994247fcce3a970473c26b48#commitcomment-12709954>
> .
>


Hi @spmaniato

There is some relationship to issue #5 - when using the plugin, the system may no longer choose output variable valuations such that the next state is a dead end for the environment player. So dead-ends do not occur in the strategy any more. But this is just a corollary of using the plugin: the aim of the plugin is to prevent the synthesized system from falsifying the environment assumptions. The technique is described in the paper "Synthesizing Cooperative Reactive Mission Plans" that will appear at IROS 2015, but the paper cannot be found online yet. If you want, I can send you a preprint, though.


Is this going to capture all use cases of the returned insert id? If I don't know what the id of the row I've just inserted is and get returned 0 because it is the same as a the previous last inserted id, will that returned value be useful to me?


You welcome!
:)


Это уже убрано в следующем коммите


html


:cat:


due to grafana log.Close bug on shutdown


Remove the `-`? The format is `<type> - <description>` If you want to change that, it should be done everywhere


agree, so maybe better rename it , to reflect this.

it's up to you, thanks for clarification


I'm having a difficult time answering your original question. Shotgun approach.

 `/piece/[pk]/?format=json` serves the json content regardless of user status.
 `/piece/[pk]/update/?format=json` serves the same content as the above, if the user is allowed to update the piece - otherwise it redirects to an HTML 403 page.

I think what you were saying is that serving an HTML 403 page when the client requested json is not good - yes?


Its not same, there is no service library here,

Thanks
Ranjan
Sent from my iPhone

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

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.

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


I think most of the code in this subroutine comes from client.pm->handle_response(), right?

I did not put effort to review this code, it should work:)

please clean up the indent.


Ummm.....
460MHz????
We don't have that frequency defined iirc


For sake of keeping functionality the same, I left this in here, but I would like to take this overwriting of `standardDataDisplay` out, as it leads to slightly unintuitive behaviour on window resizing, and also looks ugly.


First point (several InputFile created to read multiple audio stream from the same media)is done here: bfcf05e


@Ninikitos DWMR. Nice job using `ceilf()`


just add comment in commit


I passed the container in, and accessed the SVGs from within it. This seemed the best way at the moment.